fix: harden check for non-module in vm.modules linker

MikeRalphson · web-flow · commit bccbb288fa67 · 2023-04-16T17:00:35.000+01:00
diff --git a/lib/internal/vm/module.js b/lib/internal/vm/module.js
@@ -303,7 +303,7 @@ class SourceTextModule extends Module {
 
       const promises = this[kWrap].link(async (identifier, assert) => {
         const module = await linker(identifier, this, { assert });
-        if (module[kWrap] === undefined) {
+        if (module === undefined || module[kWrap] === undefined) {
           throw new ERR_VM_MODULE_NOT_MODULE();
         }
         if (module.context !== this.context) {

Original file line number	Diff line number	Diff line change
`@@ -303,7 +303,7 @@ class SourceTextModule extends Module {`
`303`	`303`
`304`	`304`	`const promises = this[kWrap].link(async (identifier, assert) => {`
`305`	`305`	`const module = await linker(identifier, this, { assert });`
`306`		`- if (module[kWrap] === undefined) {`
	`306`	`+ if (module === undefined \|\| module[kWrap] === undefined) {`
`307`	`307`	`throw new ERR_VM_MODULE_NOT_MODULE();`
`308`	`308`	`}`
`309`	`309`	`if (module.context !== this.context) {`