diff options
| -rw-r--r-- | Zend/zend_object_handlers.c | 5 | ||||
| -rw-r--r-- | Zend/zend_object_handlers.h | 4 | ||||
| -rw-r--r-- | ext/sockets/sockets.c | 1 | ||||
| -rw-r--r-- | ext/sockets/tests/bug80723.phpt | 21 |
4 files changed, 31 insertions, 0 deletions
diff --git a/Zend/zend_object_handlers.c b/Zend/zend_object_handlers.c index 64f1450daa..e99af731f9 100644 --- a/Zend/zend_object_handlers.c +++ b/Zend/zend_object_handlers.c @@ -1608,6 +1608,11 @@ ZEND_API int zend_std_compare_objects(zval *o1, zval *o2) /* {{{ */ } /* }}} */ +ZEND_API int zend_objects_not_comparable(zval *o1, zval *o2) +{ + return ZEND_UNCOMPARABLE; +} + ZEND_API int zend_std_has_property(zend_object *zobj, zend_string *name, int has_set_exists, void **cache_slot) /* {{{ */ { int result; diff --git a/Zend/zend_object_handlers.h b/Zend/zend_object_handlers.h index 1eca06f266..71c0168372 100644 --- a/Zend/zend_object_handlers.h +++ b/Zend/zend_object_handlers.h @@ -225,6 +225,10 @@ ZEND_API int zend_std_compare_objects(zval *o1, zval *o2); ZEND_API int zend_std_get_closure(zend_object *obj, zend_class_entry **ce_ptr, zend_function **fptr_ptr, zend_object **obj_ptr, bool check_only); ZEND_API void rebuild_object_properties(zend_object *zobj); +/* Handler for objects that cannot be meaningfully compared. + * Only objects with the same identity will be considered equal. */ +ZEND_API int zend_objects_not_comparable(zval *o1, zval *o2); + ZEND_API int zend_check_protected(zend_class_entry *ce, zend_class_entry *scope); ZEND_API int zend_check_property_access(zend_object *zobj, zend_string *prop_info_name, bool is_dynamic); diff --git a/ext/sockets/sockets.c b/ext/sockets/sockets.c index 204fa98a47..5bc058c4b7 100644 --- a/ext/sockets/sockets.c +++ b/ext/sockets/sockets.c @@ -443,6 +443,7 @@ static PHP_MINIT_FUNCTION(sockets) socket_object_handlers.get_constructor = socket_get_constructor; socket_object_handlers.clone_obj = NULL; socket_object_handlers.get_gc = socket_get_gc; + socket_object_handlers.compare = zend_objects_not_comparable; address_info_ce = register_class_AddressInfo(); address_info_ce->create_object = address_info_create_object; diff --git a/ext/sockets/tests/bug80723.phpt b/ext/sockets/tests/bug80723.phpt new file mode 100644 index 0000000000..a05d857900 --- /dev/null +++ b/ext/sockets/tests/bug80723.phpt @@ -0,0 +1,21 @@ +--TEST-- +Bug #80723: Different sockets compare as equal (regression in 8.0) +--FILE-- +<?php +$socket_1 = socket_create(AF_INET, SOCK_STREAM, SOL_TCP); +$socket_2 = socket_create(AF_INET, SOCK_STREAM, SOL_TCP); +var_dump($socket_1 == $socket_1); +var_dump($socket_2 == $socket_2); +var_dump($socket_1 == $socket_2); + +$vector = array(1 => $socket_1, 2 => $socket_2); +var_dump(array_search($socket_1, $vector)); +var_dump(array_search($socket_2, $vector)); + +?> +--EXPECT-- +bool(true) +bool(true) +bool(false) +int(1) +int(2) |