]> granicus.if.org Git - php/commitdiff
Fix for bug #54221 mysqli::get_warnings segfault when used in multi queries
authorAndrey Hristov <andrey@php.net>
Thu, 17 Mar 2011 10:28:53 +0000 (10:28 +0000)
committerAndrey Hristov <andrey@php.net>
Thu, 17 Mar 2011 10:28:53 +0000 (10:28 +0000)
ext/mysqli/mysqli_warning.c
ext/mysqli/tests/bug54221.phpt [new file with mode: 0644]

index 09fa5b35109c703e4cbfcdb6e4f4e0e844ccc339..ee815121b3f87cf5c0098f8229d86e968259d1f3 100644 (file)
@@ -197,7 +197,7 @@ PHP_METHOD(mysqli_warning, next)
 
                MYSQLI_FETCH_RESOURCE(w, MYSQLI_WARNING *, &mysqli_warning, "mysqli_warning", MYSQLI_STATUS_VALID);
 
-               if (w->next) {
+               if (w && w->next) {
                        w = w->next;
                ((MYSQLI_RESOURCE *)(obj->ptr))->ptr = w;
                        RETURN_TRUE;
diff --git a/ext/mysqli/tests/bug54221.phpt b/ext/mysqli/tests/bug54221.phpt
new file mode 100644 (file)
index 0000000..78c9a2c
--- /dev/null
@@ -0,0 +1,47 @@
+--TEST--
+Bug #54221 mysqli::get_warnings segfault when used in multi queries
+--SKIPIF--
+<?php
+require_once('skipif.inc');
+require_once('skipifconnectfailure.inc');
+?>
+--INI--
+mysqli.max_links = 1
+mysqli.allow_persistent = Off
+mysqli.max_persistent = 0
+mysqli.reconnect = Off
+--FILE--
+<?php
+       include ("connect.inc");
+
+       $link = mysqli_init();
+       if (!my_mysqli_real_connect($link, $host, $user, $passwd, $db, $port, $socket)) {
+               printf("[002] Connect failed, [%d] %s\n", mysqli_connect_errno(), mysqli_connect_error());
+       }
+
+       $create = "CREATE TEMPORARY TABLE IF NOT EXISTS t54221(a int)";
+
+       $query = "$create;$create;$create;";
+       if ($link->multi_query($query)) {
+               do {
+                       $sth = $link->store_result();
+
+                       if ($link->warning_count) {
+                               $warnings = $link->get_warnings();
+                               if ($warnings) {
+                                       do {
+                                               echo "Warning: ".$warnings->errno.": ".$warnings->message."\n"; 
+                                       } while ($warnings->next());
+                               }
+                       }
+               } while ($link->more_results() && $link->next_result());
+       }
+
+       mysqli_close($link);
+
+       print "done!";
+?>
+--EXPECTF--
+Warning: : 
+Warning: 1050: Table 't54221' already exists
+done!