perf_hooks: fix loop delay resolution validation

Fixes: nodejs#38160
aduh95 · Apr 9, 2021 · bc262f7 · bc262f7
1 parent 7df0fc5
commit bc262f7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/internal/perf/event_loop_delay.js b/lib/internal/perf/event_loop_delay.js
@@ -9,7 +9,7 @@ const {
 } = internalBinding('performance');
 
 const {
-  validateInteger,
+  validateInt32,
   validateObject,
 } = require('internal/validators');
 
@@ -47,7 +47,7 @@ function monitorEventLoopDelay(options = {}) {
   validateObject(options, 'options');
 
   const { resolution = 10 } = options;
-  validateInteger(resolution, 'options.resolution', 1);
+  validateInt32(resolution, 'options.resolution', 1);
 
   return new ELDHistogram(new _ELDHistogram(resolution));
 }

diff --git a/test/sequential/test-performance-eventloopdelay.js b/test/sequential/test-performance-eventloopdelay.js
@@ -39,7 +39,7 @@ const { sleep } = require('internal/util');
     );
   });
 
-  [-1, 0, Infinity].forEach((i) => {
+  [-1, 0, 2**31, Infinity].forEach((i) => {
     assert.throws(
       () => monitorEventLoopDelay({ resolution: i }),
       {