Fixed Bug #69796 mysqli_stmt::fetch doesn't assign null values to bound variables

laruence · laruence · commit 141f0ca18cd0 · 2015-06-11T13:35:43.000+08:00
diff --git a/NEWS b/NEWS
@@ -8,6 +8,10 @@ PHP                                                                        NEWS
   . Fixed bug #69781 (phpinfo() reports Professional Editions of Windows
     7/8/8.1/10 as "Business"). (Christian Wenz)
 
+- mysqlnd:
+  . Fixed Bug #69796 (mysqli_stmt::fetch doesn't assign null values to
+    bound variables). (Laruence);
+
 - PDO_pgsql:
   . Fixed bug #69752 (PDOStatement::execute() leaks memory with DML
     Statements when closeCuror() is u). (Philip Hofstetter)
diff --git a/ext/mysqlnd/mysqlnd_ps.c b/ext/mysqlnd/mysqlnd_ps.c
@@ -925,6 +925,8 @@ mysqlnd_stmt_fetch_row_unbuffered(MYSQLND_RES * result, void * param, unsigned i
 						ZVAL_COPY_VALUE(result, data);
 						/* copied data, thus also the ownership. Thus null data */
 						ZVAL_NULL(data);
+					} else {
+						ZVAL_NULL(result);
 					}
 				}
 			}

Original file line number	Diff line number	Diff line change
`@@ -925,6 +925,8 @@ mysqlnd_stmt_fetch_row_unbuffered(MYSQLND_RES * result, void * param, unsigned i`
`925`	`925`	`ZVAL_COPY_VALUE(result, data);`
`926`	`926`	`/* copied data, thus also the ownership. Thus null data */`
`927`	`927`	`ZVAL_NULL(data);`
	`928`	`+ } else {`
	`929`	`+ ZVAL_NULL(result);`
`928`	`930`	`}`
`929`	`931`	`}`
`930`	`932`	`}`