Skip to content

Commit b90cd91

Browse files
committed
Merge branch 'PHP-7.2' into PHP-7.3
* PHP-7.2: Fix #41997: SP call yields additional empty result set
2 parents d66e0f1 + 41a4379 commit b90cd91

File tree

5 files changed

+5
-8
lines changed

5 files changed

+5
-8
lines changed

NEWS

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,9 @@ PHP NEWS
3030
- ODBC:
3131
. Fixed bug #78473 (odbc_close() closes arbitrary resources). (cmb)
3232

33+
- PDO_MySQL:
34+
. Fixed bug #41997 (SP call yields additional empty result set). (cmb)
35+
3336
29 Aug 2019, PHP 7.3.9
3437

3538
- Core:

ext/pdo_mysql/mysql_statement.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -423,14 +423,14 @@ static int pdo_mysql_stmt_next_rowset(pdo_stmt_t *stmt) /* {{{ */
423423
pdo_mysql_error_stmt(stmt);
424424
PDO_DBG_RETURN(0);
425425
} else {
426-
PDO_DBG_RETURN(pdo_mysql_fill_stmt_from_result(stmt));
426+
PDO_DBG_RETURN(pdo_mysql_fill_stmt_from_result(stmt) && stmt->row_count);
427427
}
428428
#else
429429
if (mysql_next_result(H->server) > 0) {
430430
pdo_mysql_error_stmt(stmt);
431431
PDO_DBG_RETURN(0);
432432
} else {
433-
PDO_DBG_RETURN(pdo_mysql_fill_stmt_from_result(stmt));
433+
PDO_DBG_RETURN(pdo_mysql_fill_stmt_from_result(stmt) && stmt->row_count);
434434
}
435435
#endif
436436
}

ext/pdo_mysql/tests/bug_39858.phpt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,6 @@ if ($version < 50000)
1818
die(sprintf("skip Need MySQL Server 5.0.0+, found %d.%02d.%02d (%d)\n",
1919
$matches[0], $matches[1], $matches[2], $version));
2020
?>
21-
--XFAIL--
22-
nextRowset() problem with stored proc & emulation mode & mysqlnd
2321
--FILE--
2422
<?php
2523
require_once(dirname(__FILE__) . DIRECTORY_SEPARATOR . 'mysql_pdo_test.inc');

ext/pdo_mysql/tests/bug_41997.phpt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
--TEST--
22
PDO MySQL Bug #41997 (stored procedure call returning single rowset blocks future queries)
3-
--XFAIL--
4-
nextRowset() problem with stored proc & emulation mode & mysqlnd
53
--SKIPIF--
64
<?php
75
require_once(dirname(__FILE__) . DIRECTORY_SEPARATOR . 'skipif.inc');

ext/pdo_mysql/tests/pdo_mysql_stmt_variable_columncount.phpt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
--TEST--
22
MySQL Prepared Statements and different column counts
3-
--XFAIL--
4-
nextRowset() problem with stored proc & emulation mode & mysqlnd
53
--SKIPIF--
64
<?php
75
require_once(dirname(__FILE__) . DIRECTORY_SEPARATOR . 'skipif.inc');

0 commit comments

Comments
 (0)