util: avoid inline access to Symbol.iterator

PR-URL: #43683
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>

Author: cola119

lib/internal/util/inspect.js

@@ -859,7 +859,7 @@ function formatRaw(ctx, value, recurseTimes, typedArray) {
   // Iterators and the rest are split to reduce checks.
   // We have to check all values in case the constructor is set to null.
   // Otherwise it would not possible to identify all types properly.
-  if (value[SymbolIterator] || constructor === null) {
+  if (SymbolIterator in value || constructor === null) {
     noIterator = false;
     if (ArrayIsArray(value)) {
       // Only set the constructor for non ordinary ("Array [...]") arrays.

test/parallel/test-util-inspect.js

@@ -3240,3 +3240,12 @@ assert.strictEqual(
     '-123_456_789.123_456_78'
   );
 }
+
+// Regression test for https://github.com/nodejs/node/issues/41244
+{
+  assert.strictEqual(util.inspect({
+    get [Symbol.iterator]() {
+      throw new Error();
+    }
+  }), '{ [Symbol(Symbol.iterator)]: [Getter] }');
+}