wasi: use missing validator

The `wasi` lib module's `initialize()` method is missing a validator. PR-URL: #39070 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
nodejs · Jul 11, 2021 · fe1c81f · fe1c81f
1 parent 21e8720
commit fe1c81f
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 15 deletions.
diff --git a/lib/internal/validators.js b/lib/internal/validators.js
@@ -229,6 +229,11 @@ const validateFunction = hideStackFrames((value, name) => {
     throw new ERR_INVALID_ARG_TYPE(name, 'Function', value);
 });
 
+const validateUndefined = hideStackFrames((value, name) => {
+  if (value !== undefined)
+    throw new ERR_INVALID_ARG_TYPE(name, 'undefined', value);
+});
+
 module.exports = {
   isInt32,
   isUint32,
@@ -247,6 +252,7 @@ module.exports = {
   validateSignalName,
   validateString,
   validateUint32,
+  validateUndefined,
   validateCallback,
   validateAbortSignal,
 };
diff --git a/lib/wasi.js b/lib/wasi.js
@@ -21,6 +21,7 @@ const {
   validateFunction,
   validateInt32,
   validateObject,
+  validateUndefined,
 } = require('internal/validators');
 const { WASI: _WASI } = internalBinding('wasi');
 const kExitCode = Symbol('kExitCode');
@@ -120,10 +121,7 @@ class WASI {
     const { _start, _initialize } = this[kInstance].exports;
 
     validateFunction(_start, 'instance.exports._start');
-    if (_initialize !== undefined) {
-      throw new ERR_INVALID_ARG_TYPE(
-        'instance.exports._initialize', 'undefined', _initialize);
-    }
+    validateUndefined(_initialize, 'instance.exports._initialize');
 
     try {
       _start();
@@ -147,16 +145,9 @@ class WASI {
 
     const { _start, _initialize } = this[kInstance].exports;
 
-    if (typeof _initialize !== 'function' && _initialize !== undefined) {
-      throw new ERR_INVALID_ARG_TYPE(
-        'instance.exports._initialize', 'function', _initialize);
-    }
-    if (_start !== undefined) {
-      throw new ERR_INVALID_ARG_TYPE(
-        'instance.exports._start', 'undefined', _initialize);
-    }
-
+    validateUndefined(_start, 'instance.exports._start');
     if (_initialize !== undefined) {
+      validateFunction(_initialize, 'instance.exports._initialize');
       _initialize();
     }
   }

diff --git a/test/wasi/test-wasi-initialize-validation.js b/test/wasi/test-wasi-initialize-validation.js
@@ -78,7 +78,8 @@ const bufferSource = fixtures.readSync('simple.wasm');
       () => { wasi.initialize(instance); },
       {
         code: 'ERR_INVALID_ARG_TYPE',
-        message: /"instance\.exports\._start" property must be undefined/
+        message: 'The "instance.exports._start" property must be' +
+          ' undefined. Received function _start',
       }
     );
   }

diff --git a/test/wasi/test-wasi-start-validation.js b/test/wasi/test-wasi-start-validation.js
@@ -78,7 +78,8 @@ const bufferSource = fixtures.readSync('simple.wasm');
       () => { wasi.start(instance); },
       {
         code: 'ERR_INVALID_ARG_TYPE',
-        message: /"instance\.exports\._initialize" property must be undefined/
+        message: 'The "instance.exports._initialize" property must be' +
+          ' undefined. Received function _initialize',
       }
     );
   }