zlib: refactor to avoid unsafe array iteration

PR-URL: #36722 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Pooja D P <[email protected]>
nodejs · Jan 13, 2021 · 251a0ff · 251a0ff
1 parent 3a44f37
commit 251a0ff
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/lib/zlib.js b/lib/zlib.js
@@ -23,6 +23,7 @@
 
 const {
   ArrayBuffer,
+  ArrayPrototypeForEach,
   ArrayPrototypeMap,
   ArrayPrototypePush,
   Error,
@@ -803,8 +804,8 @@ function Brotli(opts, mode) {
   assert(mode === BROTLI_DECODE || mode === BROTLI_ENCODE);
 
   TypedArrayPrototypeFill(brotliInitParamsArray, -1);
-  if (opts && opts.params) {
-    for (const origKey of ObjectKeys(opts.params)) {
+  if (opts?.params) {
+    ArrayPrototypeForEach(ObjectKeys(opts.params), (origKey) => {
       const key = +origKey;
       if (NumberIsNaN(key) || key < 0 || key > kMaxBrotliParam ||
           (brotliInitParamsArray[key] | 0) !== -1) {
@@ -817,7 +818,7 @@ function Brotli(opts, mode) {
                                        'number', opts.params[origKey]);
       }
       brotliInitParamsArray[key] = value;
-    }
+    });
   }
 
   const handle = mode === BROTLI_DECODE ?