stream: refactor to avoid unsafe array iteration

nodejs · Jan 29, 2021 · 1666ca7 · 1666ca7
1 parent 6be2855
commit 1666ca7
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 10 deletions.
diff --git a/lib/internal/streams/readable.js b/lib/internal/streams/readable.js
@@ -22,6 +22,7 @@
 'use strict';
 
 const {
+  ArrayPrototypeForEach,
   ArrayPrototypeIndexOf,
   ArrayPrototypePush,
   ArrayPrototypeSplice,
@@ -854,8 +855,8 @@ Readable.prototype.unpipe = function(dest) {
     state.pipes = [];
     this.pause();
 
-    for (const dest of dests)
-      dest.emit('unpipe', this, { hasUnpiped: false });
+    ArrayPrototypeForEach(dests, (dest) =>
+      dest.emit('unpipe', this, { hasUnpiped: false }));
     return this;
   }
 
@@ -1056,11 +1057,11 @@ Readable.prototype.wrap = function(stream) {
   };
 
   // Proxy all the other methods. Important when wrapping filters and duplexes.
-  for (const i of ObjectKeys(stream)) {
+  ArrayPrototypeForEach(ObjectKeys(stream), (i) => {
     if (this[i] === undefined && typeof stream[i] === 'function') {
       this[i] = FunctionPrototypeBind(stream[i], stream);
     }
-  }
+  });
 
   return this;
 };

diff --git a/lib/internal/streams/writable.js b/lib/internal/streams/writable.js
@@ -26,6 +26,7 @@
 'use strict';
 
 const {
+  ArrayPrototypeForEach,
   ArrayPrototypePush,
   ArrayPrototypeSlice,
   ArrayPrototypeSplice,
@@ -522,9 +523,10 @@ function errorBuffer(state) {
     callback(new ERR_STREAM_DESTROYED('write'));
   }
 
-  for (const callback of ArrayPrototypeSplice(state[kOnFinished], 0)) {
-    callback(new ERR_STREAM_DESTROYED('end'));
-  }
+  ArrayPrototypeForEach(
+    ArrayPrototypeSplice(state[kOnFinished], 0),
+    (callback) => callback(new ERR_STREAM_DESTROYED('end'))
+  );
 
   resetBuffer(state);
 }
@@ -744,9 +746,8 @@ function finish(stream, state) {
 
   state.finished = true;
 
-  for (const callback of ArrayPrototypeSplice(state[kOnFinished], 0)) {
-    callback();
-  }
+  ArrayPrototypeForEach(ArrayPrototypeSplice(state[kOnFinished], 0),
+                        (callback) => callback());
 
   stream.emit('finish');