nodejs
diff --git a/Collapse file
‎src/node_contextify.cc‎
Copy file name to clipboardExpand all lines: src/node_contextify.cc
+3-1Lines changed: 3 additions & 1 deletion b/Collapse file
‎src/node_contextify.cc‎
Copy file name to clipboardExpand all lines: src/node_contextify.cc
+3-1Lines changed: 3 additions & 1 deletion
diff --git a/Collapse file
‎test/parallel/test-vm-global-symbol.js‎
Copy file name to clipboard
+15Lines changed: 15 additions & 0 deletions b/Collapse file
‎test/parallel/test-vm-global-symbol.js‎
Copy file name to clipboard
+15Lines changed: 15 additions & 0 deletions
@@ -528,7 +528,9 @@ void ContextifyContext::PropertySetterCallback(
     return;
 
   USE(ctx->sandbox()->Set(context, property, value));
-  args.GetReturnValue().Set(value);
+  if (is_contextual_store || is_function) {
+    args.GetReturnValue().Set(value);
+  }
 }
 
 // static
 
@@ -0,0 +1,15 @@
+'use strict';
+require('../common');
+const assert = require('assert');
+const vm = require('vm');
+
+const global = vm.runInContext('this', vm.createContext());
+const totoSymbol = Symbol.for('toto');
+Object.defineProperty(global, totoSymbol, {
+  enumerable: true,
+  writable: true,
+  value: 4,
+  configurable: true,
+});
+assert.strictEqual(global[totoSymbol], 4);
+assert.ok(Object.getOwnPropertySymbols(global).includes(totoSymbol));