lib: replace eval with vm.runInThisContext

MylesBorins · BridgeAR · commit 99d693da5c1b · 2018-02-17T14:35:34.000+01:00
PR-URL: nodejs#18623 Refs: nodejs#18212 (review) Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
diff --git a/lib/internal/v8_prof_processor.js b/lib/internal/v8_prof_processor.js
@@ -1,4 +1,7 @@
-/* eslint-disable strict */
+'use strict';
+
+const vm = require('vm');
+
 const scriptFiles = [
   'internal/v8_prof_polyfill',
   'internal/deps/v8/tools/splaytree',
@@ -31,9 +34,9 @@ if (process.platform === 'darwin') {
   tickArguments.push('--windows');
 }
 tickArguments.push.apply(tickArguments, process.argv.slice(1));
-script = `(function() {
+script = `(function(require) {
   arguments = ${JSON.stringify(tickArguments)};
   function write (s) { process.stdout.write(s) }
   ${script}
-})()`;
-eval(script);
+})`;
+vm.runInThisContext(script)(require);
