src: fix crash with SyntheticModule#setExport

Use the new non-deprecated V8 API for that.
nodejs · Oct 25, 2019 · 48c74ef · 48c74ef
1 parent f13686b
commit 48c74ef
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/src/module_wrap.cc b/src/module_wrap.cc
@@ -1500,7 +1500,7 @@ void ModuleWrap::SetSyntheticExport(
   Local<Value> export_value = args[1];
 
   Local<Module> module = obj->module_.Get(isolate);
-  module->SetSyntheticModuleExport(export_name, export_value);
+  USE(module->SetSyntheticModuleExport(isolate, export_name, export_value));
 }
 
 void ModuleWrap::Initialize(Local<Object> target,

diff --git a/test/parallel/test-vm-module-synthetic.js b/test/parallel/test-vm-module-synthetic.js
@@ -36,8 +36,6 @@ const assert = require('assert');
     });
   }
 
-  // https://bugs.chromium.org/p/v8/issues/detail?id=9828
-  /*
   {
     const s = new SyntheticModule([], () => {});
     await s.link(() => {});
@@ -47,7 +45,6 @@ const assert = require('assert');
       name: 'ReferenceError',
     });
   }
-  */
 
   {
     const s = new SyntheticModule([], () => {});