fix(object): check extensibility before integrity properties

quickjs.c

@@ -42017,6 +42017,12 @@ static JSValue js_object_isSealed(JSContext *ctx, JSValueConst this_val,
         return JS_TRUE;
 
     p = JS_VALUE_GET_OBJ(obj);
+    res = JS_IsExtensible(ctx, obj);
+    if (res < 0)
+        return JS_EXCEPTION;
+    if (res)
+        return JS_FALSE;
+
     flags = JS_GPN_STRING_MASK | JS_GPN_SYMBOL_MASK;
     if (JS_GetOwnPropertyNamesInternal(ctx, &props, &len, p, flags))
         return JS_EXCEPTION;
@@ -42036,10 +42042,7 @@ static JSValue js_object_isSealed(JSContext *ctx, JSValueConst this_val,
             }
         }
     }
-    res = JS_IsExtensible(ctx, obj);
-    if (res < 0)
-        return JS_EXCEPTION;
-    res ^= 1;
+    res = true;
 done:
     js_free_prop_enum(ctx, props, len);
     return js_bool(res);

tests/test_builtin.js

@@ -210,6 +210,36 @@ function test()
     assertThrows(TypeError, () => Object.setPrototypeOf(Object.prototype, {}));
 }
 
+function test_object_integrity_check_proxy_order()
+{
+    var log = [];
+    function logged(target) {
+        return new Proxy(target, {
+            isExtensible(t) {
+                log.push("isExtensible");
+                return Reflect.isExtensible(t);
+            },
+            ownKeys(t) {
+                log.push("ownKeys");
+                return Reflect.ownKeys(t);
+            },
+            getOwnPropertyDescriptor(t, k) {
+                log.push("getOwnPropertyDescriptor " + k);
+                return Reflect.getOwnPropertyDescriptor(t, k);
+            },
+        });
+    }
+
+    Object.isFrozen(logged({ x: 1 }));
+    assert(log.join(", "), "isExtensible", "isFrozen checks extensibility first");
+
+    log = [];
+    Object.isSealed(logged(Object.freeze({ x: 1 })));
+    assert(log.join(", "),
+           "isExtensible, ownKeys, getOwnPropertyDescriptor x",
+           "isSealed checks extensibility first");
+}
+
 function test_enum()
 {
     var a, tab;
@@ -1331,6 +1361,7 @@ function test_cur_pc()
 }
 
 test();
+test_object_integrity_check_proxy_order();
 test_function();
 test_enum();
 test_array();