From bb8e5de1817b4c001394124635f77a7c91a3a568 Mon Sep 17 00:00:00 2001 From: Dmitry Stogov Date: Tue, 11 Oct 2005 10:15:40 +0000 Subject: [PATCH] Fixed bug #32455 (wrong setting property to unset value) --- NEWS | 1 + ext/soap/php_encoding.c | 10 +++++++++- 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 16152a1012..c1ad8bcd29 100644 --- a/NEWS +++ b/NEWS @@ -174,6 +174,7 @@ PHP NEWS seg fault). (Dmitry) - Fixed bug #32937 (open_basedir looses trailing / in the limiter). (Adam Conrad) - Fixed bug #32589 (possible crash inside imap_mail_compose() function). (Ilia) +- Fixed bug #32455 (wrong setting property to unset value). (Dmitry) - Fixed bug #32179 (xmlrpc_encode() segfaults with recursive references). (Tony) - Fixed bug #32139 (SOAP client does not auto-handle base64 encoding). (Ilia) - Fixed bug #32010 (Memory leak in mssql_fetch_batch). (fmk) diff --git a/ext/soap/php_encoding.c b/ext/soap/php_encoding.c index 5905bc2d6a..c0494b8f98 100644 --- a/ext/soap/php_encoding.c +++ b/ext/soap/php_encoding.c @@ -991,10 +991,18 @@ static zval* get_zval_property(zval* object, char* name TSRMLS_DC) old_scope = EG(scope); EG(scope) = Z_OBJCE_P(object); data = Z_OBJ_HT_P(object)->read_property(object, &member, BP_VAR_IS TSRMLS_CC); - EG(scope) = old_scope; if (data == EG(uninitialized_zval_ptr)) { + /* Hack for bug #32455 */ + zend_property_info *property_info; + + property_info = zend_get_property_info(Z_OBJCE_P(object), &member, 1 TSRMLS_CC); + EG(scope) = old_scope; + if (property_info && zend_hash_quick_exists(Z_OBJPROP_P(object), property_info->name, property_info->name_length+1, property_info->h)) { + return data; + } return NULL; } + EG(scope) = old_scope; return data; } else if (Z_TYPE_P(object) == IS_ARRAY) { zval **data_ptr; -- 2.40.0