process: use defineProperty instead of assignment

PR-URL: nodejs/node#43907 Fixes: nodejs/node#43496 Refs: endojs/endo#576 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Mestery <mestery@protonmail.com>
noslate-project · Oct 10, 2022 · db6a5f2 · db6a5f2
1 parent 40130cc
commit db6a5f2
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/lib/internal/process/per_thread.js b/lib/internal/process/per_thread.js
@@ -15,6 +15,7 @@ const {
   Float64Array,
   NumberMAX_SAFE_INTEGER,
   ObjectFreeze,
+  ObjectDefineProperty,
   ReflectApply,
   RegExpPrototypeTest,
   SafeArrayIterator,
@@ -371,9 +372,15 @@ function buildAllowedFlags() {
       return SetPrototypeValues(this[kInternal].set);
     }
   }
-  NodeEnvironmentFlagsSet.prototype.keys =
-  NodeEnvironmentFlagsSet.prototype[SymbolIterator] =
-    NodeEnvironmentFlagsSet.prototype.values;
+  const flagSetValues = NodeEnvironmentFlagsSet.prototype.values;
+  ObjectDefineProperty(NodeEnvironmentFlagsSet.prototype, SymbolIterator, {
+    __proto__: null,
+    value: flagSetValues,
+  });
+  ObjectDefineProperty(NodeEnvironmentFlagsSet.prototype, 'keys', {
+    __proto__: null,
+    value: flagSetValues,
+  });
 
   ObjectFreeze(NodeEnvironmentFlagsSet.prototype.constructor);
   ObjectFreeze(NodeEnvironmentFlagsSet.prototype);