Merge branch 'PHP-8.0'

* PHP-8.0:
  PDO MySQL: Use stmt_next_result with libmysqlclient as well
  PDO MySQL: Fix libmysql option test

2 files changed, 44 insertions, 50 deletions

diff --git a/ext/pdo_mysql/mysql_statement.c b/ext/pdo_mysql/mysql_statement.c
index 46f39c985d..80d8747cd9 100644
--- a/ext/pdo_mysql/mysql_statement.c
+++ b/ext/pdo_mysql/mysql_statement.c
@@ -341,30 +341,29 @@ static int pdo_mysql_stmt_next_rowset(pdo_stmt_t *stmt) /* {{{ */
 	PDO_DBG_INF_FMT("stmt=%p", S->stmt);
 
 	/* ensure that we free any previous unfetched results */
-	if (S->stmt) {
-		mysql_stmt_free_result(S->stmt);
-	}
 	pdo_mysql_free_result(S);
 
-#ifdef PDO_USE_MYSQLND
 	if (S->stmt) {
-		if (mysqlnd_stmt_next_result(S->stmt)) {
+		mysql_stmt_free_result(S->stmt);
+		if (mysql_stmt_next_result(S->stmt)) {
+			pdo_mysql_error_stmt(stmt);
+			S->done = 1;
+			PDO_DBG_RETURN(0);
+		}
+	} else {
+		if (mysql_next_result(H->server)) {
 			pdo_mysql_error_stmt(stmt);
 			S->done = 1;
 			PDO_DBG_RETURN(0);
 		}
+	}
 
+#ifdef PDO_USE_MYSQLND
+	if (S->stmt) {
 		PDO_DBG_RETURN(pdo_mysql_stmt_after_execute_prepared(stmt));
 	}
 #endif
-
-	if (mysql_next_result(H->server)) {
-		pdo_mysql_error_stmt(stmt);
-		S->done = 1;
-		PDO_DBG_RETURN(0);
-	} else {
-		PDO_DBG_RETURN(pdo_mysql_fill_stmt_from_result(stmt));
-	}
+	PDO_DBG_RETURN(pdo_mysql_fill_stmt_from_result(stmt));
 }
 /* }}} */
 
diff --git a/ext/pdo_mysql/tests/pdo_mysql___construct_options_libmysql.phpt b/ext/pdo_mysql/tests/pdo_mysql___construct_options_libmysql.phpt
index 1838336050..c186689932 100644
--- a/ext/pdo_mysql/tests/pdo_mysql___construct_options_libmysql.phpt
+++ b/ext/pdo_mysql/tests/pdo_mysql___construct_options_libmysql.phpt
@@ -27,57 +27,52 @@ if (MySQLPDOTest::isPDOMySQLnd())
                     $tmp, gettype($tmp),
                     $option_desc);
         } catch (PDOException $e) {
-            printf("[%03d] %s\n", $offset, $e->getMessage());
+            echo "Failed to getAttribute() for $option_desc\n";
         }
-
     }
 
-    try {
-
-        $dsn = MySQLPDOTest::getDSN();
-        $user = PDO_MYSQL_TEST_USER;
-        $pass = PDO_MYSQL_TEST_PASS;
+    $dsn = MySQLPDOTest::getDSN();
+    $user = PDO_MYSQL_TEST_USER;
+    $pass = PDO_MYSQL_TEST_PASS;
 
-        $valid_options = array();
-        $valid_options[PDO::MYSQL_ATTR_MAX_BUFFER_SIZE] = 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE';
-        $valid_options[PDO::MYSQL_ATTR_INIT_COMMAND] = 'PDO::MYSQL_ATTR_INIT_COMMAND';
-        $valid_options[PDO::MYSQL_ATTR_READ_DEFAULT_FILE] = 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE';
-        $valid_options[PDO::MYSQL_ATTR_READ_DEFAULT_GROUP] = 'PDO::MYSQL_ATTR_READ_DEFAULT_GROUP';
+    $valid_options = array();
+    $valid_options[PDO::MYSQL_ATTR_MAX_BUFFER_SIZE] = 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE';
+    $valid_options[PDO::MYSQL_ATTR_INIT_COMMAND] = 'PDO::MYSQL_ATTR_INIT_COMMAND';
+    $valid_options[PDO::MYSQL_ATTR_READ_DEFAULT_FILE] = 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE';
+    $valid_options[PDO::MYSQL_ATTR_READ_DEFAULT_GROUP] = 'PDO::MYSQL_ATTR_READ_DEFAULT_GROUP';
 
-        $defaults[PDO::MYSQL_ATTR_MAX_BUFFER_SIZE] = 1048576;
-        /* TODO getAttribute() does not handle it */
-        $defaults[PDO::MYSQL_ATTR_INIT_COMMAND] = '';
-        $defaults[PDO::MYSQL_ATTR_READ_DEFAULT_FILE] = false;
-        $defaults[PDO::MYSQL_ATTR_READ_DEFAULT_GROUP] = false;
+    $defaults[PDO::MYSQL_ATTR_MAX_BUFFER_SIZE] = 1048576;
+    /* TODO getAttribute() does not handle it */
+    $defaults[PDO::MYSQL_ATTR_INIT_COMMAND] = '';
+    $defaults[PDO::MYSQL_ATTR_READ_DEFAULT_FILE] = false;
+    $defaults[PDO::MYSQL_ATTR_READ_DEFAULT_GROUP] = false;
 
-        $db = new PDO($dsn, $user, $pass);
-        foreach ($valid_options as $option => $name) {
-            /* TODO getAttribute() is pretty poor in supporting the options, suppress errors */
-            $tmp = @$db->getAttribute($option);
+    $db = new PDO($dsn, $user, $pass);
+    foreach ($valid_options as $option => $name) {
+        try {
+            $tmp = $db->getAttribute($option);
             if ($tmp !== $defaults[$option])
                 printf("[001] Expecting default value for '%s' of '%s'/%s, getAttribute() reports setting '%s'/%s\n",
                     $name, $defaults[$option], gettype($defaults[$option]),
                     $tmp, gettype($tmp));
+        } catch (PDOException $e) {
+            echo "Failed to getAttribute() for $name\n";
         }
+    }
 
-        set_option_and_check(26, PDO::MYSQL_ATTR_READ_DEFAULT_FILE, true, 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE');
-        set_option_and_check(27, PDO::MYSQL_ATTR_READ_DEFAULT_FILE, false, 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE');
-
-        set_option_and_check(30, PDO::MYSQL_ATTR_MAX_BUFFER_SIZE, -1, 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE', true);
-        set_option_and_check(31, PDO::MYSQL_ATTR_MAX_BUFFER_SIZE, PHP_INT_MAX, 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE');
-        set_option_and_check(32, PDO::MYSQL_ATTR_MAX_BUFFER_SIZE, 1, 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE');
-
+    set_option_and_check(26, PDO::MYSQL_ATTR_READ_DEFAULT_FILE, true, 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE');
+    set_option_and_check(27, PDO::MYSQL_ATTR_READ_DEFAULT_FILE, false, 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE');
 
-    } catch (PDOException $e) {
-        printf("[001] %s, [%s] %s\n",
-            $e->getMessage(),
-            (is_object($db)) ? $db->errorCode() : 'n/a',
-            (is_object($db)) ? implode(' ', $db->errorInfo()) : 'n/a');
-    }
+    set_option_and_check(30, PDO::MYSQL_ATTR_MAX_BUFFER_SIZE, -1, 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE', true);
+    set_option_and_check(31, PDO::MYSQL_ATTR_MAX_BUFFER_SIZE, PHP_INT_MAX, 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE');
+    set_option_and_check(32, PDO::MYSQL_ATTR_MAX_BUFFER_SIZE, 1, 'PDO::MYSQL_ATTR_MAX_BUFFER_SIZE');
 
-    print "done!";
+    print "done!\n";
 ?>
 --EXPECT--
-[001] Expecting default value for 'PDO::MYSQL_ATTR_INIT_COMMAND' of ''/string, getAttribute() reports setting ''/boolean
-[026] Expecting '1'/boolean got ''/boolean' for options 'PDO::MYSQL_ATTR_READ_DEFAULT_FILE'
+Failed to getAttribute() for PDO::MYSQL_ATTR_INIT_COMMAND
+Failed to getAttribute() for PDO::MYSQL_ATTR_READ_DEFAULT_FILE
+Failed to getAttribute() for PDO::MYSQL_ATTR_READ_DEFAULT_GROUP
+Failed to getAttribute() for PDO::MYSQL_ATTR_READ_DEFAULT_FILE
+Failed to getAttribute() for PDO::MYSQL_ATTR_READ_DEFAULT_FILE
 done!