| Lars Westermann 2007-11-08, 7:01 pm |
| lwe Thu Nov 8 19:16:27 2007 UTC
Modified files: (Branch: PHP_5_3)
/php-src/ext/interbase ibase_query.c
Log:
[DOC] PHP5: Prototype for ibase_query should be:
ibase_query([resource link_identifier, [ resource link_identifier, ]]
string query [, mixed bind_arg [, mixed bind_arg [, ...]]]
- Fixed bug #30690: (Resource handle from ibase_execute becomes invalid after return)
- Fixed bug #30907: (ibase_query() crashes (in fact the same bug as #32143)
- Fixed bug #32143: (ibase_query() causing IBserver 7 crash with NULL param as link-id)
- Fixed bug #39056: (Interbase NUMERIC data type error)
- Fixed bug #39397: (invalid statement handle in Unknown on line 0)
- Fixed bug #39700: (NUMERIC error when result precision are 7,8 or 12-14)
- Fixed bug #42284: (duplicate of #39700)
http://cvs.php.net/viewvc.cgi/php-s...1&diff_format=u
Index: php-src/ext/interbase/ibase_query.c
diff -u php-src/ext/interbase/ibase_query.c:1.23.2.1.2.10 php-src/ext/interbase/ibase_query.c:1.23.2.1.2.10.2.1
--- php-src/ext/interbase/ibase_query.c:1.23.2.1.2.10 Thu Jun 7 08:59:00 2007
+++ php-src/ext/interbase/ibase_query.c Thu Nov 8 19:16:27 2007
@@ -16,7 +16,7 @@
+----------------------------------------------------------------------+
*/
-/* $Id: ibase_query.c,v 1.23.2.1.2.10 2007/06/07 08:59:00 tony2001 Exp $ */
+/* $Id: ibase_query.c,v 1.23.2.1.2.10.2.1 2007/11/08 19:16:27 lwe Exp $ */
#ifdef HAVE_CONFIG_H
#include "config.h"
@@ -49,6 +49,7 @@
typedef struct {
ibase_db_link *link;
ibase_trans *trans;
+ struct _ib_query *query;
isc_stmt_handle stmt;
unsigned short type;
unsigned char has_more_rows, statement_type;
@@ -56,9 +57,10 @@
ibase_array out_array[1]; /* last member */
} ibase_result;
-typedef struct {
+typedef struct _ib_query {
ibase_db_link *link;
ibase_trans *trans;
+ ibase_result *result;
int result_res_id;
isc_stmt_handle stmt;
XSQLDA *in_sqlda, *out_sqlda;
@@ -116,6 +118,22 @@
}
/* }}} */
+static void _php_ibase_free_stmt_handle(ibase_db_lin
k *link, isc_stmt_handle stmt TSRMLS_DC) /* {{{ */
+{
+ if (stmt) {
+ IBDEBUG("Dropping statement handle (free_stmt_handle)...");
+ /* Only free statement if db-connection is still open */
+ char db_items[] = {isc_info_page_size}, res_buf[40];
+ if (SUCCESS == isc_database_info(IB_STATUS, &link->handle,
+ sizeof(db_items), db_items, sizeof(res_buf), res_buf)) {
+ if (isc_dsql_free_statement(IB_STATUS, &stmt, DSQL_drop)) {
+ _php_ibase_error(TSRMLS_C);
+ }
+ }
+ }
+}
+/* }}} */
+
static void _php_ibase_free_result(zend_rsrc_list_en
try *rsrc TSRMLS_DC) /* {{{ */
{
ibase_result *ib_result = (ibase_result *) rsrc->ptr;
@@ -123,9 +141,11 @@
IBDEBUG("Freeing result by dtor...");
if (ib_result) {
_php_ibase_free_xsqlda(ib_result->out_sqlda);
- if (ib_result->stmt && ib_result->type != EXECUTE_RESULT) {
- IBDEBUG("Dropping statement handle (free_result dtor)...");
- isc_dsql_free_statement(IB_STATUS, &ib_result->stmt, DSQL_drop);
+ if (ib_result->query != NULL) {
+ IBDEBUG("query still valid; don't drop statement handle");
+ ib_result->query->result = NULL; /* Indicate to query, that result is released */
+ } else {
+ _php_ibase_free_stmt_handle(ib_resul
t->link, ib_result->stmt TSRMLS_CC);
}
efree(ib_result);
}
@@ -142,11 +162,11 @@
if (ib_query->out_sqlda) {
efree(ib_query->out_sqlda);
}
- if (ib_query->stmt) {
- IBDEBUG("Dropping statement handle (free_query)...");
- if (isc_dsql_free_statement(IB_STATUS, &ib_query->stmt, DSQL_drop)) {
- _php_ibase_error(TSRMLS_C);
- }
+ if (ib_query->result != NULL) {
+ IBDEBUG("result still valid; don't drop statement handle");
+ ib_query->result->query = NULL; /* Indicate to result, that query is released */
+ } else {
+ _php_ibase_free_stmt_handle(ib_query->link, ib_query->stmt TSRMLS_CC);
}
if (ib_query->in_array) {
efree(ib_query->in_array);
@@ -166,11 +186,7 @@
if (ib_query != NULL) {
IBDEBUG("Preparing to free query by dtor...");
-
_php_ibase_free_query(ib_query TSRMLS_CC);
- if (ib_query->statement_type != isc_info_sql_stmt_exec_procedure) {
- zend_list_delete(ib_query->result_res_id);
- }
efree(ib_query);
}
}
@@ -302,9 +318,16 @@
static char info_type[] = {isc_info_sql_stmt_type};
char result[8];
+ /* Return FAILURE, if querystring is empty */
+ if (*query == '\0') {
+ php_error_docref(NULL TSRMLS_CC, E_WARNING, "Querystring empty.");
+ return FAILURE;
+ }
+
ib_query->link = link;
ib_query->trans = trans;
ib_query->result_res_id = 0;
+ ib_query->result = NULL;
ib_query->stmt = NULL;
ib_query->in_array = NULL;
ib_query->out_array = NULL;
@@ -929,7 +952,10 @@
res = emalloc(sizeof(ibase_result)+sizeof(ibas
e_array)*max(0,ib_query->out_array_cnt-1));
res->link = ib_query->link;
res->trans = ib_query->trans;
- res->stmt = ib_query->stmt;
+ res->stmt = ib_query->stmt;
+ /* ib_result and ib_query point at each other to handle release of statement handle properly */
+ res->query = ib_query;
+ ib_query->result = res;
res->statement_type = ib_query->statement_type;
res->has_more_rows = 1;
@@ -1288,9 +1314,23 @@
static int _php_ibase_var_zval(zval *val, void *data, int type, int len, /* {{{ */
int scale, int flag TSRMLS_DC)
{
- static ISC_INT64 const scales[] = { 1, 10, 100, 1000, 10000, 100000, 1000000, 100000000, 1000000000,
- 1000000000, LL_LIT(10000000000),LL_LIT(100000000000)
,LL_LIT(10000000000000),LL_LIT(100000000
000000),
- LL_LIT(1000000000000000),LL_LIT(100000
0000000000),LL_LIT(1000000000000000000) };
+ static ISC_INT64 const scales[] = { 1, 10, 100, 1000,
+ 10000,
+ 100000,
+ 1000000,
+ 10000000,
+ 100000000,
+ 1000000000,
+ LL_LIT(10000000000),
+ LL_LIT(100000000000),
+ LL_LIT(1000000000000),
+ LL_LIT(10000000000000),
+ LL_LIT(100000000000000),
+ LL_LIT(1000000000000000),
+ LL_LIT(10000000000000000),
+ LL_LIT(100000000000000000),
+ LL_LIT(1000000000000000000)
+ };
switch (type & ~1) {
unsigned short l;
|