Implement ZEND_ACC_HAS_RC_PROPS

Author: nielsdos

Zend/zend_API.c

@@ -1609,15 +1609,16 @@ static zend_always_inline void _object_properties_init(zend_object *object, zend
 		zval *dst = object->properties_table;
 		zval *end = src + class_type->default_properties_count;
 
-		if (UNEXPECTED(class_type->type == ZEND_INTERNAL_CLASS)) {
+		if (EXPECTED(class_type->ce_flags & ZEND_ACC_HAS_RC_PROPS)) {
 			do {
-				ZVAL_COPY_OR_DUP_PROP(dst, src);
+				ZVAL_COPY_PROP(dst, src);
 				src++;
 				dst++;
 			} while (src != end);
 		} else {
+			/* zend_declare_typed_property() disallows refcounted default property values in internal classes */
 			do {
-				ZVAL_COPY_PROP(dst, src);
+				ZVAL_COPY_VALUE_PROP(dst, src);
 				src++;
 				dst++;
 			} while (src != end);
@@ -4378,6 +4379,9 @@ ZEND_API zend_property_info *zend_declare_typed_property(zend_class_entry *ce, z
 				ce->properties_info_table[ce->default_properties_count - 1] = property_info;
 			}
 		}
+		if (Z_REFCOUNTED_P(property)) {
+			ce->ce_flags |= ZEND_ACC_HAS_RC_PROPS;
+		}
 		property_default_ptr = &ce->default_properties_table[OBJ_PROP_TO_NUM(property_info->offset)];
 		ZVAL_COPY_VALUE(property_default_ptr, property);
 		Z_PROP_FLAG_P(property_default_ptr) = Z_ISUNDEF_P(property) ? IS_PROP_UNINIT : 0;

Zend/zend_compile.h

@@ -239,7 +239,7 @@ typedef struct _zend_oparray_context {
 /* or IS_CONSTANT_VISITED_MARK                            |     |     |     */
 #define ZEND_CLASS_CONST_IS_CASE         (1 << 6)  /*     |     |     |  X  */
 /*                                                        |     |     |     */
-/* Class Flags (unused: 30,31)                            |     |     |     */
+/* Class Flags (unused: 31)                               |     |     |     */
 /* ===========                                            |     |     |     */
 /*                                                        |     |     |     */
 /* Special class types                                    |     |     |     */
@@ -304,6 +304,8 @@ typedef struct _zend_oparray_context {
 /*                                                        |     |     |     */
 /* Class cannot be serialized or unserialized             |     |     |     */
 #define ZEND_ACC_NOT_SERIALIZABLE        (1 << 29) /*  X  |     |     |     */
+/* Class has refcounted props                             |     |     |     */
+#define ZEND_ACC_HAS_RC_PROPS            (1 << 30) /*  X  |     |     |     */
 /*                                                        |     |     |     */
 /* Function Flags (unused: 29-30)                         |     |     |     */
 /* ==============                                         |     |     |     */

Zend/zend_inheritance.c

@@ -1672,7 +1672,7 @@ ZEND_API void zend_do_inheritance_ex(zend_class_entry *ce, zend_class_entry *par
 			ce->ce_flags |= ZEND_ACC_EXPLICIT_ABSTRACT_CLASS;
 		}
 	}
-	ce->ce_flags |= parent_ce->ce_flags & (ZEND_HAS_STATIC_IN_METHODS | ZEND_ACC_HAS_TYPE_HINTS | ZEND_ACC_HAS_READONLY_PROPS | ZEND_ACC_USE_GUARDS | ZEND_ACC_NOT_SERIALIZABLE | ZEND_ACC_ALLOW_DYNAMIC_PROPERTIES);
+	ce->ce_flags |= parent_ce->ce_flags & (ZEND_HAS_STATIC_IN_METHODS | ZEND_ACC_HAS_TYPE_HINTS | ZEND_ACC_HAS_READONLY_PROPS | ZEND_ACC_USE_GUARDS | ZEND_ACC_NOT_SERIALIZABLE | ZEND_ACC_ALLOW_DYNAMIC_PROPERTIES | ZEND_ACC_HAS_RC_PROPS);
 }
 /* }}} */
 
@@ -2537,6 +2537,7 @@ static void zend_do_traits_property_binding(zend_class_entry *ce, zend_class_ent
 				}
 			}
 		} ZEND_HASH_FOREACH_END();
+		ce->ce_flags |= traits[i]->ce_flags & ZEND_ACC_HAS_RC_PROPS;
 	}
 }
 /* }}} */