diff options
Diffstat (limited to 'ext/sqlite3/sqlite3.c')
| -rw-r--r-- | ext/sqlite3/sqlite3.c | 800 |
1 files changed, 371 insertions, 429 deletions
diff --git a/ext/sqlite3/sqlite3.c b/ext/sqlite3/sqlite3.c index f7f76cd3fb..6f2656ab51 100644 --- a/ext/sqlite3/sqlite3.c +++ b/ext/sqlite3/sqlite3.c @@ -1,6 +1,6 @@ /* +----------------------------------------------------------------------+ - | PHP Version 5 | + | PHP Version 7 | +----------------------------------------------------------------------+ | Copyright (c) 1997-2016 The PHP Group | +----------------------------------------------------------------------+ @@ -39,7 +39,7 @@ ZEND_DECLARE_MODULE_GLOBALS(sqlite3) static PHP_GINIT_FUNCTION(sqlite3); static int php_sqlite3_authorizer(void *autharg, int access_type, const char *arg3, const char *arg4, const char *arg5, const char *arg6); -static void sqlite3_param_dtor(void *data); +static void sqlite3_param_dtor(zval *data); static int php_sqlite3_compare_stmt_zval_free(php_sqlite3_free_list **free_list, zval *statement); /* {{{ Error Handler @@ -48,18 +48,17 @@ static void php_sqlite3_error(php_sqlite3_db_object *db_obj, char *format, ...) { va_list arg; char *message; - TSRMLS_FETCH(); - va_start(arg, format); + va_start(arg, format); vspprintf(&message, 0, format, arg); va_end(arg); if (db_obj && db_obj->exception) { - zend_throw_exception(zend_exception_get_default(TSRMLS_C), message, 0 TSRMLS_CC); + zend_throw_exception(zend_ce_exception, message, 0); } else { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "%s", message); + php_error_docref(NULL, E_WARNING, "%s", message); } - + if (message) { efree(message); } @@ -74,7 +73,7 @@ static void php_sqlite3_error(php_sqlite3_db_object *db_obj, char *format, ...) #define SQLITE3_CHECK_INITIALIZED_STMT(member, class_name) \ if (!(member)) { \ - php_error_docref(NULL TSRMLS_CC, E_WARNING, "The " #class_name " object has not been correctly initialised"); \ + php_error_docref(NULL, E_WARNING, "The " #class_name " object has not been correctly initialised"); \ RETURN_FALSE; \ } @@ -102,43 +101,38 @@ PHP_METHOD(sqlite3, open) php_sqlite3_db_object *db_obj; zval *object = getThis(); char *filename, *encryption_key, *fullpath; - int filename_len, encryption_key_len = 0; - long flags = SQLITE_OPEN_READWRITE | SQLITE_OPEN_CREATE; - zend_error_handling error_handling; + size_t filename_len, encryption_key_len = 0; + zend_long flags = SQLITE_OPEN_READWRITE | SQLITE_OPEN_CREATE; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); - zend_replace_error_handling(EH_THROW, NULL, &error_handling TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(object); - if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "p|ls", &filename, &filename_len, &flags, &encryption_key, &encryption_key_len)) { - zend_restore_error_handling(&error_handling TSRMLS_CC); + if (FAILURE == zend_parse_parameters_throw(ZEND_NUM_ARGS(), "p|ls", &filename, &filename_len, &flags, &encryption_key, &encryption_key_len)) { return; } - zend_restore_error_handling(&error_handling TSRMLS_CC); - if (db_obj->initialised) { - zend_throw_exception(zend_exception_get_default(TSRMLS_C), "Already initialised DB Object", 0 TSRMLS_CC); + zend_throw_exception(zend_ce_exception, "Already initialised DB Object", 0); } if (strlen(filename) != filename_len) { return; } if (memcmp(filename, ":memory:", sizeof(":memory:")) != 0) { - if (!(fullpath = expand_filepath(filename, NULL TSRMLS_CC))) { - zend_throw_exception(zend_exception_get_default(TSRMLS_C), "Unable to expand filepath", 0 TSRMLS_CC); + if (!(fullpath = expand_filepath(filename, NULL))) { + zend_throw_exception(zend_ce_exception, "Unable to expand filepath", 0); return; } #if PHP_API_VERSION < 20100412 if (PG(safe_mode) && (!php_checkuid(fullpath, NULL, CHECKUID_CHECK_FILE_AND_DIR))) { - zend_throw_exception_ex(zend_exception_get_default(TSRMLS_C), 0 TSRMLS_CC, "safe_mode prohibits opening %s", fullpath); + zend_throw_exception_ex(zend_ce_exception, 0, "safe_mode prohibits opening %s", fullpath); efree(fullpath); return; } #endif - if (php_check_open_basedir(fullpath TSRMLS_CC)) { - zend_throw_exception_ex(zend_exception_get_default(TSRMLS_C), 0 TSRMLS_CC, "open_basedir prohibits opening %s", fullpath); + if (php_check_open_basedir(fullpath)) { + zend_throw_exception_ex(zend_ce_exception, 0, "open_basedir prohibits opening %s", fullpath); efree(fullpath); return; } @@ -151,7 +145,7 @@ PHP_METHOD(sqlite3, open) #else if (sqlite3_open(fullpath, &(db_obj->db)) != SQLITE_OK) { #endif - zend_throw_exception_ex(zend_exception_get_default(TSRMLS_C), 0 TSRMLS_CC, "Unable to open database: %s", sqlite3_errmsg(db_obj->db)); + zend_throw_exception_ex(zend_ce_exception, 0, "Unable to open database: %s", sqlite3_errmsg(db_obj->db)); if (fullpath) { efree(fullpath); } @@ -161,7 +155,7 @@ PHP_METHOD(sqlite3, open) #if SQLITE_HAS_CODEC if (encryption_key_len > 0) { if (sqlite3_key(db_obj->db, encryption_key, encryption_key_len) != SQLITE_OK) { - zend_throw_exception_ex(zend_exception_get_default(TSRMLS_C), 0 TSRMLS_CC, "Unable to open database: %s", sqlite3_errmsg(db_obj->db)); + zend_throw_exception_ex(zend_ce_exception, 0, "Unable to open database: %s", sqlite3_errmsg(db_obj->db)); return; } } @@ -190,7 +184,7 @@ PHP_METHOD(sqlite3, close) php_sqlite3_db_object *db_obj; zval *object = getThis(); int errcode; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(object); if (zend_parse_parameters_none() == FAILURE) { return; @@ -218,17 +212,17 @@ PHP_METHOD(sqlite3, exec) { php_sqlite3_db_object *db_obj; zval *object = getThis(); - char *sql, *errtext = NULL; - int sql_len; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + zend_string *sql; + char *errtext = NULL; + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) - if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &sql, &sql_len)) { + if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS(), "S", &sql)) { return; } - if (sqlite3_exec(db_obj->db, sql, NULL, NULL, &errtext) != SQLITE_OK) { + if (sqlite3_exec(db_obj->db, ZSTR_VAL(sql), NULL, NULL, &errtext) != SQLITE_OK) { php_sqlite3_error(db_obj, "%s", errtext); sqlite3_free(errtext); RETURN_FALSE; @@ -248,7 +242,7 @@ PHP_METHOD(sqlite3, version) array_init(return_value); - add_assoc_string(return_value, "versionString", (char*)sqlite3_libversion(), 1); + add_assoc_string(return_value, "versionString", (char*)sqlite3_libversion()); add_assoc_long(return_value, "versionNumber", sqlite3_libversion_number()); return; @@ -261,7 +255,7 @@ PHP_METHOD(sqlite3, lastInsertRowID) { php_sqlite3_db_object *db_obj; zval *object = getThis(); - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) @@ -279,7 +273,7 @@ PHP_METHOD(sqlite3, lastErrorCode) { php_sqlite3_db_object *db_obj; zval *object = getThis(); - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->db, SQLite3) @@ -301,7 +295,7 @@ PHP_METHOD(sqlite3, lastErrorMsg) { php_sqlite3_db_object *db_obj; zval *object = getThis(); - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->db, SQLite3) @@ -310,7 +304,7 @@ PHP_METHOD(sqlite3, lastErrorMsg) } if (db_obj->initialised) { - RETURN_STRING((char *)sqlite3_errmsg(db_obj->db), 1); + RETURN_STRING((char *)sqlite3_errmsg(db_obj->db)); } else { RETURN_EMPTY_STRING(); } @@ -323,13 +317,13 @@ PHP_METHOD(sqlite3, busyTimeout) { php_sqlite3_db_object *db_obj; zval *object = getThis(); - long ms; + zend_long ms; int return_code; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) - if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &ms)) { + if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS(), "l", &ms)) { return; } @@ -353,12 +347,12 @@ PHP_METHOD(sqlite3, loadExtension) zval *object = getThis(); char *extension, *lib_path, *extension_dir, *errtext = NULL; char fullpath[MAXPATHLEN]; - int extension_len, extension_dir_len; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + size_t extension_len, extension_dir_len; + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) - if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &extension, &extension_len)) { + if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS(), "s", &extension, &extension_len)) { return; } @@ -424,7 +418,7 @@ PHP_METHOD(sqlite3, changes) { php_sqlite3_db_object *db_obj; zval *object = getThis(); - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) @@ -440,17 +434,17 @@ PHP_METHOD(sqlite3, changes) Returns a string that has been properly escaped. */ PHP_METHOD(sqlite3, escapeString) { - char *sql, *ret; - int sql_len; + zend_string *sql; + char *ret; - if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &sql, &sql_len)) { + if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS(), "S", &sql)) { return; } - if (sql_len) { - ret = sqlite3_mprintf("%q", sql); + if (ZSTR_LEN(sql)) { + ret = sqlite3_mprintf("%q", ZSTR_VAL(sql)); if (ret) { - RETVAL_STRING(ret, 1); + RETVAL_STRING(ret); sqlite3_free(ret); } } else { @@ -466,30 +460,28 @@ PHP_METHOD(sqlite3, prepare) php_sqlite3_db_object *db_obj; php_sqlite3_stmt *stmt_obj; zval *object = getThis(); - char *sql; - int sql_len, errcode; + zend_string *sql; + int errcode; php_sqlite3_free_list *free_item; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) - if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &sql, &sql_len)) { + if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS(), "S", &sql)) { return; } - if (!sql_len) { + if (!ZSTR_LEN(sql)) { RETURN_FALSE; } object_init_ex(return_value, php_sqlite3_stmt_entry); - stmt_obj = (php_sqlite3_stmt *)zend_object_store_get_object(return_value TSRMLS_CC); + stmt_obj = Z_SQLITE3_STMT_P(return_value); stmt_obj->db_obj = db_obj; - stmt_obj->db_obj_zval = getThis(); - - Z_ADDREF_P(object); + ZVAL_COPY(&stmt_obj->db_obj_zval, object); - errcode = sqlite3_prepare_v2(db_obj->db, sql, sql_len, &(stmt_obj->stmt), NULL); + errcode = sqlite3_prepare_v2(db_obj->db, ZSTR_VAL(sql), ZSTR_LEN(sql), &(stmt_obj->stmt), NULL); if (errcode != SQLITE_OK) { php_sqlite3_error(db_obj, "Unable to prepare statement: %d, %s", errcode, sqlite3_errmsg(db_obj->db)); zval_dtor(return_value); @@ -500,7 +492,7 @@ PHP_METHOD(sqlite3, prepare) free_item = emalloc(sizeof(php_sqlite3_free_list)); free_item->stmt_obj = stmt_obj; - free_item->stmt_obj_zval = return_value; + ZVAL_COPY_VALUE(&free_item->stmt_obj_zval, return_value); zend_llist_add_element(&(db_obj->free_list), &free_item); } @@ -514,40 +506,37 @@ PHP_METHOD(sqlite3, query) php_sqlite3_result *result; php_sqlite3_stmt *stmt_obj; zval *object = getThis(); - zval *stmt = NULL; - char *sql, *errtext = NULL; - int sql_len, return_code; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + zval stmt; + zend_string *sql; + char *errtext = NULL; + int return_code; + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) - if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &sql, &sql_len)) { + if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS(), "S", &sql)) { return; } - if (!sql_len) { + if (!ZSTR_LEN(sql)) { RETURN_FALSE; } /* If there was no return value then just execute the query */ - if (!return_value_used) { - if (sqlite3_exec(db_obj->db, sql, NULL, NULL, &errtext) != SQLITE_OK) { + if (!USED_RET()) { + if (sqlite3_exec(db_obj->db, ZSTR_VAL(sql), NULL, NULL, &errtext) != SQLITE_OK) { php_sqlite3_error(db_obj, "%s", errtext); sqlite3_free(errtext); } return; } - MAKE_STD_ZVAL(stmt); - - object_init_ex(stmt, php_sqlite3_stmt_entry); - stmt_obj = (php_sqlite3_stmt *)zend_object_store_get_object(stmt TSRMLS_CC); + object_init_ex(&stmt, php_sqlite3_stmt_entry); + stmt_obj = Z_SQLITE3_STMT_P(&stmt); stmt_obj->db_obj = db_obj; - stmt_obj->db_obj_zval = getThis(); - - Z_ADDREF_P(object); + ZVAL_COPY(&stmt_obj->db_obj_zval, object); - return_code = sqlite3_prepare_v2(db_obj->db, sql, sql_len, &(stmt_obj->stmt), NULL); + return_code = sqlite3_prepare_v2(db_obj->db, ZSTR_VAL(sql), ZSTR_LEN(sql), &(stmt_obj->stmt), NULL); if (return_code != SQLITE_OK) { php_sqlite3_error(db_obj, "Unable to prepare statement: %d, %s", return_code, sqlite3_errmsg(db_obj->db)); zval_ptr_dtor(&stmt); @@ -557,10 +546,10 @@ PHP_METHOD(sqlite3, query) stmt_obj->initialised = 1; object_init_ex(return_value, php_sqlite3_result_entry); - result = (php_sqlite3_result *)zend_object_store_get_object(return_value TSRMLS_CC); + result = Z_SQLITE3_RESULT_P(return_value); result->db_obj = db_obj; result->stmt_obj = stmt_obj; - result->stmt_obj_zval = stmt; + ZVAL_COPY_VALUE(&result->stmt_obj_zval, &stmt); return_code = sqlite3_step(result->stmt_obj->stmt); @@ -586,14 +575,12 @@ PHP_METHOD(sqlite3, query) } /* }}} */ -static zval* sqlite_value_to_zval(sqlite3_stmt *stmt, int column) /* {{{ */ +static void sqlite_value_to_zval(sqlite3_stmt *stmt, int column, zval *data) /* {{{ */ { - zval *data; - MAKE_STD_ZVAL(data); switch (sqlite3_column_type(stmt, column)) { case SQLITE_INTEGER: if ((sqlite3_column_int64(stmt, column)) >= INT_MAX || sqlite3_column_int64(stmt, column) <= INT_MIN) { - ZVAL_STRINGL(data, (char *)sqlite3_column_text(stmt, column), sqlite3_column_bytes(stmt, column), 1); + ZVAL_STRINGL(data, (char *)sqlite3_column_text(stmt, column), sqlite3_column_bytes(stmt, column)); } else { ZVAL_LONG(data, sqlite3_column_int64(stmt, column)); } @@ -608,14 +595,13 @@ static zval* sqlite_value_to_zval(sqlite3_stmt *stmt, int column) /* {{{ */ break; case SQLITE3_TEXT: - ZVAL_STRING(data, (char*)sqlite3_column_text(stmt, column), 1); + ZVAL_STRING(data, (char*)sqlite3_column_text(stmt, column)); break; case SQLITE_BLOB: default: - ZVAL_STRINGL(data, (char*)sqlite3_column_blob(stmt, column), sqlite3_column_bytes(stmt, column), 1); + ZVAL_STRINGL(data, (char*)sqlite3_column_blob(stmt, column), sqlite3_column_bytes(stmt, column)); } - return data; } /* }}} */ @@ -625,32 +611,33 @@ PHP_METHOD(sqlite3, querySingle) { php_sqlite3_db_object *db_obj; zval *object = getThis(); - char *sql, *errtext = NULL; - int sql_len, return_code; + zend_string *sql; + char *errtext = NULL; + int return_code; zend_bool entire_row = 0; sqlite3_stmt *stmt; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) - if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s|b", &sql, &sql_len, &entire_row)) { + if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS(), "S|b", &sql, &entire_row)) { return; } - if (!sql_len) { + if (!ZSTR_LEN(sql)) { RETURN_FALSE; } /* If there was no return value then just execute the query */ - if (!return_value_used) { - if (sqlite3_exec(db_obj->db, sql, NULL, NULL, &errtext) != SQLITE_OK) { + if (!USED_RET()) { + if (sqlite3_exec(db_obj->db, ZSTR_VAL(sql), NULL, NULL, &errtext) != SQLITE_OK) { php_sqlite3_error(db_obj, "%s", errtext); sqlite3_free(errtext); } return; } - return_code = sqlite3_prepare_v2(db_obj->db, sql, sql_len, &stmt, NULL); + return_code = sqlite3_prepare_v2(db_obj->db, ZSTR_VAL(sql), ZSTR_LEN(sql), &stmt, NULL); if (return_code != SQLITE_OK) { php_sqlite3_error(db_obj, "Unable to prepare statement: %d, %s", return_code, sqlite3_errmsg(db_obj->db)); RETURN_FALSE; @@ -662,19 +649,14 @@ PHP_METHOD(sqlite3, querySingle) case SQLITE_ROW: /* Valid Row */ { if (!entire_row) { - zval *data; - data = sqlite_value_to_zval(stmt, 0); - *return_value = *data; - zval_copy_ctor(return_value); - zval_dtor(data); - FREE_ZVAL(data); + sqlite_value_to_zval(stmt, 0, return_value); } else { int i = 0; array_init(return_value); for (i = 0; i < sqlite3_data_count(stmt); i++) { - zval *data; - data = sqlite_value_to_zval(stmt, i); - add_assoc_zval(return_value, (char*)sqlite3_column_name(stmt, i), data); + zval data; + sqlite_value_to_zval(stmt, i, &data); + add_assoc_zval(return_value, (char*)sqlite3_column_name(stmt, i), &data); } } break; @@ -696,10 +678,10 @@ PHP_METHOD(sqlite3, querySingle) } /* }}} */ -static int sqlite3_do_callback(struct php_sqlite3_fci *fc, zval *cb, int argc, sqlite3_value **argv, sqlite3_context *context, int is_agg TSRMLS_DC) /* {{{ */ +static int sqlite3_do_callback(struct php_sqlite3_fci *fc, zval *cb, int argc, sqlite3_value **argv, sqlite3_context *context, int is_agg) /* {{{ */ { - zval ***zargs = NULL; - zval *retval = NULL; + zval *zargs = NULL; + zval retval; int i; int ret; int fake_argc; @@ -713,77 +695,72 @@ static int sqlite3_do_callback(struct php_sqlite3_fci *fc, zval *cb, int argc, s fc->fci.size = sizeof(fc->fci); fc->fci.function_table = EG(function_table); - fc->fci.function_name = cb; + ZVAL_COPY_VALUE(&fc->fci.function_name, cb); fc->fci.symbol_table = NULL; - fc->fci.object_ptr = NULL; - fc->fci.retval_ptr_ptr = &retval; + fc->fci.object = NULL; + fc->fci.retval = &retval; fc->fci.param_count = fake_argc; /* build up the params */ if (fake_argc) { - zargs = (zval ***)safe_emalloc(fake_argc, sizeof(zval **), 0); + zargs = (zval *)safe_emalloc(fake_argc, sizeof(zval), 0); } if (is_agg) { /* summon the aggregation context */ agg_context = (php_sqlite3_agg_context *)sqlite3_aggregate_context(context, sizeof(php_sqlite3_agg_context)); - if (!agg_context->zval_context) { - MAKE_STD_ZVAL(agg_context->zval_context); - ZVAL_NULL(agg_context->zval_context); + if (Z_ISUNDEF(agg_context->zval_context)) { + ZVAL_NULL(&agg_context->zval_context); } - zargs[0] = &agg_context->zval_context; - - zargs[1] = emalloc(sizeof(zval*)); - MAKE_STD_ZVAL(*zargs[1]); - ZVAL_LONG(*zargs[1], agg_context->row_count); + ZVAL_DUP(&zargs[0], &agg_context->zval_context); + ZVAL_LONG(&zargs[1], agg_context->row_count); } for (i = 0; i < argc; i++) { - zargs[i + is_agg] = emalloc(sizeof(zval *)); - MAKE_STD_ZVAL(*zargs[i + is_agg]); - switch (sqlite3_value_type(argv[i])) { case SQLITE_INTEGER: -#if LONG_MAX > 2147483647 - ZVAL_LONG(*zargs[i + is_agg], sqlite3_value_int64(argv[i])); +#if ZEND_LONG_MAX > 2147483647 + ZVAL_LONG(&zargs[i + is_agg], sqlite3_value_int64(argv[i])); #else - ZVAL_LONG(*zargs[i + is_agg], sqlite3_value_int(argv[i])); + ZVAL_LONG(&zargs[i + is_agg], sqlite3_value_int(argv[i])); #endif break; case SQLITE_FLOAT: - ZVAL_DOUBLE(*zargs[i + is_agg], sqlite3_value_double(argv[i])); + ZVAL_DOUBLE(&zargs[i + is_agg], sqlite3_value_double(argv[i])); break; case SQLITE_NULL: - ZVAL_NULL(*zargs[i + is_agg]); + ZVAL_NULL(&zargs[i + is_agg]); break; case SQLITE_BLOB: case SQLITE3_TEXT: default: - ZVAL_STRINGL(*zargs[i + is_agg], (char*)sqlite3_value_text(argv[i]), sqlite3_value_bytes(argv[i]), 1); + ZVAL_STRINGL(&zargs[i + is_agg], (char*)sqlite3_value_text(argv[i]), sqlite3_value_bytes(argv[i])); break; } } fc->fci.params = zargs; - if ((ret = zend_call_function(&fc->fci, &fc->fcc TSRMLS_CC)) == FAILURE) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "An error occurred while invoking the callback"); + if ((ret = zend_call_function(&fc->fci, &fc->fcc)) == FAILURE) { + php_error_docref(NULL, E_WARNING, "An error occurred while invoking the callback"); + } + + if (is_agg) { + zval_ptr_dtor(&zargs[0]); } /* clean up the params */ if (fake_argc) { for (i = is_agg; i < argc + is_agg; i++) { - zval_ptr_dtor(zargs[i]); - efree(zargs[i]); + zval_ptr_dtor(&zargs[i]); } if (is_agg) { - zval_ptr_dtor(zargs[1]); - efree(zargs[1]); + zval_ptr_dtor(&zargs[1]); } efree(zargs); } @@ -791,13 +768,13 @@ static int sqlite3_do_callback(struct php_sqlite3_fci *fc, zval *cb, int argc, s if (!is_agg || !argv) { /* only set the sqlite return value if we are a scalar function, * or if we are finalizing an aggregate */ - if (retval) { - switch (Z_TYPE_P(retval)) { + if (!Z_ISUNDEF(retval)) { + switch (Z_TYPE(retval)) { case IS_LONG: -#if LONG_MAX > 2147483647 - sqlite3_result_int64(context, Z_LVAL_P(retval)); +#if ZEND_LONG_MAX > 2147483647 + sqlite3_result_int64(context, Z_LVAL(retval)); #else - sqlite3_result_int(context, Z_LVAL_P(retval)); + sqlite3_result_int(context, Z_LVAL(retval)); #endif break; @@ -806,36 +783,32 @@ static int sqlite3_do_callback(struct php_sqlite3_fci *fc, zval *cb, int argc, s break; case IS_DOUBLE: - sqlite3_result_double(context, Z_DVAL_P(retval)); + sqlite3_result_double(context, Z_DVAL(retval)); break; default: convert_to_string_ex(&retval); - sqlite3_result_text(context, Z_STRVAL_P(retval), Z_STRLEN_P(retval), SQLITE_TRANSIENT); + sqlite3_result_text(context, Z_STRVAL(retval), Z_STRLEN(retval), SQLITE_TRANSIENT); break; } } else { sqlite3_result_error(context, "failed to invoke callback", 0); } - if (agg_context && agg_context->zval_context) { + if (agg_context && !Z_ISUNDEF(agg_context->zval_context)) { zval_ptr_dtor(&agg_context->zval_context); } } else { /* we're stepping in an aggregate; the return value goes into * the context */ - if (agg_context && agg_context->zval_context) { + if (agg_context && !Z_ISUNDEF(agg_context->zval_context)) { zval_ptr_dtor(&agg_context->zval_context); } - if (retval) { - agg_context->zval_context = retval; - retval = NULL; - } else { - agg_context->zval_context = NULL; - } + ZVAL_COPY_VALUE(&agg_context->zval_context, &retval); + ZVAL_UNDEF(&retval); } - if (retval) { + if (!Z_ISUNDEF(retval)) { zval_ptr_dtor(&retval); } return ret; @@ -845,9 +818,8 @@ static int sqlite3_do_callback(struct php_sqlite3_fci *fc, zval *cb, int argc, s static void php_sqlite3_callback_func(sqlite3_context *context, int argc, sqlite3_value **argv) /* {{{ */ { php_sqlite3_func *func = (php_sqlite3_func *)sqlite3_user_data(context); - TSRMLS_FETCH(); - sqlite3_do_callback(&func->afunc, func->func, argc, argv, context, 0 TSRMLS_CC); + sqlite3_do_callback(&func->afunc, &func->func, argc, argv, context, 0); } /* }}}*/ @@ -856,10 +828,9 @@ static void php_sqlite3_callback_step(sqlite3_context *context, int argc, sqlite php_sqlite3_func *func = (php_sqlite3_func *)sqlite3_user_data(context); php_sqlite3_agg_context *agg_context = (php_sqlite3_agg_context *)sqlite3_aggregate_context(context, sizeof(php_sqlite3_agg_context)); - TSRMLS_FETCH(); agg_context->row_count++; - sqlite3_do_callback(&func->astep, func->step, argc, argv, context, 1 TSRMLS_CC); + sqlite3_do_callback(&func->astep, &func->step, argc, argv, context, 1); } /* }}} */ @@ -868,67 +839,58 @@ static void php_sqlite3_callback_final(sqlite3_context *context) /* {{{ */ php_sqlite3_func *func = (php_sqlite3_func *)sqlite3_user_data(context); php_sqlite3_agg_context *agg_context = (php_sqlite3_agg_context *)sqlite3_aggregate_context(context, sizeof(php_sqlite3_agg_context)); - TSRMLS_FETCH(); agg_context->row_count = 0; - sqlite3_do_callback(&func->afini, func->fini, 0, NULL, context, 1 TSRMLS_CC); + sqlite3_do_callback(&func->afini, &func->fini, 0, NULL, context, 1); } /* }}} */ static int php_sqlite3_callback_compare(void *coll, int a_len, const void *a, int b_len, const void* b) /* {{{ */ { php_sqlite3_collation *collation = (php_sqlite3_collation*)coll; - zval ***zargs = NULL; - zval *retval = NULL; + zval *zargs = NULL; + zval retval; int ret; - TSRMLS_FETCH(); - collation->fci.fci.size = (sizeof(collation->fci.fci)); collation->fci.fci.function_table = EG(function_table); - collation->fci.fci.function_name = collation->cmp_func; + ZVAL_COPY_VALUE(&collation->fci.fci.function_name, &collation->cmp_func); collation->fci.fci.symbol_table = NULL; - collation->fci.fci.object_ptr = NULL; - collation->fci.fci.retval_ptr_ptr = &retval; + collation->fci.fci.object = NULL; + collation->fci.fci.retval = &retval; collation->fci.fci.param_count = 2; - zargs = (zval***)safe_emalloc(2, sizeof(zval**), 0); - zargs[0] = emalloc(sizeof(zval*)); - zargs[1] = emalloc(sizeof(zval*)); - - MAKE_STD_ZVAL(*zargs[0]); - ZVAL_STRINGL(*zargs[0], a, a_len, 1); + zargs = safe_emalloc(2, sizeof(zval), 0); + ZVAL_STRINGL(&zargs[0], a, a_len); + ZVAL_STRINGL(&zargs[1], b, b_len); - MAKE_STD_ZVAL(*zargs[1]); - ZVAL_STRINGL(*zargs[1], b, b_len, 1); - collation->fci.fci.params = zargs; - if ((ret = zend_call_function(&collation->fci.fci, &collation->fci.fcc TSRMLS_CC)) == FAILURE) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "An error occurred while invoking the compare callback"); + if (!EG(exception)) { + //Exception occurred on previous callback. Don't attempt to call function + if ((ret = zend_call_function(&collation->fci.fci, &collation->fci.fcc)) == FAILURE) { + php_error_docref(NULL, E_WARNING, "An error occurred while invoking the compare callback"); + } + } else { + ZVAL_UNDEF(&retval); } - zval_ptr_dtor(zargs[0]); - zval_ptr_dtor(zargs[1]); - efree(zargs[0]); - efree(zargs[1]); + zval_ptr_dtor(&zargs[0]); + zval_ptr_dtor(&zargs[1]); efree(zargs); - if (!retval) { - //Exception was thrown by callback, default to 0 for compare + if (EG(exception)) { ret = 0; - } else if (Z_TYPE_P(retval) != IS_LONG) { + } else if (Z_TYPE(retval) != IS_LONG){ //retval ought to contain a ZVAL_LONG by now - // (the result of a comparison, i.e. most likely -1, 0, or 1) - //I suppose we could accept any scalar return type, though. - php_error_docref(NULL TSRMLS_CC, E_WARNING, "An error occurred while invoking the compare callback (invalid return type). Collation behaviour is undefined."); + // (the result of a comparison, i.e. most likely -1, 0, or 1) + //I suppose we could accept any scalar return type, though. + php_error_docref(NULL, E_WARNING, "An error occurred while invoking the compare callback (invalid return type). Collation behaviour is undefined."); } else { - ret = Z_LVAL_P(retval); + ret = Z_LVAL(retval); } - if (retval) { - zval_ptr_dtor(&retval); - } + zval_ptr_dtor(&retval); return ret; } @@ -941,15 +903,16 @@ PHP_METHOD(sqlite3, createFunction) php_sqlite3_db_object *db_obj; zval *object = getThis(); php_sqlite3_func *func; - char *sql_func, *callback_name; - int sql_func_len; + char *sql_func; + size_t sql_func_len; zval *callback_func; - long sql_func_num_args = -1; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + zend_string *callback_name; + zend_long sql_func_num_args = -1; + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "sz|l", &sql_func, &sql_func_len, &callback_func, &sql_func_num_args) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "sz|l", &sql_func, &sql_func_len, &callback_func, &sql_func_num_args) == FAILURE) { return; } @@ -957,20 +920,19 @@ PHP_METHOD(sqlite3, createFunction) RETURN_FALSE; } - if (!zend_is_callable(callback_func, 0, &callback_name TSRMLS_CC)) { - php_sqlite3_error(db_obj, "Not a valid callback function %s", callback_name); - efree(callback_name); + if (!zend_is_callable(callback_func, 0, &callback_name)) { + php_sqlite3_error(db_obj, "Not a valid callback function %s", ZSTR_VAL(callback_name)); + zend_string_release(callback_name); RETURN_FALSE; } - efree(callback_name); + zend_string_release(callback_name); func = (php_sqlite3_func *)ecalloc(1, sizeof(*func)); if (sqlite3_create_function(db_obj->db, sql_func, sql_func_num_args, SQLITE_UTF8, func, php_sqlite3_callback_func, NULL, NULL) == SQLITE_OK) { func->func_name = estrdup(sql_func); - MAKE_STD_ZVAL(func->func); - MAKE_COPY_ZVAL(&callback_func, func->func); + ZVAL_COPY(&func->func, callback_func); func->argc = sql_func_num_args; func->next = db_obj->funcs; @@ -991,15 +953,16 @@ PHP_METHOD(sqlite3, createAggregate) php_sqlite3_db_object *db_obj; zval *object = getThis(); php_sqlite3_func *func; - char *sql_func, *callback_name; - int sql_func_len; + char *sql_func; + zend_string *callback_name; + size_t sql_func_len; zval *step_callback, *fini_callback; - long sql_func_num_args = -1; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + zend_long sql_func_num_args = -1; + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "szz|l", &sql_func, &sql_func_len, &step_callback, &fini_callback, &sql_func_num_args) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "szz|l", &sql_func, &sql_func_len, &step_callback, &fini_callback, &sql_func_num_args) == FAILURE) { return; } @@ -1007,30 +970,27 @@ PHP_METHOD(sqlite3, createAggregate) RETURN_FALSE; } - if (!zend_is_callable(step_callback, 0, &callback_name TSRMLS_CC)) { - php_sqlite3_error(db_obj, "Not a valid callback function %s", callback_name); - efree(callback_name); + if (!zend_is_callable(step_callback, 0, &callback_name)) { + php_sqlite3_error(db_obj, "Not a valid callback function %s", ZSTR_VAL(callback_name)); + zend_string_release(callback_name); RETURN_FALSE; } - efree(callback_name); + zend_string_release(callback_name); - if (!zend_is_callable(fini_callback, 0, &callback_name TSRMLS_CC)) { - php_sqlite3_error(db_obj, "Not a valid callback function %s", callback_name); - efree(callback_name); + if (!zend_is_callable(fini_callback, 0, &callback_name)) { + php_sqlite3_error(db_obj, "Not a valid callback function %s", ZSTR_VAL(callback_name)); + zend_string_release(callback_name); RETURN_FALSE; } - efree(callback_name); + zend_string_release(callback_name); func = (php_sqlite3_func *)ecalloc(1, sizeof(*func)); if (sqlite3_create_function(db_obj->db, sql_func, sql_func_num_args, SQLITE_UTF8, func, NULL, php_sqlite3_callback_step, php_sqlite3_callback_final) == SQLITE_OK) { func->func_name = estrdup(sql_func); - MAKE_STD_ZVAL(func->step); - MAKE_COPY_ZVAL(&step_callback, func->step); - - MAKE_STD_ZVAL(func->fini); - MAKE_COPY_ZVAL(&fini_callback, func->fini); + ZVAL_COPY(&func->step, step_callback); + ZVAL_COPY(&func->fini, fini_callback); func->argc = sql_func_num_args; func->next = db_obj->funcs; @@ -1051,14 +1011,15 @@ PHP_METHOD(sqlite3, createCollation) php_sqlite3_db_object *db_obj; zval *object = getThis(); php_sqlite3_collation *collation; - char *collation_name, *callback_name; - int collation_name_len; + char *collation_name; + zend_string *callback_name; + size_t collation_name_len; zval *callback_func; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "sz", &collation_name, &collation_name_len, &callback_func) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "sz", &collation_name, &collation_name_len, &callback_func) == FAILURE) { RETURN_FALSE; } @@ -1066,19 +1027,18 @@ PHP_METHOD(sqlite3, createCollation) RETURN_FALSE; } - if (!zend_is_callable(callback_func, 0, &callback_name TSRMLS_CC)) { - php_sqlite3_error(db_obj, "Not a valid callback function %s", callback_name); - efree(callback_name); + if (!zend_is_callable(callback_func, 0, &callback_name)) { + php_sqlite3_error(db_obj, "Not a valid callback function %s", ZSTR_VAL(callback_name)); + zend_string_release(callback_name); RETURN_FALSE; } - efree(callback_name); + zend_string_release(callback_name); collation = (php_sqlite3_collation *)ecalloc(1, sizeof(*collation)); if (sqlite3_create_collation(db_obj->db, collation_name, SQLITE_UTF8, collation, php_sqlite3_callback_compare) == SQLITE_OK) { collation->collation_name = estrdup(collation_name); - MAKE_STD_ZVAL(collation->cmp_func); - MAKE_COPY_ZVAL(&callback_func, collation->cmp_func); + ZVAL_COPY(&collation->cmp_func, callback_func); collation->next = db_obj->collations; db_obj->collations = collation; @@ -1097,14 +1057,14 @@ typedef struct { size_t size; } php_stream_sqlite3_data; -static size_t php_sqlite3_stream_write(php_stream *stream, const char *buf, size_t count TSRMLS_DC) +static size_t php_sqlite3_stream_write(php_stream *stream, const char *buf, size_t count) { /* php_stream_sqlite3_data *sqlite3_stream = (php_stream_sqlite3_data *) stream->abstract; */ - + return 0; } -static size_t php_sqlite3_stream_read(php_stream *stream, char *buf, size_t count TSRMLS_DC) +static size_t php_sqlite3_stream_read(php_stream *stream, char *buf, size_t count) { php_stream_sqlite3_data *sqlite3_stream = (php_stream_sqlite3_data *) stream->abstract; @@ -1121,27 +1081,27 @@ static size_t php_sqlite3_stream_read(php_stream *stream, char *buf, size_t coun return count; } -static int php_sqlite3_stream_close(php_stream *stream, int close_handle TSRMLS_DC) +static int php_sqlite3_stream_close(php_stream *stream, int close_handle) { php_stream_sqlite3_data *sqlite3_stream = (php_stream_sqlite3_data *) stream->abstract; - + if (sqlite3_blob_close(sqlite3_stream->blob) != SQLITE_OK) { /* Error occurred, but it still closed */ } efree(sqlite3_stream); - + return 0; } -static int php_sqlite3_stream_flush(php_stream *stream TSRMLS_DC) +static int php_sqlite3_stream_flush(php_stream *stream) { /* do nothing */ return 0; } /* {{{ */ -static int php_sqlite3_stream_seek(php_stream *stream, off_t offset, int whence, off_t *newoffs TSRMLS_DC) +static int php_sqlite3_stream_seek(php_stream *stream, zend_off_t offset, int whence, zend_off_t *newoffs) { php_stream_sqlite3_data *sqlite3_stream = (php_stream_sqlite3_data *) stream->abstract; @@ -1204,12 +1164,12 @@ static int php_sqlite3_stream_seek(php_stream *stream, off_t offset, int whence, /* }}} */ -static int php_sqlite3_stream_cast(php_stream *stream, int castas, void **ret TSRMLS_DC) +static int php_sqlite3_stream_cast(php_stream *stream, int castas, void **ret) { return FAILURE; } -static int php_sqlite3_stream_stat(php_stream *stream, php_stream_statbuf *ssb TSRMLS_DC) +static int php_sqlite3_stream_stat(php_stream *stream, php_stream_statbuf *ssb) { php_stream_sqlite3_data *sqlite3_stream = (php_stream_sqlite3_data *) stream->abstract; ssb->sb.st_size = sqlite3_stream->size; @@ -1234,17 +1194,17 @@ PHP_METHOD(sqlite3, openBlob) php_sqlite3_db_object *db_obj; zval *object = getThis(); char *table, *column, *dbname = "main"; - int table_len, column_len, dbname_len; - long rowid, flags = 0; + size_t table_len, column_len, dbname_len; + zend_long rowid, flags = 0; sqlite3_blob *blob = NULL; php_stream_sqlite3_data *sqlite3_stream; php_stream *stream; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(object); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ssl|s", &table, &table_len, &column, &column_len, &rowid, &dbname, &dbname_len) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "ssl|s", &table, &table_len, &column, &column_len, &rowid, &dbname, &dbname_len) == FAILURE) { return; } @@ -1257,7 +1217,7 @@ PHP_METHOD(sqlite3, openBlob) sqlite3_stream->blob = blob; sqlite3_stream->position = 0; sqlite3_stream->size = sqlite3_blob_bytes(blob); - + stream = php_stream_alloc(&php_stream_sqlite3_ops, sqlite3_stream, 0, "rb"); if (stream) { @@ -1276,9 +1236,9 @@ PHP_METHOD(sqlite3, enableExceptions) zval *object = getThis(); zend_bool enableExceptions = 0; - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(object TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(object); - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|b", &enableExceptions) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "|b", &enableExceptions) == FAILURE) { return; } @@ -1294,14 +1254,13 @@ PHP_METHOD(sqlite3stmt, paramCount) { php_sqlite3_stmt *stmt_obj; zval *object = getThis(); - stmt_obj = (php_sqlite3_stmt *)zend_object_store_get_object(object TSRMLS_CC); - - SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3) + stmt_obj = Z_SQLITE3_STMT_P(object); if (zend_parse_parameters_none() == FAILURE) { return; } + SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3); SQLITE3_CHECK_INITIALIZED_STMT(stmt_obj->stmt, SQLite3Stmt); RETURN_LONG(sqlite3_bind_parameter_count(stmt_obj->stmt)); @@ -1314,14 +1273,14 @@ PHP_METHOD(sqlite3stmt, close) { php_sqlite3_stmt *stmt_obj; zval *object = getThis(); - stmt_obj = (php_sqlite3_stmt *)zend_object_store_get_object(object TSRMLS_CC); - - SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3) + stmt_obj = Z_SQLITE3_STMT_P(object); if (zend_parse_parameters_none() == FAILURE) { return; } + SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3); + if(stmt_obj->db_obj) { zend_llist_del_element(&(stmt_obj->db_obj->free_list), object, (int (*)(void *, void *)) php_sqlite3_compare_stmt_zval_free); } @@ -1336,14 +1295,13 @@ PHP_METHOD(sqlite3stmt, reset) { php_sqlite3_stmt *stmt_obj; zval *object = getThis(); - stmt_obj = (php_sqlite3_stmt *)zend_object_store_get_object(object TSRMLS_CC); - - SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3) + stmt_obj = Z_SQLITE3_STMT_P(object); if (zend_parse_parameters_none() == FAILURE) { return; } + SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3); SQLITE3_CHECK_INITIALIZED_STMT(stmt_obj->stmt, SQLite3Stmt); if (sqlite3_reset(stmt_obj->stmt) != SQLITE_OK) { @@ -1360,14 +1318,13 @@ PHP_METHOD(sqlite3stmt, clear) { php_sqlite3_stmt *stmt_obj; zval *object = getThis(); - stmt_obj = (php_sqlite3_stmt *)zend_object_store_get_object(object TSRMLS_CC); - - SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3) + stmt_obj = Z_SQLITE3_STMT_P(object); if (zend_parse_parameters_none() == FAILURE) { return; } + SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3); SQLITE3_CHECK_INITIALIZED_STMT(stmt_obj->stmt, SQLite3Stmt); if (sqlite3_clear_bindings(stmt_obj->stmt) != SQLITE_OK) { @@ -1385,14 +1342,13 @@ PHP_METHOD(sqlite3stmt, readOnly) { php_sqlite3_stmt *stmt_obj; zval *object = getThis(); - stmt_obj = (php_sqlite3_stmt *)zend_object_store_get_object(object TSRMLS_CC); - - SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3) + stmt_obj = Z_SQLITE3_STMT_P(object); if (zend_parse_parameters_none() == FAILURE) { return; } + SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3); SQLITE3_CHECK_INITIALIZED_STMT(stmt_obj->stmt, SQLite3Stmt); #if SQLITE_VERSION_NUMBER >= 3007004 @@ -1404,7 +1360,7 @@ PHP_METHOD(sqlite3stmt, readOnly) } /* }}} */ -static int register_bound_parameter_to_sqlite(struct php_sqlite3_bound_param *param, php_sqlite3_stmt *stmt TSRMLS_DC) /* {{{ */ +static int register_bound_parameter_to_sqlite(struct php_sqlite3_bound_param *param, php_sqlite3_stmt *stmt) /* {{{ */ { HashTable *hash; hash = stmt->bound_params; @@ -1417,21 +1373,21 @@ static int register_bound_parameter_to_sqlite(struct php_sqlite3_bound_param *pa /* We need a : prefix to resolve a name to a parameter number */ if (param->name) { - if (param->name[0] != ':') { + if (ZSTR_VAL(param->name)[0] != ':') { /* pre-increment for character + 1 for null */ - char *temp = emalloc(++param->name_len + 1); - temp[0] = ':'; - memmove(temp+1, param->name, param->name_len); + zend_string *temp = zend_string_alloc(ZSTR_LEN(param->name) + 1, 0); + ZSTR_VAL(temp)[0] = ':'; + memmove(ZSTR_VAL(temp) + 1, ZSTR_VAL(param->name), ZSTR_LEN(param->name) + 1); param->name = temp; } else { - param->name = estrndup(param->name, param->name_len); + param->name = zend_string_init(ZSTR_VAL(param->name), ZSTR_LEN(param->name), 0); } /* do lookup*/ - param->param_number = sqlite3_bind_parameter_index(stmt->stmt, param->name); + param->param_number = sqlite3_bind_parameter_index(stmt->stmt, ZSTR_VAL(param->name)); } if (param->param_number < 1) { - efree(param->name); + zend_string_release(param->name); return 0; } @@ -1440,9 +1396,9 @@ static int register_bound_parameter_to_sqlite(struct php_sqlite3_bound_param *pa } if (param->name) { - zend_hash_update(hash, param->name, param->name_len, param, sizeof(*param), NULL); + zend_hash_update_mem(hash, param->name, param, sizeof(struct php_sqlite3_bound_param)); } else { - zend_hash_index_update(hash, param->param_number, param, sizeof(*param), NULL); + zend_hash_index_update_mem(hash, param->param_number, param, sizeof(struct php_sqlite3_bound_param)); } return 1; @@ -1456,27 +1412,27 @@ PHP_METHOD(sqlite3stmt, bindParam) php_sqlite3_stmt *stmt_obj; zval *object = getThis(); struct php_sqlite3_bound_param param = {0}; - stmt_obj = (php_sqlite3_stmt *)zend_object_store_get_object(object TSRMLS_CC); - - SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3) + zval *parameter; + stmt_obj = Z_SQLITE3_STMT_P(object); param.param_number = -1; param.type = SQLITE3_TEXT; - if (zend_parse_parameters_ex(ZEND_PARSE_PARAMS_QUIET, ZEND_NUM_ARGS() TSRMLS_CC, "lz|l", ¶m.param_number, ¶m.parameter, ¶m.type) == FAILURE) { - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "sz|l", ¶m.name, ¶m.name_len, ¶m.parameter, ¶m.type) == FAILURE) { + if (zend_parse_parameters_ex(ZEND_PARSE_PARAMS_QUIET, ZEND_NUM_ARGS(), "lz|l", ¶m.param_number, ¶meter, ¶m.type) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "Sz|l", ¶m.name, ¶meter, ¶m.type) == FAILURE) { return; } } + SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3); SQLITE3_CHECK_INITIALIZED_STMT(stmt_obj->stmt, SQLite3Stmt); - Z_ADDREF_P(param.parameter); + ZVAL_COPY(¶m.parameter, parameter); - if (!register_bound_parameter_to_sqlite(¶m, stmt_obj TSRMLS_CC)) { - if (param.parameter) { + if (!register_bound_parameter_to_sqlite(¶m, stmt_obj)) { + if (!Z_ISUNDEF(param.parameter)) { zval_ptr_dtor(&(param.parameter)); - param.parameter = NULL; + ZVAL_UNDEF(¶m.parameter); } RETURN_FALSE; } @@ -1491,27 +1447,27 @@ PHP_METHOD(sqlite3stmt, bindValue) php_sqlite3_stmt *stmt_obj; zval *object = getThis(); struct php_sqlite3_bound_param param = {0}; - stmt_obj = (php_sqlite3_stmt *)zend_object_store_get_object(object TSRMLS_CC); - - SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3) + zval *parameter; + stmt_obj = Z_SQLITE3_STMT_P(object); param.param_number = -1; param.type = SQLITE3_TEXT; - if (zend_parse_parameters_ex(ZEND_PARSE_PARAMS_QUIET, ZEND_NUM_ARGS() TSRMLS_CC, "lz/|l", ¶m.param_number, ¶m.parameter, ¶m.type) == FAILURE) { - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "sz/|l", ¶m.name, ¶m.name_len, ¶m.parameter, ¶m.type) == FAILURE) { + if (zend_parse_parameters_ex(ZEND_PARSE_PARAMS_QUIET, ZEND_NUM_ARGS(), "lz/|l", ¶m.param_number, ¶meter, ¶m.type) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "Sz/|l", ¶m.name, ¶meter, ¶m.type) == FAILURE) { return; } } + SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3); SQLITE3_CHECK_INITIALIZED_STMT(stmt_obj->stmt, SQLite3Stmt); - Z_ADDREF_P(param.parameter); + ZVAL_COPY(¶m.parameter, parameter); - if (!register_bound_parameter_to_sqlite(¶m, stmt_obj TSRMLS_CC)) { - if (param.parameter) { + if (!register_bound_parameter_to_sqlite(¶m, stmt_obj)) { + if (!Z_ISUNDEF(param.parameter)) { zval_ptr_dtor(&(param.parameter)); - param.parameter = NULL; + ZVAL_UNDEF(¶m.parameter); } RETURN_FALSE; } @@ -1529,70 +1485,72 @@ PHP_METHOD(sqlite3stmt, execute) int return_code = 0; struct php_sqlite3_bound_param *param; - stmt_obj = (php_sqlite3_stmt *)zend_object_store_get_object(object TSRMLS_CC); - - SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3) + stmt_obj = Z_SQLITE3_STMT_P(object); if (zend_parse_parameters_none() == FAILURE) { return; } - SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3) + SQLITE3_CHECK_INITIALIZED(stmt_obj->db_obj, stmt_obj->initialised, SQLite3); if (stmt_obj->bound_params) { - zend_hash_internal_pointer_reset(stmt_obj->bound_params); - while (zend_hash_get_current_data(stmt_obj->bound_params, (void **)¶m) == SUCCESS) { + ZEND_HASH_FOREACH_PTR(stmt_obj->bound_params, param) { + zval *parameter; + /* parameter must be a reference? */ + if (Z_ISREF(param->parameter)) { + parameter = Z_REFVAL(param->parameter); + } else { + parameter = ¶m->parameter; + } + /* If the ZVAL is null then it should be bound as that */ - if (Z_TYPE_P(param->parameter) == IS_NULL) { + if (Z_TYPE_P(parameter) == IS_NULL) { sqlite3_bind_null(stmt_obj->stmt, param->param_number); - zend_hash_move_forward(stmt_obj->bound_params); continue; } switch (param->type) { case SQLITE_INTEGER: - convert_to_long(param->parameter); -#if LONG_MAX > 2147483647 - sqlite3_bind_int64(stmt_obj->stmt, param->param_number, Z_LVAL_P(param->parameter)); + convert_to_long(parameter); +#if ZEND_LONG_MAX > 2147483647 + sqlite3_bind_int64(stmt_obj->stmt, param->param_number, Z_LVAL_P(parameter)); #else - sqlite3_bind_int(stmt_obj->stmt, param->param_number, Z_LVAL_P(param->parameter)); + sqlite3_bind_int(stmt_obj->stmt, param->param_number, Z_LVAL_P(parameter)); #endif break; case SQLITE_FLOAT: - /* convert_to_double(param->parameter);*/ - sqlite3_bind_double(stmt_obj->stmt, param->param_number, Z_DVAL_P(param->parameter)); + /* convert_to_double(parameter);*/ + sqlite3_bind_double(stmt_obj->stmt, param->param_number, Z_DVAL_P(parameter)); break; case SQLITE_BLOB: { php_stream *stream = NULL; - int blength; - char *buffer = NULL; - if (Z_TYPE_P(param->parameter) == IS_RESOURCE) { - php_stream_from_zval_no_verify(stream, ¶m->parameter); + zend_string *buffer = NULL; + if (Z_TYPE_P(parameter) == IS_RESOURCE) { + php_stream_from_zval_no_verify(stream, parameter); if (stream == NULL) { php_sqlite3_error(stmt_obj->db_obj, "Unable to read stream for parameter %ld", param->param_number); RETURN_FALSE; } - blength = php_stream_copy_to_mem(stream, (void *)&buffer, PHP_STREAM_COPY_ALL, 0); + buffer = php_stream_copy_to_mem(stream, PHP_STREAM_COPY_ALL, 0); } else { - convert_to_string(param->parameter); - blength = Z_STRLEN_P(param->parameter); - buffer = Z_STRVAL_P(param->parameter); + buffer = zval_get_string(parameter); } - sqlite3_bind_blob(stmt_obj->stmt, param->param_number, buffer, blength, SQLITE_TRANSIENT); - - if (stream) { - pefree(buffer, 0); + if (buffer) { + sqlite3_bind_blob(stmt_obj->stmt, param->param_number, ZSTR_VAL(buffer), ZSTR_LEN(buffer), SQLITE_TRANSIENT); + zend_string_release(buffer); + } else { + sqlite3_bind_null(stmt_obj->stmt, param->param_number); } break; } case SQLITE3_TEXT: - convert_to_string(param->parameter); - sqlite3_bind_text(stmt_obj->stmt, param->param_number, Z_STRVAL_P(param->parameter), Z_STRLEN_P(param->parameter), SQLITE_STATIC); + convert_to_string(parameter); + sqlite3_bind_text(stmt_obj->stmt, param->param_number, Z_STRVAL_P(parameter), Z_STRLEN_P(parameter), SQLITE_STATIC); break; case SQLITE_NULL: @@ -1600,11 +1558,10 @@ PHP_METHOD(sqlite3stmt, execute) break; default: - php_sqlite3_error(stmt_obj->db_obj, "Unknown parameter type: %ld for parameter %ld", param->type, param->param_number); + php_sqlite3_error(stmt_obj->db_obj, "Unknown parameter type: %pd for parameter %pd", param->type, param->param_number); RETURN_FALSE; } - zend_hash_move_forward(stmt_obj->bound_params); - } + } ZEND_HASH_FOREACH_END(); } return_code = sqlite3_step(stmt_obj->stmt); @@ -1615,14 +1572,12 @@ PHP_METHOD(sqlite3stmt, execute) { sqlite3_reset(stmt_obj->stmt); object_init_ex(return_value, php_sqlite3_result_entry); - result = (php_sqlite3_result *)zend_object_store_get_object(return_value TSRMLS_CC); + result = Z_SQLITE3_RESULT_P(return_value); - Z_ADDREF_P(object); - result->is_prepared_statement = 1; result->db_obj = stmt_obj->db_obj; result->stmt_obj = stmt_obj; - result->stmt_obj_zval = getThis(); + ZVAL_COPY(&result->stmt_obj_zval, object); break; } @@ -1647,35 +1602,31 @@ PHP_METHOD(sqlite3stmt, __construct) php_sqlite3_db_object *db_obj; zval *object = getThis(); zval *db_zval; - char *sql; - int sql_len, errcode; + zend_string *sql; + int errcode; zend_error_handling error_handling; php_sqlite3_free_list *free_item; - stmt_obj = (php_sqlite3_stmt *)zend_object_store_get_object(object TSRMLS_CC); - zend_replace_error_handling(EH_THROW, NULL, &error_handling TSRMLS_CC); + stmt_obj = Z_SQLITE3_STMT_P(object); - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "Os", &db_zval, php_sqlite3_sc_entry, &sql, &sql_len) == FAILURE) { - zend_restore_error_handling(&error_handling TSRMLS_CC); + if (zend_parse_parameters_throw(ZEND_NUM_ARGS(), "OS", &db_zval, php_sqlite3_sc_entry, &sql) == FAILURE) { return; } - db_obj = (php_sqlite3_db_object *)zend_object_store_get_object(db_zval TSRMLS_CC); + db_obj = Z_SQLITE3_DB_P(db_zval); + zend_replace_error_handling(EH_THROW, NULL, &error_handling); SQLITE3_CHECK_INITIALIZED(db_obj, db_obj->initialised, SQLite3) + zend_restore_error_handling(&error_handling); - zend_restore_error_handling(&error_handling TSRMLS_CC); - - if (!sql_len) { + if (!ZSTR_LEN(sql)) { RETURN_FALSE; } stmt_obj->db_obj = db_obj; - stmt_obj->db_obj_zval = db_zval; + ZVAL_COPY(&stmt_obj->db_obj_zval, db_zval); - Z_ADDREF_P(db_zval); - - errcode = sqlite3_prepare_v2(db_obj->db, sql, sql_len, &(stmt_obj->stmt), NULL); + errcode = sqlite3_prepare_v2(db_obj->db, ZSTR_VAL(sql), ZSTR_LEN(sql), &(stmt_obj->stmt), NULL); if (errcode != SQLITE_OK) { php_sqlite3_error(db_obj, "Unable to prepare statement: %d, %s", errcode, sqlite3_errmsg(db_obj->db)); zval_dtor(return_value); @@ -1685,7 +1636,8 @@ PHP_METHOD(sqlite3stmt, __construct) free_item = emalloc(sizeof(php_sqlite3_free_list)); free_item->stmt_obj = stmt_obj; - free_item->stmt_obj_zval = getThis(); + //?? free_item->stmt_obj_zval = getThis(); + ZVAL_COPY_VALUE(&free_item->stmt_obj_zval, object); zend_llist_add_element(&(db_obj->free_list), &free_item); } @@ -1697,7 +1649,7 @@ PHP_METHOD(sqlite3result, numColumns) { php_sqlite3_result *result_obj; zval *object = getThis(); - result_obj = (php_sqlite3_result *)zend_object_store_get_object(object TSRMLS_CC); + result_obj = Z_SQLITE3_RESULT_P(object); SQLITE3_CHECK_INITIALIZED(result_obj->db_obj, result_obj->stmt_obj->initialised, SQLite3Result) @@ -1715,13 +1667,13 @@ PHP_METHOD(sqlite3result, columnName) { php_sqlite3_result *result_obj; zval *object = getThis(); - long column = 0; + zend_long column = 0; char *column_name; - result_obj = (php_sqlite3_result *)zend_object_store_get_object(object TSRMLS_CC); + result_obj = Z_SQLITE3_RESULT_P(object); SQLITE3_CHECK_INITIALIZED(result_obj->db_obj, result_obj->stmt_obj->initialised, SQLite3Result) - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &column) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "l", &column) == FAILURE) { return; } column_name = (char*) sqlite3_column_name(result_obj->stmt_obj->stmt, column); @@ -1729,8 +1681,8 @@ PHP_METHOD(sqlite3result, columnName) if (column_name == NULL) { RETURN_FALSE; } - - RETVAL_STRING(column_name, 1); + + RETVAL_STRING(column_name); } /* }}} */ @@ -1740,12 +1692,12 @@ PHP_METHOD(sqlite3result, columnType) { php_sqlite3_result *result_obj; zval *object = getThis(); - long column = 0; - result_obj = (php_sqlite3_result *)zend_object_store_get_object(object TSRMLS_CC); + zend_long column = 0; + result_obj = Z_SQLITE3_RESULT_P(object); SQLITE3_CHECK_INITIALIZED(result_obj->db_obj, result_obj->stmt_obj->initialised, SQLite3Result) - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &column) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "l", &column) == FAILURE) { return; } @@ -1764,12 +1716,12 @@ PHP_METHOD(sqlite3result, fetchArray) php_sqlite3_result *result_obj; zval *object = getThis(); int i, ret; - long mode = PHP_SQLITE3_BOTH; - result_obj = (php_sqlite3_result *)zend_object_store_get_object(object TSRMLS_CC); + zend_long mode = PHP_SQLITE3_BOTH; + result_obj = Z_SQLITE3_RESULT_P(object); SQLITE3_CHECK_INITIALIZED(result_obj->db_obj, result_obj->stmt_obj->initialised, SQLite3Result) - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "|l", &mode) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "|l", &mode) == FAILURE) { return; } @@ -1777,26 +1729,28 @@ PHP_METHOD(sqlite3result, fetchArray) switch (ret) { case SQLITE_ROW: /* If there was no return value then just skip fetching */ - if (!return_value_used) { + if (!USED_RET()) { return; } array_init(return_value); for (i = 0; i < sqlite3_data_count(result_obj->stmt_obj->stmt); i++) { - zval *data; + zval data; - data = sqlite_value_to_zval(result_obj->stmt_obj->stmt, i); + sqlite_value_to_zval(result_obj->stmt_obj->stmt, i, &data); if (mode & PHP_SQLITE3_NUM) { - add_index_zval(return_value, i, data); + add_index_zval(return_value, i, &data); } if (mode & PHP_SQLITE3_ASSOC) { if (mode & PHP_SQLITE3_NUM) { - Z_ADDREF_P(data); + if (Z_REFCOUNTED(data)) { + Z_ADDREF(data); + } } - add_assoc_zval(return_value, (char*)sqlite3_column_name(result_obj->stmt_obj->stmt, i), data); + add_assoc_zval(return_value, (char*)sqlite3_column_name(result_obj->stmt_obj->stmt, i), &data); } } break; @@ -1818,7 +1772,7 @@ PHP_METHOD(sqlite3result, reset) { php_sqlite3_result *result_obj; zval *object = getThis(); - result_obj = (php_sqlite3_result *)zend_object_store_get_object(object TSRMLS_CC); + result_obj = Z_SQLITE3_RESULT_P(object); SQLITE3_CHECK_INITIALIZED(result_obj->db_obj, result_obj->stmt_obj->initialised, SQLite3Result) @@ -1842,7 +1796,7 @@ PHP_METHOD(sqlite3result, finalize) { php_sqlite3_result *result_obj; zval *object = getThis(); - result_obj = (php_sqlite3_result *)zend_object_store_get_object(object TSRMLS_CC); + result_obj = Z_SQLITE3_RESULT_P(object); SQLITE3_CHECK_INITIALIZED(result_obj->db_obj, result_obj->stmt_obj->initialised, SQLite3Result) @@ -1852,7 +1806,7 @@ PHP_METHOD(sqlite3result, finalize) /* We need to finalize an internal statement */ if (result_obj->is_prepared_statement == 0) { - zend_llist_del_element(&(result_obj->db_obj->free_list), result_obj->stmt_obj_zval, + zend_llist_del_element(&(result_obj->db_obj->free_list), &result_obj->stmt_obj_zval, (int (*)(void *, void *)) php_sqlite3_compare_stmt_zval_free); } else { sqlite3_reset(result_obj->stmt_obj->stmt); @@ -1866,12 +1820,12 @@ PHP_METHOD(sqlite3result, finalize) __constructor for SQLite3Result. */ PHP_METHOD(sqlite3result, __construct) { - zend_throw_exception(zend_exception_get_default(TSRMLS_C), "SQLite3Result cannot be directly instantiated", 0 TSRMLS_CC); + zend_throw_exception(zend_ce_exception, "SQLite3Result cannot be directly instantiated", 0); } /* }}} */ /* {{{ arginfo */ -ZEND_BEGIN_ARG_INFO(arginfo_sqlite3_open, 0) +ZEND_BEGIN_ARG_INFO_EX(arginfo_sqlite3_open, 0, 0, 1) ZEND_ARG_INFO(0, filename) ZEND_ARG_INFO(0, flags) ZEND_ARG_INFO(0, encryption_key) @@ -2018,7 +1972,7 @@ static zend_function_entry php_sqlite3_result_class_methods[] = { }; /* }}} */ -/* {{{ Authorization Callback +/* {{{ Authorization Callback */ static int php_sqlite3_authorizer(void *autharg, int access_type, const char *arg3, const char *arg4, const char *arg5, const char *arg6) { @@ -2026,7 +1980,6 @@ static int php_sqlite3_authorizer(void *autharg, int access_type, const char *ar case SQLITE_ATTACH: { if (memcmp(arg3, ":memory:", sizeof(":memory:")) && *arg3) { - TSRMLS_FETCH(); #if PHP_API_VERSION < 20100412 if (PG(safe_mode) && (!php_checkuid(arg3, NULL, CHECKUID_CHECK_FILE_AND_DIR))) { @@ -2034,7 +1987,7 @@ static int php_sqlite3_authorizer(void *autharg, int access_type, const char *ar } #endif - if (php_check_open_basedir(arg3 TSRMLS_CC)) { + if (php_check_open_basedir(arg3)) { return SQLITE_DENY; } } @@ -2062,9 +2015,9 @@ static void php_sqlite3_free_list_dtor(void **item) } /* }}} */ -static int php_sqlite3_compare_stmt_zval_free( php_sqlite3_free_list **free_list, zval *statement ) /* {{{ */ +static int php_sqlite3_compare_stmt_zval_free(php_sqlite3_free_list **free_list, zval *statement ) /* {{{ */ { - return ((*free_list)->stmt_obj->initialised && statement == (*free_list)->stmt_obj_zval); + return ((*free_list)->stmt_obj->initialised && Z_PTR_P(statement) == Z_PTR((*free_list)->stmt_obj_zval)); } /* }}} */ @@ -2074,9 +2027,9 @@ static int php_sqlite3_compare_stmt_free( php_sqlite3_free_list **free_list, sql } /* }}} */ -static void php_sqlite3_object_free_storage(void *object TSRMLS_DC) /* {{{ */ +static void php_sqlite3_object_free_storage(zend_object *object) /* {{{ */ { - php_sqlite3_db_object *intern = (php_sqlite3_db_object *)object; + php_sqlite3_db_object *intern = php_sqlite3_db_from_obj(object); php_sqlite3_func *func; php_sqlite3_collation *collation; @@ -2093,13 +2046,13 @@ static void php_sqlite3_object_free_storage(void *object TSRMLS_DC) /* {{{ */ efree((char*)func->func_name); - if (func->func) { + if (!Z_ISUNDEF(func->func)) { zval_ptr_dtor(&func->func); } - if (func->step) { + if (!Z_ISUNDEF(func->step)) { zval_ptr_dtor(&func->step); } - if (func->fini) { + if (!Z_ISUNDEF(func->fini)) { zval_ptr_dtor(&func->fini); } efree(func); @@ -2112,7 +2065,7 @@ static void php_sqlite3_object_free_storage(void *object TSRMLS_DC) /* {{{ */ sqlite3_create_collation(intern->db, collation->collation_name, SQLITE_UTF8, NULL, NULL); } efree((char*)collation->collation_name); - if (collation->cmp_func){ + if (!Z_ISUNDEF(collation->cmp_func)) { zval_ptr_dtor(&collation->cmp_func); } efree(collation); @@ -2123,14 +2076,13 @@ static void php_sqlite3_object_free_storage(void *object TSRMLS_DC) /* {{{ */ intern->initialised = 0; } - zend_object_std_dtor(&intern->zo TSRMLS_CC); - efree(intern); + zend_object_std_dtor(&intern->zo); } /* }}} */ -static void php_sqlite3_stmt_object_free_storage(void *object TSRMLS_DC) /* {{{ */ +static void php_sqlite3_stmt_object_free_storage(zend_object *object) /* {{{ */ { - php_sqlite3_stmt *intern = (php_sqlite3_stmt *)object; + php_sqlite3_stmt *intern = php_sqlite3_stmt_from_obj(object); if (!intern) { return; @@ -2147,120 +2099,98 @@ static void php_sqlite3_stmt_object_free_storage(void *object TSRMLS_DC) /* {{{ (int (*)(void *, void *)) php_sqlite3_compare_stmt_free); } - if (intern->db_obj_zval) { + if (!Z_ISUNDEF(intern->db_obj_zval)) { zval_ptr_dtor(&intern->db_obj_zval); } - zend_object_std_dtor(&intern->zo TSRMLS_CC); - efree(intern); + zend_object_std_dtor(&intern->zo); } /* }}} */ -static void php_sqlite3_result_object_free_storage(void *object TSRMLS_DC) /* {{{ */ +static void php_sqlite3_result_object_free_storage(zend_object *object) /* {{{ */ { - php_sqlite3_result *intern = (php_sqlite3_result *)object; + php_sqlite3_result *intern = php_sqlite3_result_from_obj(object); if (!intern) { return; } - if (intern->stmt_obj_zval) { - if (intern->stmt_obj->initialised) { + if (!Z_ISNULL(intern->stmt_obj_zval)) { + if (intern->stmt_obj && intern->stmt_obj->initialised) { sqlite3_reset(intern->stmt_obj->stmt); } - if (intern->is_prepared_statement == 0) { - zval_dtor(intern->stmt_obj_zval); - FREE_ZVAL(intern->stmt_obj_zval); - } else { - zval_ptr_dtor(&intern->stmt_obj_zval); - } + zval_ptr_dtor(&intern->stmt_obj_zval); } - zend_object_std_dtor(&intern->zo TSRMLS_CC); - efree(intern); + zend_object_std_dtor(&intern->zo); } /* }}} */ -static zend_object_value php_sqlite3_object_new(zend_class_entry *class_type TSRMLS_DC) /* {{{ */ +static zend_object *php_sqlite3_object_new(zend_class_entry *class_type) /* {{{ */ { - zend_object_value retval; php_sqlite3_db_object *intern; /* Allocate memory for it */ - intern = emalloc(sizeof(php_sqlite3_db_object)); - memset(intern, 0, sizeof(php_sqlite3_db_object)); - intern->exception = 0; + intern = ecalloc(1, sizeof(php_sqlite3_db_object) + zend_object_properties_size(class_type)); /* Need to keep track of things to free */ - zend_llist_init(&(intern->free_list), sizeof(php_sqlite3_free_list *), (llist_dtor_func_t)php_sqlite3_free_list_dtor, 0); + zend_llist_init(&(intern->free_list), sizeof(php_sqlite3_free_list *), (llist_dtor_func_t)php_sqlite3_free_list_dtor, 0); - zend_object_std_init(&intern->zo, class_type TSRMLS_CC); + zend_object_std_init(&intern->zo, class_type); object_properties_init(&intern->zo, class_type); - retval.handle = zend_objects_store_put(intern, NULL, (zend_objects_free_object_storage_t) php_sqlite3_object_free_storage, NULL TSRMLS_CC); - retval.handlers = (zend_object_handlers *) &sqlite3_object_handlers; + intern->zo.handlers = &sqlite3_object_handlers; - return retval; + return &intern->zo; } /* }}} */ -static zend_object_value php_sqlite3_stmt_object_new(zend_class_entry *class_type TSRMLS_DC) /* {{{ */ +static zend_object *php_sqlite3_stmt_object_new(zend_class_entry *class_type) /* {{{ */ { - zend_object_value retval; php_sqlite3_stmt *intern; /* Allocate memory for it */ - intern = emalloc(sizeof(php_sqlite3_stmt)); - memset(intern, 0, sizeof(php_sqlite3_stmt)); + intern = ecalloc(1, sizeof(php_sqlite3_stmt) + zend_object_properties_size(class_type)); - intern->db_obj_zval = NULL; - - zend_object_std_init(&intern->zo, class_type TSRMLS_CC); + zend_object_std_init(&intern->zo, class_type); object_properties_init(&intern->zo, class_type); - retval.handle = zend_objects_store_put(intern, NULL, (zend_objects_free_object_storage_t) php_sqlite3_stmt_object_free_storage, NULL TSRMLS_CC); - retval.handlers = (zend_object_handlers *) &sqlite3_stmt_object_handlers; + intern->zo.handlers = &sqlite3_stmt_object_handlers; - return retval; + return &intern->zo; } /* }}} */ -static zend_object_value php_sqlite3_result_object_new(zend_class_entry *class_type TSRMLS_DC) /* {{{ */ +static zend_object *php_sqlite3_result_object_new(zend_class_entry *class_type) /* {{{ */ { - zend_object_value retval; php_sqlite3_result *intern; /* Allocate memory for it */ - intern = emalloc(sizeof(php_sqlite3_result)); - memset(intern, 0, sizeof(php_sqlite3_result)); - - intern->complete = 0; - intern->is_prepared_statement = 0; - intern->stmt_obj_zval = NULL; + intern = ecalloc(1, sizeof(php_sqlite3_result) + zend_object_properties_size(class_type)); - zend_object_std_init(&intern->zo, class_type TSRMLS_CC); + zend_object_std_init(&intern->zo, class_type); object_properties_init(&intern->zo, class_type); - retval.handle = zend_objects_store_put(intern, NULL, (zend_objects_free_object_storage_t) php_sqlite3_result_object_free_storage, NULL TSRMLS_CC); - retval.handlers = (zend_object_handlers *) &sqlite3_result_object_handlers; + intern->zo.handlers = &sqlite3_result_object_handlers; - return retval; + return &intern->zo; } /* }}} */ -static void sqlite3_param_dtor(void *data) /* {{{ */ +static void sqlite3_param_dtor(zval *data) /* {{{ */ { - struct php_sqlite3_bound_param *param = (struct php_sqlite3_bound_param*)data; + struct php_sqlite3_bound_param *param = (struct php_sqlite3_bound_param*)Z_PTR_P(data); if (param->name) { - efree(param->name); + zend_string_release(param->name); } - if (param->parameter) { + if (!Z_ISNULL(param->parameter)) { zval_ptr_dtor(&(param->parameter)); - param->parameter = NULL; + ZVAL_UNDEF(¶m->parameter); } + efree(param); } /* }}} */ @@ -2273,7 +2203,7 @@ PHP_MINIT_FUNCTION(sqlite3) #if defined(ZTS) /* Refuse to load if this wasn't a threasafe library loaded */ if (!sqlite3_threadsafe()) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "A thread safe version of SQLite is required when using a thread safe version of PHP."); + php_error_docref(NULL, E_WARNING, "A thread safe version of SQLite is required when using a thread safe version of PHP."); return FAILURE; } #endif @@ -2285,20 +2215,26 @@ PHP_MINIT_FUNCTION(sqlite3) /* Register SQLite 3 Class */ INIT_CLASS_ENTRY(ce, "SQLite3", php_sqlite3_class_methods); ce.create_object = php_sqlite3_object_new; + sqlite3_object_handlers.offset = XtOffsetOf(php_sqlite3_db_object, zo); sqlite3_object_handlers.clone_obj = NULL; - php_sqlite3_sc_entry = zend_register_internal_class(&ce TSRMLS_CC); + sqlite3_object_handlers.free_obj = php_sqlite3_object_free_storage; + php_sqlite3_sc_entry = zend_register_internal_class(&ce); /* Register SQLite 3 Prepared Statement Class */ INIT_CLASS_ENTRY(ce, "SQLite3Stmt", php_sqlite3_stmt_class_methods); ce.create_object = php_sqlite3_stmt_object_new; + sqlite3_stmt_object_handlers.offset = XtOffsetOf(php_sqlite3_stmt, zo); sqlite3_stmt_object_handlers.clone_obj = NULL; - php_sqlite3_stmt_entry = zend_register_internal_class(&ce TSRMLS_CC); + sqlite3_stmt_object_handlers.free_obj = php_sqlite3_stmt_object_free_storage; + php_sqlite3_stmt_entry = zend_register_internal_class(&ce); /* Register SQLite 3 Result Class */ INIT_CLASS_ENTRY(ce, "SQLite3Result", php_sqlite3_result_class_methods); ce.create_object = php_sqlite3_result_object_new; + sqlite3_result_object_handlers.offset = XtOffsetOf(php_sqlite3_result, zo); sqlite3_result_object_handlers.clone_obj = NULL; - php_sqlite3_result_entry = zend_register_internal_class(&ce TSRMLS_CC); + sqlite3_result_object_handlers.free_obj = php_sqlite3_result_object_free_storage; + php_sqlite3_result_entry = zend_register_internal_class(&ce); REGISTER_INI_ENTRIES(); @@ -2348,6 +2284,9 @@ PHP_MINFO_FUNCTION(sqlite3) */ static PHP_GINIT_FUNCTION(sqlite3) { +#if defined(COMPILE_DL_SQLITE3) && defined(ZTS) + ZEND_TSRMLS_CACHE_UPDATE(); +#endif memset(sqlite3_globals, 0, sizeof(*sqlite3_globals)); } /* }}} */ @@ -2373,6 +2312,9 @@ zend_module_entry sqlite3_module_entry = { /* }}} */ #ifdef COMPILE_DL_SQLITE3 +#ifdef ZTS +ZEND_TSRMLS_CACHE_DEFINE(); +#endif ZEND_GET_MODULE(sqlite3) #endif |