Handle refs, add pure enum test

Author: iluuu1994

Zend/tests/gh9775.phpt renamed to Zend/tests/gh9775_1.phpt

@@ -1,5 +1,5 @@
 --TEST--
-GH-9775: Enum in array_unique()
+GH-9775: Backed enum in array_unique()
 --FILE--
 <?php
 
@@ -15,6 +15,8 @@ enum Test: string
     case COUPONS_ADMIN = 'coupons.admin';
 }
 
+$instructorsAdmin = Test::INSTRUCTORS_ADMIN;
+
 $data = [
     Test::COURSES_ADMIN,
     Test::COURSES_REPORTING_ACCESS,
@@ -23,7 +25,7 @@ $data = [
     Test::B2B_DASHBOARD_ACCESS,
     Test::B2B_DASHBOARD_ACCESS,
     Test::INSTRUCTORS_ADMIN,
-    Test::INSTRUCTORS_ADMIN,
+    &$instructorsAdmin,
     Test::COUPONS_ADMIN,
     Test::AUTHENTICATED,
 ];

Zend/tests/gh9775_2.phpt

@@ -0,0 +1,56 @@
+--TEST--
+GH-9775: Pure enum in array_unique()
+--FILE--
+<?php
+
+enum Test
+{
+    case AUTHENTICATED;
+    case COURSES_ADMIN;
+    case BUNDLES_ADMIN;
+    case COURSES_REPORTING_ACCESS;
+    case B2B_DASHBOARD_ACCESS;
+    case INSTRUCTORS_ADMIN;
+    case USERS_ADMIN;
+    case COUPONS_ADMIN;
+}
+
+$instructorsAdmin = Test::INSTRUCTORS_ADMIN;
+
+$data = [
+    Test::COURSES_ADMIN,
+    Test::COURSES_REPORTING_ACCESS,
+    Test::BUNDLES_ADMIN,
+    Test::USERS_ADMIN,
+    Test::B2B_DASHBOARD_ACCESS,
+    Test::B2B_DASHBOARD_ACCESS,
+    Test::INSTRUCTORS_ADMIN,
+    &$instructorsAdmin,
+    Test::COUPONS_ADMIN,
+    Test::AUTHENTICATED,
+];
+
+$data = array_unique($data, flags: SORT_REGULAR);
+
+var_dump($data);
+
+?>
+--EXPECT--
+array(8) {
+  [0]=>
+  enum(Test::COURSES_ADMIN)
+  [1]=>
+  enum(Test::COURSES_REPORTING_ACCESS)
+  [2]=>
+  enum(Test::BUNDLES_ADMIN)
+  [3]=>
+  enum(Test::USERS_ADMIN)
+  [4]=>
+  enum(Test::B2B_DASHBOARD_ACCESS)
+  [6]=>
+  enum(Test::INSTRUCTORS_ADMIN)
+  [8]=>
+  enum(Test::COUPONS_ADMIN)
+  [9]=>
+  enum(Test::AUTHENTICATED)
+}

ext/standard/array.c

@@ -346,12 +346,15 @@ static zend_always_inline int php_array_key_compare_string_locale_unstable_i(Buc
 static zend_always_inline int php_array_data_compare_unstable_i(Bucket *f, Bucket *s) /* {{{ */
 {
 	int result = zend_compare(&f->val, &s->val);
-	// Special handling for enums
+	/* Special enums handling for array_unique. We don't want to add this logic to zend_compare as
+	 * that would be observable via comparison operators. */
 	zval *rhs = &s->val;
+	ZVAL_DEREF(rhs);
 	if (UNEXPECTED(Z_TYPE_P(rhs) == IS_OBJECT)
 	 && result == ZEND_UNCOMPARABLE
 	 && (Z_OBJCE_P(rhs)->ce_flags & ZEND_ACC_ENUM)) {
 		zval *lhs = &f->val;
+		ZVAL_DEREF(lhs);
 		if (Z_TYPE_P(lhs) == IS_OBJECT && (Z_OBJCE_P(lhs)->ce_flags & ZEND_ACC_ENUM)) {
 			// Order doesn't matter, we just need to group the same enum values
 			uintptr_t lhs_uintptr = (uintptr_t)Z_OBJ_P(lhs);