worker: refactor to avoid unsafe array iteration

PR-URL: #37346 Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Harshitha K P <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Feb 16, 2021 · 05a16e7 · 05a16e7
1 parent 6d53e79
commit 05a16e7
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/lib/internal/main/worker_thread.js b/lib/internal/main/worker_thread.js
@@ -5,6 +5,7 @@
 
 const {
   ArrayPrototypeConcat,
+  ArrayPrototypeForEach,
   ArrayPrototypeSplice,
   ObjectDefineProperty,
   PromisePrototypeCatch,
@@ -178,8 +179,9 @@ port.on('message', (message) => {
     }
   } else if (message.type === STDIO_PAYLOAD) {
     const { stream, chunks } = message;
-    for (const { chunk, encoding } of chunks)
+    ArrayPrototypeForEach(chunks, ({ chunk, encoding }) => {
       process[stream].push(chunk, encoding);
+    });
   } else {
     assert(
       message.type === STDIO_WANTS_MORE_DATA,