From f6280c0f47b07d5c6079f7074491b6173c89599f Mon Sep 17 00:00:00 2001 From: Michael Wallner Date: Fri, 22 Jul 2005 09:47:43 +0000 Subject: [PATCH] - proper handling of static members --- http.c | 10 --------- php_http_std_defs.h | 51 ++++++++++++++++++++++++++++++++++++++++++--- 2 files changed, 48 insertions(+), 13 deletions(-) diff --git a/http.c b/http.c index 192834b..3b05887 100644 --- a/http.c +++ b/http.c @@ -240,19 +240,9 @@ PHP_RINIT_FUNCTION(http) } /* }}} */ -static int cleanup_statics(void *p TSRMLS_DC) -{ - zval_dtor(*(zval **)p); - return 0; -} - /* {{{ PHP_RSHUTDOWN_FUNCTION */ PHP_RSHUTDOWN_FUNCTION(http) { - /* ZE kludge */ -#ifdef ZEND_ENGINE_2 - zend_hash_apply(http_response_object_ce->static_members, cleanup_statics TSRMLS_CC); -#endif http_globals_free(HTTP_GLOBALS); return SUCCESS; } diff --git a/php_http_std_defs.h b/php_http_std_defs.h index 9b17bcc..6e1c7ab 100644 --- a/php_http_std_defs.h +++ b/php_http_std_defs.h @@ -193,7 +193,7 @@ typedef int STATUS; # define getObject(t, o) getObjectEx(t, o, getThis()) # define getObjectEx(t, o, v) t * o = ((t *) zend_object_store_get_object(v TSRMLS_CC)) -# define OBJ_PROP(o) o->zo.properties +# define OBJ_PROP(o) (o)->zo.properties # define DCL_STATIC_PROP(a, t, n, v) zend_declare_property_ ##t(ce, (#n), sizeof(#n), (v), (ZEND_ACC_ ##a | ZEND_ACC_STATIC) TSRMLS_CC) # define DCL_STATIC_PROP_Z(a, n, v) zend_declare_property(ce, (#n), sizeof(#n), (v), (ZEND_ACC_ ##a | ZEND_ACC_STATIC) TSRMLS_CC) # define DCL_STATIC_PROP_N(a, n) zend_declare_property_null(ce, (#n), sizeof(#n), (ZEND_ACC_ ##a | ZEND_ACC_STATIC) TSRMLS_CC) @@ -207,9 +207,54 @@ typedef int STATUS; \ refcount = (*__static)->refcount; \ is_ref = (*__static)->is_ref; \ - zval_dtor(*__static); \ + switch (Z_TYPE_PP(__static)) \ + { \ + case IS_BOOL: case IS_LONG: case IS_NULL: \ + break; \ + case IS_RESOURCE: \ + zend_list_delete(Z_LVAL_PP(__static)); \ + break; \ + case IS_STRING: case IS_CONSTANT: \ + free(Z_STRVAL_PP(__static)); \ + break; \ + case IS_OBJECT: \ + Z_OBJ_HT_PP(__static)->del_ref(*__static TSRMLS_CC); \ + break; \ + case IS_ARRAY: case IS_CONSTANT_ARRAY: \ + if (Z_ARRVAL_PP(__static) && Z_ARRVAL_PP(__static) != &EG(symbol_table)) { \ + zend_hash_destroy(Z_ARRVAL_PP(__static)); \ + free(Z_ARRVAL_PP(__static)); \ + } \ + break; \ + } \ **__static = *(v); \ - zval_copy_ctor(*__static); \ + switch (Z_TYPE_PP(__static)) \ + { \ + case IS_BOOL: case IS_LONG: case IS_NULL: \ + break; \ + case IS_RESOURCE: \ + zend_list_addref(Z_LVAL_PP(__static)); \ + break; \ + case IS_STRING: case IS_CONSTANT: \ + Z_STRVAL_PP(__static) = (char *) zend_strndup(Z_STRVAL_PP(__static), Z_STRLEN_PP(__static)); \ + break; \ + case IS_OBJECT: \ + { \ + Z_OBJ_HT_PP(__static)->add_ref(*__static TSRMLS_CC); \ + } \ + break; \ + case IS_ARRAY: case IS_CONSTANT_ARRAY: \ + { \ + if (Z_ARRVAL_PP(__static) != &EG(symbol_table)) { \ + zval *tmp; \ + HashTable *old = Z_ARRVAL_PP(__static); \ + Z_ARRVAL_PP(__static) = (HashTable *) malloc(sizeof(HashTable)); \ + zend_hash_init(Z_ARRVAL_PP(__static), 0, NULL, ZVAL_PTR_DTOR, 0); \ + zend_hash_copy(Z_ARRVAL_PP(__static), old, (copy_ctor_func_t) zval_add_ref, (void *) &tmp, sizeof(zval *)); \ + } \ + } \ + break; \ + } \ (*__static)->refcount = refcount; \ (*__static)->is_ref = is_ref; \ } -- 2.30.2