summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristoph M. Becker <cmb@php.net>2016-06-27 16:55:15 +0200
committerChristoph M. Becker <cmb@php.net>2016-06-27 17:06:27 +0200
commit57940605da718174cdcf5f6bf19b7ed7df27ffa6 (patch)
treed1a5e18edd43e25804d020985650a4c73fa0d86b
parentb4873e57b434bb65a21dc189ac18a618f223a15e (diff)
downloadphp-git-57940605da718174cdcf5f6bf19b7ed7df27ffa6.tar.gz
Fix #70628: Clearing bindings on an SQLite3 statement doesn't work
Obiously, it isn't sufficient to call sqlite3_clear_bindings() alone, but also the bound_params of the php_sqlite3_stmt have to be cleared.
-rw-r--r--ext/sqlite3/sqlite3.c6
-rw-r--r--ext/sqlite3/tests/bug70628.phpt55
2 files changed, 61 insertions, 0 deletions
diff --git a/ext/sqlite3/sqlite3.c b/ext/sqlite3/sqlite3.c
index ce9472a714..4513b77d0f 100644
--- a/ext/sqlite3/sqlite3.c
+++ b/ext/sqlite3/sqlite3.c
@@ -1376,6 +1376,12 @@ PHP_METHOD(sqlite3stmt, clear)
RETURN_FALSE;
}
+ if (stmt_obj->bound_params) {
+ zend_hash_destroy(stmt_obj->bound_params);
+ FREE_HASHTABLE(stmt_obj->bound_params);
+ stmt_obj->bound_params = NULL;
+ }
+
RETURN_TRUE;
}
/* }}} */
diff --git a/ext/sqlite3/tests/bug70628.phpt b/ext/sqlite3/tests/bug70628.phpt
new file mode 100644
index 0000000000..3d807baaa8
--- /dev/null
+++ b/ext/sqlite3/tests/bug70628.phpt
@@ -0,0 +1,55 @@
+--TEST--
+Bug #70628 (Clearing bindings on an SQLite3 statement doesn't work)
+--SKIPIF--
+<?php
+if (!extension_loaded('sqlite3')) die('skip'); ?>
+--FILE--
+<?php
+$db = new SQLite3(':memory:');
+
+$db->exec("CREATE TABLE Dogs (Id INTEGER PRIMARY KEY, Breed TEXT, Name TEXT, Age INTEGER)");
+
+$sth = $db->prepare("INSERT INTO Dogs (Breed, Name, Age) VALUES (:breed,:name,:age)");
+
+$sth->bindValue(':breed', 'canis', SQLITE3_TEXT);
+$sth->bindValue(':name', 'jack', SQLITE3_TEXT);
+$sth->bindValue(':age', 7, SQLITE3_INTEGER);
+$sth->execute();
+
+$sth->clear();
+$sth->reset();
+
+$sth->bindValue(':breed', 'russel', SQLITE3_TEXT);
+$sth->bindValue(':age', 3, SQLITE3_INTEGER);
+$sth->execute();
+
+$res = $db->query('SELECT * FROM Dogs');
+while (($row = $res->fetchArray(SQLITE3_ASSOC))) {
+ var_dump($row);
+}
+$res->finalize();
+
+$sth->close();
+$db->close();
+?>
+--EXPECT--
+array(4) {
+ ["Id"]=>
+ int(1)
+ ["Breed"]=>
+ string(5) "canis"
+ ["Name"]=>
+ string(4) "jack"
+ ["Age"]=>
+ int(7)
+}
+array(4) {
+ ["Id"]=>
+ int(2)
+ ["Breed"]=>
+ string(6) "russel"
+ ["Name"]=>
+ NULL
+ ["Age"]=>
+ int(3)
+}