fix for bug #63530 mysqlnd_stmt::bind_one_parameter uses wrong alloc for stmt->param_bind

faizshukri · faizshukri · commit f1e2edff8bef · 2013-03-13T13:37:51.000+01:00
diff --git a/NEWS b/NEWS
@@ -4,6 +4,10 @@ PHP                                                                        NEWS
 - PCRE:
   . Merged PCRE 8.32). (Anatol)
 
+- mysqlnd
+  . Fixed bug #63530 (mysqlnd_stmt::bind_one_parameter crashes, uses wrong alloc
+    for stmt->param_bind). (Andrey)
+
 
 28 Feb 2013, PHP 5.3.23RC1
 
diff --git a/ext/mysqlnd/mysqlnd_ps.c b/ext/mysqlnd/mysqlnd_ps.c
@@ -1485,7 +1485,7 @@ MYSQLND_METHOD(mysqlnd_stmt, bind_one_parameter)(MYSQLND_STMT * const s, unsigne
 
 	if (stmt->param_count) {
 		if (!stmt->param_bind) {
-			stmt->param_bind = mnd_ecalloc(stmt->param_count, sizeof(MYSQLND_PARAM_BIND));
+			stmt->param_bind = mnd_pecalloc(stmt->param_count, sizeof(MYSQLND_PARAM_BIND), stmt->persistent);
 			if (!stmt->param_bind) {
 				DBG_RETURN(FAIL);
 			}

Original file line number	Diff line number	Diff line change
`@@ -1485,7 +1485,7 @@ MYSQLND_METHOD(mysqlnd_stmt, bind_one_parameter)(MYSQLND_STMT * const s, unsigne`
`1485`	`1485`
`1486`	`1486`	`if (stmt->param_count) {`
`1487`	`1487`	`if (!stmt->param_bind) {`
`1488`		`- stmt->param_bind = mnd_ecalloc(stmt->param_count, sizeof(MYSQLND_PARAM_BIND));`
	`1488`	`+ stmt->param_bind = mnd_pecalloc(stmt->param_count, sizeof(MYSQLND_PARAM_BIND), stmt->persistent);`
`1489`	`1489`	`if (!stmt->param_bind) {`
`1490`	`1490`	`DBG_RETURN(FAIL);`
`1491`	`1491`	`}`