Fixed Bug #71049 (SQLite3Stmt::execute() releases bound parameter instead of internal buffer)

Author: laruence

NEWS

@@ -67,6 +67,10 @@ PHP                                                                        NEWS
 - SPL:
   . Fixed bug #71028 (Undefined index with ArrayIterator). (Laruence)
 
+- SQLite3:
+  . Fixed bug #71049 (SQLite3Stmt::execute() releases bound parameter instead
+    of internal buffer). (Laruence)
+
 - Standard:
   . Fixed bug #70999 (php_random_bytes: called object is not a function).
     (Scott)

ext/sqlite3/sqlite3.c

@@ -1536,8 +1536,7 @@ PHP_METHOD(sqlite3stmt, execute)
 						}
 						buffer = php_stream_copy_to_mem(stream, PHP_STREAM_COPY_ALL, 0);
 					} else {
-						convert_to_string(parameter);
-						buffer = Z_STR_P(parameter);
+						buffer = zval_get_string(parameter);
 					}
 
 					if (buffer) {

ext/sqlite3/tests/bug71049.phpt

@@ -0,0 +1,21 @@
+--TEST--
+Bug #71049 (SQLite3Stmt::execute() releases bound parameter instead of internal buffer)
+--SKIPIF--
+<?php
+if (!extension_loaded('sqlite3')) die('skip'); ?>
+--FILE--
+<?php
+
+require(__DIR__ . '/new_db.inc');
+
+$db->exec('CREATE TABLE test (age INTEGER, id STRING)');
+
+$stmt = $db->prepare("SELECT * FROM test WHERE id = ? ORDER BY id ASC");
+$foo = "alive" . chr(33);
+$stmt->bindParam(1, $foo, SQLITE3_BLOB);
+$results = $stmt->execute();
+var_dump($foo);
+$db->close();
+?>
+--EXPECT--
+string(6) "alive!"