]> granicus.if.org Git - php/commitdiff
Refactoring ext/snmp
authorXinchen Hui <laruence@php.net>
Tue, 20 May 2014 09:02:29 +0000 (17:02 +0800)
committerXinchen Hui <laruence@php.net>
Tue, 20 May 2014 09:02:29 +0000 (17:02 +0800)
ext/snmp/php_snmp.h
ext/snmp/snmp.c

index 09ab8b9e8101642623d382e7accf3834efaf0b39..50bd272204293f899efc3c77dabb6f445a11eb4b 100644 (file)
@@ -85,7 +85,6 @@ PHP_METHOD(SNMP, getErrno);
 PHP_METHOD(SNMP, getError);
 
 typedef struct _php_snmp_object {
-       zend_object zo;
        struct snmp_session *session;
        int max_oids;
        int valueretrieval;
@@ -96,10 +95,16 @@ typedef struct _php_snmp_object {
        int oid_increasing_check;
        int exceptions_enabled;
        char snmp_errstr[256];
+       zend_object zo;
 } php_snmp_object;
 
+static inline php_snmp_object *php_snmp_fetch_object(zend_object *obj) {
+       return (php_snmp_object *)((char*)(obj) - XtOffsetOf(php_snmp_object, zo));
+}
+
+#define Z_SNMP_P(zv) php_snmp_fetch_object(Z_OBJ_P((zv)))
 
-typedef int (*php_snmp_read_t)(php_snmp_object *snmp_object, zval **retval TSRMLS_DC);
+typedef int (*php_snmp_read_t)(php_snmp_object *snmp_object, zval *retval TSRMLS_DC);
 typedef int (*php_snmp_write_t)(php_snmp_object *snmp_object, zval *newval TSRMLS_DC);
 
 typedef struct _ptp_snmp_prop_handler {
index 8b6afdee3a0155971c1af2e592dd4834212b6121..3fa75830318fd84493af98ca160c7d8cccb0384b 100644 (file)
@@ -454,7 +454,7 @@ static PHP_GINIT_FUNCTION(snmp)
        } \
 }
 
-static void netsnmp_session_free(php_snmp_session **session)
+static void netsnmp_session_free(php_snmp_session **session) /* {{{ */
 {
        if (*session) {
                PHP_SNMP_SESSION_FREE(peername);
@@ -465,16 +465,18 @@ static void netsnmp_session_free(php_snmp_session **session)
                *session = NULL;
        }
 }
+/* }}} */
 
-static void php_snmp_session_destructor(zend_rsrc_list_entry *rsrc TSRMLS_DC)
+static void php_snmp_session_destructor(zend_resource *rsrc TSRMLS_DC) /* {{{ */
 {
        php_snmp_session *session = (php_snmp_session *)rsrc->ptr;
        netsnmp_session_free(&session);
 }
+/* }}} */
 
-static void php_snmp_object_free_storage(void *object TSRMLS_DC)
+static void php_snmp_object_free_storage(zend_object *object TSRMLS_DC) /* {{{ */
 {
-       php_snmp_object *intern = (php_snmp_object *)object;
+       php_snmp_object *intern = php_snmp_fetch_object(object);
        
        if (!intern) {
                return;
@@ -483,28 +485,25 @@ static void php_snmp_object_free_storage(void *object TSRMLS_DC)
        netsnmp_session_free(&(intern->session));
 
        zend_object_std_dtor(&intern->zo TSRMLS_CC);
-       
-       efree(intern);
 }
+/* }}} */
 
-static zend_object_value php_snmp_object_new(zend_class_entry *class_type TSRMLS_DC) /* {{{ */
+static zend_object *php_snmp_object_new(zend_class_entry *class_type TSRMLS_DC) /* {{{ */
 {
-       zend_object_value retval;
        php_snmp_object *intern;
 
        /* Allocate memory for it */
-       intern = emalloc(sizeof(php_snmp_object));
-       memset(&intern->zo, 0, sizeof(php_snmp_object));
+       intern = ecalloc(1, sizeof(php_snmp_object) + sizeof(zval) *  (class_type->default_properties_count - 1));
 
        zend_object_std_init(&intern->zo, class_type TSRMLS_CC);
        object_properties_init(&intern->zo, class_type);
 
-       retval.handle = zend_objects_store_put(intern, (zend_objects_store_dtor_t)zend_objects_destroy_object, (zend_objects_free_object_storage_t) php_snmp_object_free_storage, NULL TSRMLS_CC);
-       retval.handlers = (zend_object_handlers *) &php_snmp_object_handlers;
+       intern->zo.handlers = &php_snmp_object_handlers;
 
-       return retval;
+       return &intern->zo;
        
 }
+/* }}} */
 
 /* {{{ php_snmp_error
  *
@@ -517,7 +516,7 @@ static void php_snmp_error(zval *object, const char *docref TSRMLS_DC, int type,
        php_snmp_object *snmp_object = NULL;
 
        if (object) {
-               snmp_object = (php_snmp_object *)zend_object_store_get_object(object TSRMLS_CC);
+               snmp_object = Z_SNMP_P(object);
                if (type == PHP_SNMP_ERRNO_NOERROR) {
                        memset(snmp_object->snmp_errstr, 0, sizeof(snmp_object->snmp_errstr));
                } else {
@@ -550,7 +549,7 @@ static void php_snmp_error(zval *object, const char *docref TSRMLS_DC, int type,
 */
 static void php_snmp_getvalue(struct variable_list *vars, zval *snmpval TSRMLS_DC, int valueretrieval)
 {
-       zval *val;
+       zval val;
        char sbuf[512];
        char *buf = &(sbuf[0]);
        char *dbuf = (char *)NULL;
@@ -601,27 +600,25 @@ static void php_snmp_getvalue(struct variable_list *vars, zval *snmpval TSRMLS_D
                }
        }
 
-       MAKE_STD_ZVAL(val);
-
        if (valueretrieval & SNMP_VALUE_PLAIN) {
                *buf = 0;
                switch (vars->type) {
                case ASN_BIT_STR:               /* 0x03, asn1.h */
-                       ZVAL_STRINGL(val, (char *)vars->val.bitstring, vars->val_len, 1);
+                       ZVAL_STRINGL(&val, (char *)vars->val.bitstring, vars->val_len);
                        break;
 
                case ASN_OCTET_STR:             /* 0x04, asn1.h */
                case ASN_OPAQUE:                /* 0x44, snmp_impl.h */
-                       ZVAL_STRINGL(val, (char *)vars->val.string, vars->val_len, 1);
+                       ZVAL_STRINGL(&val, (char *)vars->val.string, vars->val_len);
                        break;
 
                case ASN_NULL:                  /* 0x05, asn1.h */
-                       ZVAL_NULL(val);
+                       ZVAL_NULL(&val);
                        break;
 
                case ASN_OBJECT_ID:             /* 0x06, asn1.h */
                        snprint_objid(buf, buflen, vars->val.objid, vars->val_len / sizeof(oid));
-                       ZVAL_STRING(val, buf, 1);
+                       ZVAL_STRING(&val, buf);
                        break;
 
                case ASN_IPADDRESS:             /* 0x40, snmp_impl.h */
@@ -629,7 +626,7 @@ static void php_snmp_getvalue(struct variable_list *vars, zval *snmpval TSRMLS_D
                                 (vars->val.string)[0], (vars->val.string)[1],
                                 (vars->val.string)[2], (vars->val.string)[3]);
                        buf[buflen]=0;
-                       ZVAL_STRING(val, buf, 1);
+                       ZVAL_STRING(&val, buf);
                        break;
 
                case ASN_COUNTER:               /* 0x41, snmp_impl.h */
@@ -639,59 +636,58 @@ static void php_snmp_getvalue(struct variable_list *vars, zval *snmpval TSRMLS_D
                case ASN_UINTEGER:              /* 0x47, snmp_impl.h */
                        snprintf(buf, buflen, "%lu", *vars->val.integer);
                        buf[buflen]=0;
-                       ZVAL_STRING(val, buf, 1);
+                       ZVAL_STRING(&val, buf);
                        break;
 
                case ASN_INTEGER:               /* 0x02, asn1.h */
                        snprintf(buf, buflen, "%ld", *vars->val.integer);
                        buf[buflen]=0;
-                       ZVAL_STRING(val, buf, 1);
+                       ZVAL_STRING(&val, buf);
                        break;
 
 #if defined(NETSNMP_WITH_OPAQUE_SPECIAL_TYPES) || defined(OPAQUE_SPECIAL_TYPES)
                case ASN_OPAQUE_FLOAT:          /* 0x78, asn1.h */
                        snprintf(buf, buflen, "%f", *vars->val.floatVal);
-                       ZVAL_STRING(val, buf, 1);
+                       ZVAL_STRING(&val, buf);
                        break;
 
                case ASN_OPAQUE_DOUBLE:         /* 0x79, asn1.h */
                        snprintf(buf, buflen, "%Lf", *vars->val.doubleVal);
-                       ZVAL_STRING(val, buf, 1);
+                       ZVAL_STRING(&val, buf);
                        break;
 
                case ASN_OPAQUE_I64:            /* 0x80, asn1.h */
                        printI64(buf, vars->val.counter64);
-                       ZVAL_STRING(val, buf, 1);
+                       ZVAL_STRING(&val, buf);
                        break;
 
                case ASN_OPAQUE_U64:            /* 0x81, asn1.h */
 #endif
                case ASN_COUNTER64:             /* 0x46, snmp_impl.h */
                        printU64(buf, vars->val.counter64);
-                       ZVAL_STRING(val, buf, 1);
+                       ZVAL_STRING(&val, buf);
                        break;
 
                default:
-                       ZVAL_STRING(val, "Unknown value type", 1);
+                       ZVAL_STRING(&val, "Unknown value type");
                        php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unknown value type: %u", vars->type);
                        break;
                }
        } else /* use Net-SNMP value translation */ {
                /* we have desired string in buffer, just use it */
-               ZVAL_STRING(val, buf, 1);
+               ZVAL_STRING(&val, buf);
        }
 
        if (valueretrieval & SNMP_VALUE_OBJECT) {
                object_init(snmpval);
                add_property_long(snmpval, "type", vars->type);
-               add_property_zval(snmpval, "value", val);
+               add_property_zval(snmpval, "value", &val);
        } else  {
-               *snmpval = *val;
-               zval_copy_ctor(snmpval);
+               ZVAL_COPY(snmpval, &val);
        }
        zval_ptr_dtor(&val);
 
-       if(dbuf){ /* malloc was used to store value */
+       if (dbuf){ /* malloc was used to store value */
                efree(dbuf);
        }
 }
@@ -716,7 +712,7 @@ static void php_snmp_internal(INTERNAL_FUNCTION_PARAMETERS, int st,
        char buf2[2048];
        int keepwalking=1;
        char *err;
-       zval *snmpval = NULL;
+       zval snmpval;
        int snmp_errno;
 
        /* we start with retval=FALSE. If any actual data is acquired, retval will be set to appropriate type */
@@ -831,15 +827,15 @@ retry:
                                                break;
                                        }
 
-                                       MAKE_STD_ZVAL(snmpval);
-                                       php_snmp_getvalue(vars, snmpval TSRMLS_CC, objid_query->valueretrieval);
+                                       ZVAL_NULL(&snmpval);
+                                       php_snmp_getvalue(vars, &snmpval TSRMLS_CC, objid_query->valueretrieval);
 
                                        if (objid_query->array_output) {
-                                               if (Z_TYPE_P(return_value) == IS_BOOL) {
+                                               if (Z_TYPE_P(return_value) == IS_TRUE || Z_TYPE_P(return_value) == IS_FALSE) {
                                                        array_init(return_value);
                                                }
                                                if (st & SNMP_NUMERIC_KEYS) {
-                                                       add_next_index_zval(return_value, snmpval);
+                                                       add_next_index_zval(return_value, &snmpval);
                                                } else if (st & SNMP_ORIGINAL_NAMES_AS_KEYS && st & SNMP_CMD_GET) {
                                                        found = 0;
                                                        for (count = 0; count < objid_query->count; count++) {
@@ -850,7 +846,7 @@ retry:
                                                                }
                                                        }
                                                        if (found) {
-                                                               add_assoc_zval(return_value, objid_query->vars[count].oid, snmpval);
+                                                               add_assoc_zval(return_value, objid_query->vars[count].oid, &snmpval);
                                                        } else {
                                                                snprint_objid(buf2, sizeof(buf2), vars->name, vars->name_length);
                                                                php_error_docref(NULL TSRMLS_CC, E_WARNING, "Could not find original OID name for '%s'", buf2);
@@ -867,15 +863,13 @@ retry:
                                                                }
                                                                buf2[strlen(buf2) - 1] = '\0'; /* remove trailing '.' */
                                                        }
-                                                       add_assoc_zval(return_value, buf2, snmpval);
+                                                       add_assoc_zval(return_value, buf2, &snmpval);
                                                } else {
                                                        snprint_objid(buf2, sizeof(buf2), vars->name, vars->name_length);
-                                                       add_assoc_zval(return_value, buf2, snmpval);
+                                                       add_assoc_zval(return_value, buf2, &snmpval);
                                                }
                                        } else {
-                                               *return_value = *snmpval;
-                                               zval_copy_ctor(return_value);
-                                               zval_ptr_dtor(&snmpval);
+                                               ZVAL_COPY_VALUE(return_value, &snmpval);
                                                break;
                                        }
 
@@ -902,8 +896,8 @@ retry:
                                        keepwalking = 1;
                                        continue;
                                }
-                               if (!(st & SNMP_CMD_WALK) || response->errstat != SNMP_ERR_NOSUCHNAME || Z_TYPE_P(return_value) == IS_BOOL) {
-                                       for (   count=1, vars = response->variables;
+                               if (!(st & SNMP_CMD_WALK) || response->errstat != SNMP_ERR_NOSUCHNAME || Z_TYPE_P(return_value) == IS_TRUE || Z_TYPE_P(return_value) == IS_FALSE) {
+                                       for (count=1, vars = response->variables;
                                                vars && count != response->errindex;
                                                vars = vars->next_variable, count++);
 
@@ -929,7 +923,7 @@ retry:
                                        snmp_free_pdu(response);
                                        snmp_close(ss);
                                        if (objid_query->array_output) {
-                                               zval_dtor(return_value);
+                                               zval_ptr_dtor(return_value);
                                        }
                                        RETVAL_FALSE;
                                        return;
@@ -938,7 +932,7 @@ retry:
                } else if (status == STAT_TIMEOUT) {
                        php_snmp_error(getThis(), NULL TSRMLS_CC, PHP_SNMP_ERRNO_TIMEOUT, "No response from %s", session->peername);
                        if (objid_query->array_output) {
-                               zval_dtor(return_value);
+                               zval_ptr_dtor(return_value);
                        }
                        snmp_close(ss);
                        RETVAL_FALSE;
@@ -948,7 +942,7 @@ retry:
                        php_snmp_error(getThis(), NULL TSRMLS_CC, PHP_SNMP_ERRNO_GENERIC, "Fatal error: %s", err);
                        free(err);
                        if (objid_query->array_output) {
-                               zval_dtor(return_value);
+                               zval_ptr_dtor(return_value);
                        }
                        snmp_close(ss);
                        RETVAL_FALSE;
@@ -967,61 +961,67 @@ retry:
 * OID parser (and type, value for SNMP_SET command)
 */
 
-static int php_snmp_parse_oid(zval *object, int st, struct objid_query *objid_query, zval **oid, zval **type, zval **value TSRMLS_DC)
+static int php_snmp_parse_oid(zval *object, int st, struct objid_query *objid_query, zval *oid, zval *type, zval *value TSRMLS_DC)
 {
        char *pptr;
        HashPosition pos_oid, pos_type, pos_value;
-       zval **tmp_oid, **tmp_type, **tmp_value;
+       zval *tmp_oid, *tmp_type, *tmp_value;
 
-       if (Z_TYPE_PP(oid) != IS_ARRAY) {
+       if (Z_TYPE_P(oid) != IS_ARRAY) {
+               /*
                if (Z_ISREF_PP(oid)) {
                        SEPARATE_ZVAL(oid);
                }
+               */
                convert_to_string_ex(oid);
-       } else if (Z_TYPE_PP(oid) == IS_ARRAY) {
-               zend_hash_internal_pointer_reset_ex(Z_ARRVAL_PP(oid), &pos_oid);
+       } else if (Z_TYPE_P(oid) == IS_ARRAY) {
+               zend_hash_internal_pointer_reset_ex(Z_ARRVAL_P(oid), &pos_oid);
        }
 
        if (st & SNMP_CMD_SET) {
-               if (Z_TYPE_PP(type) != IS_ARRAY) {
+               if (Z_TYPE_P(type) != IS_ARRAY) {
+                       /*
                        if (Z_ISREF_PP(type)) {
                                SEPARATE_ZVAL(type);
                        }
+                       */
                        convert_to_string_ex(type);
-               } else if (Z_TYPE_PP(type) == IS_ARRAY) {
-                       zend_hash_internal_pointer_reset_ex(Z_ARRVAL_PP(type), &pos_type);
+               } else if (Z_TYPE_P(type) == IS_ARRAY) {
+                       zend_hash_internal_pointer_reset_ex(Z_ARRVAL_P(type), &pos_type);
                }
 
-               if (Z_TYPE_PP(value) != IS_ARRAY) {
+               if (Z_TYPE_P(value) != IS_ARRAY) {
+                       /*
                        if (Z_ISREF_PP(value)) {
                                SEPARATE_ZVAL(value);
                        }
+                       */
                        convert_to_string_ex(value);
-               } else if (Z_TYPE_PP(value) == IS_ARRAY) {
-                       zend_hash_internal_pointer_reset_ex(Z_ARRVAL_PP(value), &pos_value);
+               } else if (Z_TYPE_P(value) == IS_ARRAY) {
+                       zend_hash_internal_pointer_reset_ex(Z_ARRVAL_P(value), &pos_value);
                }
        }
 
        objid_query->count = 0;
        objid_query->array_output = ((st & SNMP_CMD_WALK) ? TRUE : FALSE);
-       if (Z_TYPE_PP(oid) == IS_STRING) {
+       if (Z_TYPE_P(oid) == IS_STRING) {
                objid_query->vars = (snmpobjarg *)emalloc(sizeof(snmpobjarg));
                if (objid_query->vars == NULL) {
                        php_error_docref(NULL TSRMLS_CC, E_WARNING, "emalloc() failed while parsing oid: %s", strerror(errno));
                        efree(objid_query->vars);
                        return FALSE;
                }
-               objid_query->vars[objid_query->count].oid = Z_STRVAL_PP(oid);
+               objid_query->vars[objid_query->count].oid = Z_STRVAL_P(oid);
                if (st & SNMP_CMD_SET) {
-                       if (Z_TYPE_PP(type) == IS_STRING && Z_TYPE_PP(value) == IS_STRING) {
-                               if (Z_STRLEN_PP(type) != 1) {
-                                       php_error_docref(NULL TSRMLS_CC, E_WARNING, "Bogus type '%s', should be single char, got %u", Z_STRVAL_PP(type), Z_STRLEN_PP(type));
+                       if (Z_TYPE_P(type) == IS_STRING && Z_TYPE_P(value) == IS_STRING) {
+                               if (Z_STRLEN_P(type) != 1) {
+                                       php_error_docref(NULL TSRMLS_CC, E_WARNING, "Bogus type '%s', should be single char, got %u", Z_STRVAL_P(type), Z_STRLEN_P(type));
                                        efree(objid_query->vars);
                                        return FALSE;
                                }
-                               pptr = Z_STRVAL_PP(type);
+                               pptr = Z_STRVAL_P(type);
                                objid_query->vars[objid_query->count].type = *pptr;
-                               objid_query->vars[objid_query->count].value = Z_STRVAL_PP(value);
+                               objid_query->vars[objid_query->count].value = Z_STRVAL_P(value);
                        } else {
                                php_error_docref(NULL TSRMLS_CC, E_WARNING, "Single objid and multiple type or values are not supported");
                                efree(objid_query->vars);
@@ -1029,55 +1029,55 @@ static int php_snmp_parse_oid(zval *object, int st, struct objid_query *objid_qu
                        }
                }
                objid_query->count++;
-       } else if (Z_TYPE_PP(oid) == IS_ARRAY) { /* we got objid array */
-               if (zend_hash_num_elements(Z_ARRVAL_PP(oid)) == 0) {
+       } else if (Z_TYPE_P(oid) == IS_ARRAY) { /* we got objid array */
+               if (zend_hash_num_elements(Z_ARRVAL_P(oid)) == 0) {
                        php_error_docref(NULL TSRMLS_CC, E_WARNING, "Got empty OID array");
                        return FALSE;
                }
-               objid_query->vars = (snmpobjarg *)emalloc(sizeof(snmpobjarg) * zend_hash_num_elements(Z_ARRVAL_PP(oid)));
+               objid_query->vars = (snmpobjarg *)emalloc(sizeof(snmpobjarg) * zend_hash_num_elements(Z_ARRVAL_P(oid)));
                if (objid_query->vars == NULL) {
                        php_error_docref(NULL TSRMLS_CC, E_WARNING, "emalloc() failed while parsing oid array: %s", strerror(errno));
                        efree(objid_query->vars);
                        return FALSE;
                }
                objid_query->array_output = ( (st & SNMP_CMD_SET) ? FALSE : TRUE );
-               for (   zend_hash_internal_pointer_reset_ex(Z_ARRVAL_PP(oid), &pos_oid);
-                       zend_hash_get_current_data_ex(Z_ARRVAL_PP(oid), (void **) &tmp_oid, &pos_oid) == SUCCESS;
-                       zend_hash_move_forward_ex(Z_ARRVAL_PP(oid), &pos_oid) ) {
+               for (zend_hash_internal_pointer_reset_ex(Z_ARRVAL_P(oid), &pos_oid);
+                       (tmp_oid = zend_hash_get_current_data_ex(Z_ARRVAL_P(oid), &pos_oid)) != NULL;
+                       zend_hash_move_forward_ex(Z_ARRVAL_P(oid), &pos_oid) ) {
 
                        convert_to_string_ex(tmp_oid);
-                       objid_query->vars[objid_query->count].oid = Z_STRVAL_PP(tmp_oid);
+                       objid_query->vars[objid_query->count].oid = Z_STRVAL_P(tmp_oid);
                        if (st & SNMP_CMD_SET) {
-                               if (Z_TYPE_PP(type) == IS_STRING) {
-                                       pptr = Z_STRVAL_PP(type);
+                               if (Z_TYPE_P(type) == IS_STRING) {
+                                       pptr = Z_STRVAL_P(type);
                                        objid_query->vars[objid_query->count].type = *pptr;
-                               } else if (Z_TYPE_PP(type) == IS_ARRAY) {
-                                       if (SUCCESS == zend_hash_get_current_data_ex(Z_ARRVAL_PP(type), (void **) &tmp_type, &pos_type)) {
+                               } else if (Z_TYPE_P(type) == IS_ARRAY) {
+                                       if ((tmp_type = zend_hash_get_current_data_ex(Z_ARRVAL_P(type), &pos_type)) != NULL) {
                                                convert_to_string_ex(tmp_type);
-                                               if (Z_STRLEN_PP(tmp_type) != 1) {
-                                                       php_error_docref(NULL TSRMLS_CC, E_WARNING, "'%s': bogus type '%s', should be single char, got %u", Z_STRVAL_PP(tmp_oid), Z_STRVAL_PP(tmp_type), Z_STRLEN_PP(tmp_type));
+                                               if (Z_STRLEN_P(tmp_type) != 1) {
+                                                       php_error_docref(NULL TSRMLS_CC, E_WARNING, "'%s': bogus type '%s', should be single char, got %u", Z_STRVAL_P(tmp_oid), Z_STRVAL_P(tmp_type), Z_STRLEN_P(tmp_type));
                                                        efree(objid_query->vars);
                                                        return FALSE;
                                                }
-                                               pptr = Z_STRVAL_PP(tmp_type);
+                                               pptr = Z_STRVAL_P(tmp_type);
                                                objid_query->vars[objid_query->count].type = *pptr;
-                                               zend_hash_move_forward_ex(Z_ARRVAL_PP(type), &pos_type);
+                                               zend_hash_move_forward_ex(Z_ARRVAL_P(type), &pos_type);
                                        } else {
-                                               php_error_docref(NULL TSRMLS_CC, E_WARNING, "'%s': no type set", Z_STRVAL_PP(tmp_oid));
+                                               php_error_docref(NULL TSRMLS_CC, E_WARNING, "'%s': no type set", Z_STRVAL_P(tmp_oid));
                                                efree(objid_query->vars);
                                                return FALSE;
                                        }
                                }
 
-                               if (Z_TYPE_PP(value) == IS_STRING) {
-                                       objid_query->vars[objid_query->count].value = Z_STRVAL_PP(value);
-                               } else if (Z_TYPE_PP(value) == IS_ARRAY) {
-                                       if (SUCCESS == zend_hash_get_current_data_ex(Z_ARRVAL_PP(value), (void **) &tmp_value, &pos_value)) {
+                               if (Z_TYPE_P(value) == IS_STRING) {
+                                       objid_query->vars[objid_query->count].value = Z_STRVAL_P(value);
+                               } else if (Z_TYPE_P(value) == IS_ARRAY) {
+                                       if ((tmp_value = zend_hash_get_current_data_ex(Z_ARRVAL_P(value), &pos_value)) != NULL) {
                                                convert_to_string_ex(tmp_value);
-                                               objid_query->vars[objid_query->count].value = Z_STRVAL_PP(tmp_value);
-                                               zend_hash_move_forward_ex(Z_ARRVAL_PP(value), &pos_value);
+                                               objid_query->vars[objid_query->count].value = Z_STRVAL_P(tmp_value);
+                                               zend_hash_move_forward_ex(Z_ARRVAL_P(value), &pos_value);
                                        } else {
-                                               php_error_docref(NULL TSRMLS_CC, E_WARNING, "'%s': no value set", Z_STRVAL_PP(tmp_oid));
+                                               php_error_docref(NULL TSRMLS_CC, E_WARNING, "'%s': no value set", Z_STRVAL_P(tmp_oid));
                                                efree(objid_query->vars);
                                                return FALSE;
                                        }
@@ -1425,7 +1425,7 @@ static int netsnmp_session_set_security(struct snmp_session *session, char *sec_
 */
 static void php_snmp(INTERNAL_FUNCTION_PARAMETERS, int st, int version)
 {
-       zval **oid, **value, **type;
+       zval *oid, *value, *type;
        char *a1, *a2, *a3, *a4, *a5, *a6, *a7;
        int a1_len, a2_len, a3_len, a4_len, a5_len, a6_len, a7_len;
        zend_bool use_orignames = 0, suffix_keys = 0;
@@ -1446,7 +1446,7 @@ static void php_snmp(INTERNAL_FUNCTION_PARAMETERS, int st, int version)
        if (session_less_mode) {
                if (version == SNMP_VERSION_3) {
                        if (st & SNMP_CMD_SET) {
-                               if (zend_parse_parameters(argc TSRMLS_CC, "sssssssZZZ|ll", &a1, &a1_len, &a2, &a2_len, &a3, &a3_len,
+                               if (zend_parse_parameters(argc TSRMLS_CC, "ssssssszzz|ll", &a1, &a1_len, &a2, &a2_len, &a3, &a3_len,
                                        &a4, &a4_len, &a5, &a5_len, &a6, &a6_len, &a7, &a7_len, &oid, &type, &value, &timeout, &retries) == FAILURE) {
                                        RETURN_FALSE;
                                }
@@ -1455,14 +1455,14 @@ static void php_snmp(INTERNAL_FUNCTION_PARAMETERS, int st, int version)
                                 * SNMP_CMD_GETNEXT
                                 * SNMP_CMD_WALK
                                 */
-                               if (zend_parse_parameters(argc TSRMLS_CC, "sssssssZ|ll", &a1, &a1_len, &a2, &a2_len, &a3, &a3_len,
+                               if (zend_parse_parameters(argc TSRMLS_CC, "sssssssz|ll", &a1, &a1_len, &a2, &a2_len, &a3, &a3_len,
                                        &a4, &a4_len, &a5, &a5_len, &a6, &a6_len, &a7, &a7_len, &oid, &timeout, &retries) == FAILURE) {
                                        RETURN_FALSE;
                                }
                        }
                } else {
                        if (st & SNMP_CMD_SET) {
-                               if (zend_parse_parameters(argc TSRMLS_CC, "ssZZZ|ll", &a1, &a1_len, &a2, &a2_len, &oid, &type, &value, &timeout, &retries) == FAILURE) {
+                               if (zend_parse_parameters(argc TSRMLS_CC, "sszzz|ll", &a1, &a1_len, &a2, &a2_len, &oid, &type, &value, &timeout, &retries) == FAILURE) {
                                        RETURN_FALSE;
                                }
                        } else {
@@ -1470,25 +1470,25 @@ static void php_snmp(INTERNAL_FUNCTION_PARAMETERS, int st, int version)
                                 * SNMP_CMD_GETNEXT
                                 * SNMP_CMD_WALK
                                 */
-                               if (zend_parse_parameters(argc TSRMLS_CC, "ssZ|ll", &a1, &a1_len, &a2, &a2_len, &oid, &timeout, &retries) == FAILURE) {
+                               if (zend_parse_parameters(argc TSRMLS_CC, "ssz|ll", &a1, &a1_len, &a2, &a2_len, &oid, &timeout, &retries) == FAILURE) {
                                        RETURN_FALSE;
                                }
                        }
                }
        } else {
                if (st & SNMP_CMD_SET) {
-                       if (zend_parse_parameters(argc TSRMLS_CC, "ZZZ", &oid, &type, &value) == FAILURE) {
+                       if (zend_parse_parameters(argc TSRMLS_CC, "zzz", &oid, &type, &value) == FAILURE) {
                                RETURN_FALSE;
                        }
                } else if (st & SNMP_CMD_WALK) {
-                       if (zend_parse_parameters(argc TSRMLS_CC, "Z|bll", &oid, &suffix_keys, &(objid_query.max_repetitions), &(objid_query.non_repeaters)) == FAILURE) {
+                       if (zend_parse_parameters(argc TSRMLS_CC, "z|bll", &oid, &suffix_keys, &(objid_query.max_repetitions), &(objid_query.non_repeaters)) == FAILURE) {
                                RETURN_FALSE;
                        }
                        if (suffix_keys) {
                                st |= SNMP_USE_SUFFIX_AS_KEYS;
                        }
                } else if (st & SNMP_CMD_GET) {
-                       if (zend_parse_parameters(argc TSRMLS_CC, "Z|b", &oid, &use_orignames) == FAILURE) {
+                       if (zend_parse_parameters(argc TSRMLS_CC, "z|b", &oid, &use_orignames) == FAILURE) {
                                RETURN_FALSE;
                        }
                        if (use_orignames) {
@@ -1497,7 +1497,7 @@ static void php_snmp(INTERNAL_FUNCTION_PARAMETERS, int st, int version)
                } else {
                        /* SNMP_CMD_GETNEXT
                         */
-                       if (zend_parse_parameters(argc TSRMLS_CC, "Z", &oid) == FAILURE) {
+                       if (zend_parse_parameters(argc TSRMLS_CC, "z", &oid) == FAILURE) {
                                RETURN_FALSE;
                        }
                }
@@ -1521,7 +1521,7 @@ static void php_snmp(INTERNAL_FUNCTION_PARAMETERS, int st, int version)
                }
        } else {
                zval *object = getThis();
-               snmp_object = (php_snmp_object *)zend_object_store_get_object(object TSRMLS_CC);
+               snmp_object = Z_SNMP_P(object);
                session = snmp_object->session;
                if (!session) {
                        php_error_docref(NULL TSRMLS_CC, E_WARNING, "Invalid or uninitialized SNMP object");
@@ -1819,7 +1819,7 @@ PHP_METHOD(snmp, __construct)
        int argc = ZEND_NUM_ARGS();
        zend_error_handling error_handling;
 
-       snmp_object = (php_snmp_object *)zend_object_store_get_object(object TSRMLS_CC);
+       snmp_object = Z_SNMP_P(object);
        zend_replace_error_handling(EH_THROW, NULL, &error_handling TSRMLS_CC);
        
        if (zend_parse_parameters(argc TSRMLS_CC, "lss|ll", &version, &a1, &a1_len, &a2, &a2_len, &timeout, &retries) == FAILURE) {
@@ -1864,7 +1864,7 @@ PHP_METHOD(snmp, close)
        php_snmp_object *snmp_object;
        zval *object = getThis();
 
-       snmp_object = (php_snmp_object *)zend_object_store_get_object(object TSRMLS_CC);
+       snmp_object = Z_SNMP_P(object);
 
        if (zend_parse_parameters_none() == FAILURE) {
                RETURN_FALSE;
@@ -1906,6 +1906,7 @@ PHP_METHOD(snmp, set)
 {
        php_snmp(INTERNAL_FUNCTION_PARAM_PASSTHRU, SNMP_CMD_SET, (-1));
 }
+/* }}} */
 
 /* {{{ proto bool SNMP::setSecurity(string sec_level, [ string auth_protocol, string auth_passphrase [, string priv_protocol, string priv_passphrase [, string contextName [, string contextEngineID]]]])
        Set SNMPv3 security-related session parameters */
@@ -1917,7 +1918,7 @@ PHP_METHOD(snmp, setSecurity)
        int a1_len = 0, a2_len = 0, a3_len = 0, a4_len = 0, a5_len = 0, a6_len = 0, a7_len = 0;
        int argc = ZEND_NUM_ARGS();
 
-       snmp_object = (php_snmp_object *)zend_object_store_get_object(object TSRMLS_CC);
+       snmp_object = Z_SNMP_P(object);
        
        if (zend_parse_parameters(argc TSRMLS_CC, "s|ssssss", &a1, &a1_len, &a2, &a2_len, &a3, &a3_len,
                &a4, &a4_len, &a5, &a5_len, &a6, &a6_len, &a7, &a7_len) == FAILURE) {
@@ -1939,7 +1940,7 @@ PHP_METHOD(snmp, getErrno)
        php_snmp_object *snmp_object;
        zval *object = getThis();
 
-       snmp_object = (php_snmp_object *)zend_object_store_get_object(object TSRMLS_CC);
+       snmp_object = Z_SNMP_P(object);
 
        RETVAL_LONG(snmp_object->snmp_errno);
        return;
@@ -1953,10 +1954,9 @@ PHP_METHOD(snmp, getError)
        php_snmp_object *snmp_object;
        zval *object = getThis();
 
-       snmp_object = (php_snmp_object *)zend_object_store_get_object(object TSRMLS_CC);
+       snmp_object = Z_SNMP_P(object);
 
-       RETVAL_STRING(snmp_object->snmp_errstr, 1);
-       return;
+       RETURN_STRING(snmp_object->snmp_errstr);
 }
 /* }}} */
 
@@ -1969,13 +1969,13 @@ void php_snmp_add_property(HashTable *h, const char *name, size_t name_length, p
        p.name_length = name_length;
        p.read_func = (read_func) ? read_func : NULL;
        p.write_func = (write_func) ? write_func : NULL;
-       zend_hash_add(h, (char *)name, name_length + 1, &p, sizeof(php_snmp_prop_handler), NULL);
+       zend_hash_str_add_mem(h, (char *)name, name_length, &p, sizeof(php_snmp_prop_handler));
 }
 /* }}} */
 
 /* {{{ php_snmp_read_property(zval *object, zval *member, int type[, const zend_literal *key])
    Generic object property reader */
-zval *php_snmp_read_property(zval *object, zval *member, int type, const zend_literal *key TSRMLS_DC)
+zval *php_snmp_read_property(zval *object, zval *member, int type, zend_uint cache_slot, zval *rv TSRMLS_DC)
 {
        zval tmp_member;
        zval *retval;
@@ -1983,113 +1983,106 @@ zval *php_snmp_read_property(zval *object, zval *member, int type, const zend_li
        php_snmp_prop_handler *hnd;
        int ret;
 
-       ret = FAILURE;
-       obj = (php_snmp_object *)zend_objects_get_address(object TSRMLS_CC);
+       obj = Z_SNMP_P(object); 
 
        if (Z_TYPE_P(member) != IS_STRING) {
-               tmp_member = *member;
-               zval_copy_ctor(&tmp_member);
+               ZVAL_COPY(&tmp_member, member);
                convert_to_string(&tmp_member);
                member = &tmp_member;
        }
 
-       ret = zend_hash_find(&php_snmp_properties, Z_STRVAL_P(member), Z_STRLEN_P(member)+1, (void **) &hnd);
+       hnd = zend_hash_find_ptr(&php_snmp_properties, Z_STR_P(member));
 
-       if (ret == SUCCESS && hnd->read_func) {
-               ret = hnd->read_func(obj, &retval TSRMLS_CC);
+       if (hnd && hnd->read_func) {
+               ret = hnd->read_func(obj, rv TSRMLS_CC);
                if (ret == SUCCESS) {
-                       /* ensure we're creating a temporary variable */
-                       Z_SET_REFCOUNT_P(retval, 0);
+                       retval = rv;
                } else {
-                       retval = EG(uninitialized_zval_ptr);
+                       retval = &EG(uninitialized_zval);
                }
        } else {
                zend_object_handlers * std_hnd = zend_get_std_object_handlers();
-               retval = std_hnd->read_property(object, member, type, key TSRMLS_CC);
+               retval = std_hnd->read_property(object, member, type, cache_slot, rv TSRMLS_CC);
        }
 
        if (member == &tmp_member) {
-               zval_dtor(member);
+               zval_ptr_dtor(member);
        }
-       return(retval);
+
+       return retval;
 }
 /* }}} */
 
 /* {{{ php_snmp_write_property(zval *object, zval *member, zval *value[, const zend_literal *key])
    Generic object property writer */
-void php_snmp_write_property(zval *object, zval *member, zval *value, const zend_literal *key TSRMLS_DC)
+void php_snmp_write_property(zval *object, zval *member, zval *value, zend_uint cache_slot TSRMLS_DC)
 {
        zval tmp_member;
        php_snmp_object *obj;
        php_snmp_prop_handler *hnd;
-       int ret;
 
        if (Z_TYPE_P(member) != IS_STRING) {
-               tmp_member = *member;
-               zval_copy_ctor(&tmp_member);
+               ZVAL_COPY(&tmp_member, member);
                convert_to_string(&tmp_member);
                member = &tmp_member;
        }
 
-       ret = FAILURE;
-       obj = (php_snmp_object *)zend_objects_get_address(object TSRMLS_CC);
+       obj = Z_SNMP_P(object);
 
-       ret = zend_hash_find(&php_snmp_properties, Z_STRVAL_P(member), Z_STRLEN_P(member) + 1, (void **) &hnd);
+       hnd = zend_hash_find_ptr(&php_snmp_properties, Z_STR_P(member));
 
-       if (ret == SUCCESS && hnd->write_func) {
+       if (hnd && hnd->write_func) {
                hnd->write_func(obj, value TSRMLS_CC);
-               if (! PZVAL_IS_REF(value) && Z_REFCOUNT_P(value) == 0) {
+               /*
+               if (!PZVAL_IS_REF(value) && Z_REFCOUNT_P(value) == 0) {
                        Z_ADDREF_P(value);
                        zval_ptr_dtor(&value);
                }
+               */
        } else {
                zend_object_handlers * std_hnd = zend_get_std_object_handlers();
-               std_hnd->write_property(object, member, value, key TSRMLS_CC);
+               std_hnd->write_property(object, member, value, cache_slot TSRMLS_CC);
        }
 
        if (member == &tmp_member) {
-               zval_dtor(member);
+               zval_ptr_dtor(member);
        }
 }
 /* }}} */
 
 /* {{{ php_snmp_has_property(zval *object, zval *member, int has_set_exists[, const zend_literal *key])
    Generic object property checker */
-static int php_snmp_has_property(zval *object, zval *member, int has_set_exists, const zend_literal *key TSRMLS_DC)
+static int php_snmp_has_property(zval *object, zval *member, int has_set_exists, zend_uint cache_slot TSRMLS_DC)
 {
+       zval rv;
        php_snmp_prop_handler *hnd;
        int ret = 0;
 
-       if (zend_hash_find(&php_snmp_properties, Z_STRVAL_P(member), Z_STRLEN_P(member) + 1, (void **)&hnd) == SUCCESS) {
+       if ((hnd = zend_hash_find_ptr(&php_snmp_properties, Z_STR_P(member))) != NULL) {
                switch (has_set_exists) {
                        case 2:
                                ret = 1;
                                break;
                        case 0: {
-                               zval *value = php_snmp_read_property(object, member, BP_VAR_IS, key TSRMLS_CC);
-                               if (value != EG(uninitialized_zval_ptr)) {
-                                       ret = Z_TYPE_P(value) != IS_NULL? 1:0;
-                                       /* refcount is 0 */
-                                       Z_ADDREF_P(value);
-                                       zval_ptr_dtor(&value);
+                               zval *value = php_snmp_read_property(object, member, BP_VAR_IS, cache_slot, &rv TSRMLS_CC);
+                               if (value != &EG(uninitialized_zval)) {
+                                       ret = Z_TYPE_P(value) != IS_NULL? 1 : 0;
+                                       zval_ptr_dtor(value);
                                }
                                break;
                        }
                        default: {
-                               zval *value = php_snmp_read_property(object, member, BP_VAR_IS, key TSRMLS_CC);
-                               if (value != EG(uninitialized_zval_ptr)) {
+                               zval *value = php_snmp_read_property(object, member, BP_VAR_IS, cache_slot, &rv TSRMLS_CC);
+                               if (value != &EG(uninitialized_zval)) {
                                        convert_to_boolean(value);
-                                       ret = Z_BVAL_P(value)? 1:0;
-                                       /* refcount is 0 */
-                                       Z_ADDREF_P(value);
-                                       zval_ptr_dtor(&value);
+                                       ret = Z_TYPE_P(value) == IS_TRUE? 1:0;
                                }
                                break;
                        }
                }
        } else {
-               zend_object_handlers * std_hnd = zend_get_std_object_handlers();
-               ret = std_hnd->has_property(object, member, has_set_exists, key TSRMLS_CC);
+               zend_object_handlers *std_hnd = zend_get_std_object_handlers();
+               ret = std_hnd->has_property(object, member, has_set_exists, cache_slot TSRMLS_CC);
        }
        return ret;
 }
@@ -2102,80 +2095,67 @@ static HashTable *php_snmp_get_properties(zval *object TSRMLS_DC)
        php_snmp_object *obj;
        php_snmp_prop_handler *hnd;
        HashTable *props;
-       zval *val;
-       char *key;
-       uint key_len;
-       HashPosition pos;
+       zval rv;
+       zend_string *key;
        ulong num_key;
 
-       obj = (php_snmp_object *)zend_objects_get_address(object TSRMLS_CC);
+       obj = Z_SNMP_P(object);
        props = zend_std_get_properties(object TSRMLS_CC);
 
-       zend_hash_internal_pointer_reset_ex(&php_snmp_properties, &pos);
-
-       while (zend_hash_get_current_data_ex(&php_snmp_properties, (void**)&hnd, &pos) == SUCCESS) {
-               zend_hash_get_current_key_ex(&php_snmp_properties, &key, &key_len, &num_key, 0, &pos);
-               if (!hnd->read_func || hnd->read_func(obj, &val TSRMLS_CC) != SUCCESS) {
-                       val = EG(uninitialized_zval_ptr);
-                       Z_ADDREF_P(val);
+       ZEND_HASH_FOREACH_KEY_PTR(&php_snmp_properties, num_key, key, hnd) {
+               if (!hnd->read_func || hnd->read_func(obj, &rv TSRMLS_CC) != SUCCESS) {
+                       ZVAL_NULL(&rv);
                }
-               zend_hash_update(props, key, key_len, (void *)&val, sizeof(zval *), NULL);
-               zend_hash_move_forward_ex(&php_snmp_properties, &pos);
-       }
+               zend_hash_update(props, key, &rv);
+       } ZEND_HASH_FOREACH_END();
+
        return obj->zo.properties;
 }
 /* }}} */
 
 /* {{{ */
-static int php_snmp_read_info(php_snmp_object *snmp_object, zval **retval TSRMLS_DC)
+static int php_snmp_read_info(php_snmp_object *snmp_object, zval *retval TSRMLS_DC)
 {
-       zval *val;
+       zval val;
 
-       MAKE_STD_ZVAL(*retval);
-       array_init(*retval);
+       array_init(retval);
 
        if (snmp_object->session == NULL) {
                return SUCCESS;
        }
                
-       MAKE_STD_ZVAL(val);
-       ZVAL_STRINGL(val, snmp_object->session->peername, strlen(snmp_object->session->peername), 1);
-       add_assoc_zval(*retval, "hostname", val);
+       ZVAL_STRINGL(&val, snmp_object->session->peername, strlen(snmp_object->session->peername));
+       add_assoc_zval(retval, "hostname", &val);
        
-       MAKE_STD_ZVAL(val);
-       ZVAL_LONG(val, snmp_object->session->remote_port);
-       add_assoc_zval(*retval, "port", val);
+       ZVAL_LONG(&val, snmp_object->session->remote_port);
+       add_assoc_zval(retval, "port", &val);
        
-       MAKE_STD_ZVAL(val);
-       ZVAL_LONG(val, snmp_object->session->timeout);
-       add_assoc_zval(*retval, "timeout", val);
+       ZVAL_LONG(&val, snmp_object->session->timeout);
+       add_assoc_zval(retval, "timeout", &val);
        
-       MAKE_STD_ZVAL(val);
-       ZVAL_LONG(val, snmp_object->session->retries);
-       add_assoc_zval(*retval, "retries", val);
+       ZVAL_LONG(&val, snmp_object->session->retries);
+       add_assoc_zval(retval, "retries", &val);
        
        return SUCCESS;
 }
 /* }}} */
 
 /* {{{ */
-static int php_snmp_read_max_oids(php_snmp_object *snmp_object, zval **retval TSRMLS_DC)
+static int php_snmp_read_max_oids(php_snmp_object *snmp_object, zval *retval TSRMLS_DC)
 {
-       MAKE_STD_ZVAL(*retval);
        if (snmp_object->max_oids > 0) {
-               ZVAL_LONG(*retval, snmp_object->max_oids);
+               ZVAL_LONG(retval, snmp_object->max_oids);
        } else {
-               ZVAL_NULL(*retval);
+               ZVAL_NULL(retval);
        }
        return SUCCESS;
 }
 /* }}} */
 
 #define PHP_SNMP_BOOL_PROPERTY_READER_FUNCTION(name) \
-       static int php_snmp_read_##name(php_snmp_object *snmp_object, zval **retval TSRMLS_DC) \
+       static int php_snmp_read_##name(php_snmp_object *snmp_object, zval *retval TSRMLS_DC) \
        { \
-               MAKE_STD_ZVAL(*retval); \
-               ZVAL_BOOL(*retval, snmp_object->name); \
+               ZVAL_BOOL(retval, snmp_object->name); \
                return SUCCESS; \
        }
 
@@ -2184,10 +2164,9 @@ PHP_SNMP_BOOL_PROPERTY_READER_FUNCTION(quick_print)
 PHP_SNMP_BOOL_PROPERTY_READER_FUNCTION(enum_print)
 
 #define PHP_SNMP_LONG_PROPERTY_READER_FUNCTION(name) \
-       static int php_snmp_read_##name(php_snmp_object *snmp_object, zval **retval TSRMLS_DC) \
+       static int php_snmp_read_##name(php_snmp_object *snmp_object, zval *retval TSRMLS_DC) \
        { \
-               MAKE_STD_ZVAL(*retval); \
-               ZVAL_LONG(*retval, snmp_object->name); \
+               ZVAL_LONG(retval, snmp_object->name); \
                return SUCCESS; \
        }
 
@@ -2267,18 +2246,12 @@ static int php_snmp_write_valueretrieval(php_snmp_object *snmp_object, zval *new
 static int php_snmp_write_##name(php_snmp_object *snmp_object, zval *newval TSRMLS_DC) \
 { \
        zval ztmp; \
-       if (Z_TYPE_P(newval) != IS_BOOL) { \
-               ztmp = *newval; \
-               zval_copy_ctor(&ztmp); \
-               convert_to_boolean(&ztmp); \
-               newval = &ztmp; \
-       } \
+       ZVAL_COPY(&ztmp, newval); \
+       convert_to_boolean(&ztmp); \
+       newval = &ztmp; \
 \
-       snmp_object->name = Z_LVAL_P(newval); \
+       snmp_object->name = Z_TYPE_P(newval) == IS_TRUE? 1 : 0; \
 \
-       if (newval == &ztmp) { \
-               zval_dtor(newval); \
-       } \
        return SUCCESS; \
 }
 
@@ -2292,8 +2265,7 @@ static int php_snmp_write_oid_output_format(php_snmp_object *snmp_object, zval *
        zval ztmp;
        int ret = SUCCESS;
        if (Z_TYPE_P(newval) != IS_LONG) {
-               ztmp = *newval;
-               zval_copy_ctor(&ztmp);
+               ZVAL_COPY(&ztmp, newval);
                convert_to_long(&ztmp);
                newval = &ztmp;
        }
@@ -2314,7 +2286,7 @@ static int php_snmp_write_oid_output_format(php_snmp_object *snmp_object, zval *
        }
 
        if (newval == &ztmp) {
-               zval_dtor(newval);
+               zval_ptr_dtor(newval);
        }
        return ret;
 }
@@ -2326,8 +2298,7 @@ static int php_snmp_write_exceptions_enabled(php_snmp_object *snmp_object, zval
        zval ztmp;
        int ret = SUCCESS;
        if (Z_TYPE_P(newval) != IS_LONG) {
-               ztmp = *newval;
-               zval_copy_ctor(&ztmp);
+               ZVAL_COPY(&ztmp, newval);
                convert_to_long(&ztmp);
                newval = &ztmp;
        }
@@ -2335,12 +2306,18 @@ static int php_snmp_write_exceptions_enabled(php_snmp_object *snmp_object, zval
        snmp_object->exceptions_enabled = Z_LVAL_P(newval);     
 
        if (newval == &ztmp) {
-               zval_dtor(newval);
+               zval_ptr_dtor(newval);
        }
        return ret;
 }
 /* }}} */
 
+static void free_php_snmp_properties(zval *el)  /* {{{ */
+{
+       pefree(Z_PTR_P(el), 1);
+}
+/* }}} */
+
 /* {{{ php_snmp_class_methods[] */
 static zend_function_entry php_snmp_class_methods[] = {
        PHP_ME(snmp,     __construct,                   arginfo_snmp_create,            ZEND_ACC_PUBLIC)
@@ -2405,11 +2382,13 @@ PHP_MINIT_FUNCTION(snmp)
        /* Register SNMP Class */
        INIT_CLASS_ENTRY(ce, "SNMP", php_snmp_class_methods);
        ce.create_object = php_snmp_object_new;
+       php_snmp_object_handlers.offset = XtOffsetOf(php_snmp_object, zo);
        php_snmp_object_handlers.clone_obj = NULL;
+       php_snmp_object_handlers.free_obj = php_snmp_object_free_storage;
        php_snmp_ce = zend_register_internal_class(&ce TSRMLS_CC);
 
        /* Register SNMP Class properties */
-       zend_hash_init(&php_snmp_properties, 0, NULL, NULL, 1);
+       zend_hash_init(&php_snmp_properties, 0, NULL, free_php_snmp_properties, 1);
        PHP_SNMP_ADD_PROPERTIES(&php_snmp_properties, php_snmp_property_entries);
 
        REGISTER_LONG_CONSTANT("SNMP_OID_OUTPUT_SUFFIX",        NETSNMP_OID_OUTPUT_SUFFIX,      CONST_CS | CONST_PERSISTENT);