From: Antony Dovgal Date: Fri, 3 Feb 2006 14:46:42 +0000 (+0000) Subject: fix #36010 (Segfault when re-creating and re-executing statements with bound parameters) X-Git-Tag: php-5.1.3RC1~163 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=52232a7f7aaeb72e25f0134ac9142cfb3f6795d9;p=php fix #36010 (Segfault when re-creating and re-executing statements with bound parameters) --- diff --git a/NEWS b/NEWS index 0707bcd48e..148c94a7f1 100644 --- a/NEWS +++ b/NEWS @@ -51,6 +51,8 @@ PHP NEWS - Fixed bug #36016 (realpath cache memleaks). (Dmitry, Nuno) - Fixed bug #36011 (Strict errormsg wrong for call_user_func() and the likes). (Marcus) +- Fixed bug #36010 (Segfault when re-creating and re-executing statements with + bound parameters). (Tony) - Fixed bug #36006 (Problem with $this in __destruct()). (Dmitry) - Fixed bug #35999 (recursive mkdir() does not work with relative path like "foo/bar"). (Tony) diff --git a/ext/oci8/oci8_statement.c b/ext/oci8/oci8_statement.c index dcb3130b7e..5f6559fcf4 100644 --- a/ext/oci8/oci8_statement.c +++ b/ext/oci8/oci8_statement.c @@ -70,7 +70,7 @@ php_oci_statement *php_oci_statement_create (php_oci_connection *connection, cha php_oci_error(connection->err, connection->errcode TSRMLS_CC); #if HAVE_OCI_STMT_PREPARE2 - PHP_OCI_CALL(OCIStmtRelease, (statement->stmt, statement->err, NULL, 0, OCI_DEFAULT)); + PHP_OCI_CALL(OCIStmtRelease, (statement->stmt, statement->err, NULL, 0, OCI_STRLS_CACHE_DELETE)); PHP_OCI_CALL(OCIHandleFree,(statement->err, OCI_HTYPE_ERROR)); #else PHP_OCI_CALL(OCIHandleFree,(statement->stmt, OCI_HTYPE_STMT)); @@ -578,7 +578,7 @@ void php_oci_statement_free(php_oci_statement *statement TSRMLS_DC) if (statement->stmt) { #if HAVE_OCI_STMT_PREPARE2 if (statement->last_query_len) { /* FIXME: magical */ - PHP_OCI_CALL(OCIStmtRelease, (statement->stmt, statement->err, NULL, 0, OCI_DEFAULT)); + PHP_OCI_CALL(OCIStmtRelease, (statement->stmt, statement->err, NULL, 0, OCI_STRLS_CACHE_DELETE)); } else { PHP_OCI_CALL(OCIHandleFree, (statement->stmt, OCI_HTYPE_STMT));