process: stub unsupported worker methods

Some process methods are not supported in workers. This commit
adds stubs that throw more informative errors.

PR-URL: https://github.com/nodejs/node/pull/25587
Fixes: https://github.com/nodejs/node/issues/25448
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

2 files changed, 38 insertions, 2 deletions

diff --git a/lib/internal/bootstrap/node.js b/lib/internal/bootstrap/node.js
index 358c4cb8c7..071b68903d 100644
--- a/lib/internal/bootstrap/node.js
+++ b/lib/internal/bootstrap/node.js
@@ -84,6 +84,8 @@ if (isMainThread) {
 } else {
   const wrapped = workerThreadSetup.wrapProcessMethods(rawMethods);
 
+  process.abort = workerThreadSetup.unavailable('process.abort()');
+  process.chdir = workerThreadSetup.unavailable('process.chdir()');
   process.umask = wrapped.umask;
 }
 
@@ -148,6 +150,14 @@ if (credentials.implementsPosixCredentials) {
     process.seteuid = wrapped.seteuid;
     process.setgid = wrapped.setgid;
     process.setuid = wrapped.setuid;
+  } else {
+    process.initgroups =
+      workerThreadSetup.unavailable('process.initgroups()');
+    process.setgroups = workerThreadSetup.unavailable('process.setgroups()');
+    process.setegid = workerThreadSetup.unavailable('process.setegid()');
+    process.seteuid = workerThreadSetup.unavailable('process.seteuid()');
+    process.setgid = workerThreadSetup.unavailable('process.setgid()');
+    process.setuid = workerThreadSetup.unavailable('process.setuid()');
   }
 }
 
@@ -174,8 +184,24 @@ if (config.hasInspector) {
 // This attaches some internal event listeners and creates:
 // process.send(), process.channel, process.connected,
 // process.disconnect()
-if (isMainThread && process.env.NODE_CHANNEL_FD) {
-  mainThreadSetup.setupChildProcessIpcChannel();
+if (process.env.NODE_CHANNEL_FD) {
+  if (ownsProcessState) {
+    mainThreadSetup.setupChildProcessIpcChannel();
+  } else {
+    Object.defineProperty(process, 'channel', {
+      enumerable: false,
+      get: workerThreadSetup.unavailable('process.channel')
+    });
+
+    Object.defineProperty(process, 'connected', {
+      enumerable: false,
+      get: workerThreadSetup.unavailable('process.connected')
+    });
+
+    process.send = workerThreadSetup.unavailable('process.send()');
+    process.disconnect =
+      workerThreadSetup.unavailable('process.disconnect()');
+  }
 }
 
 const browserGlobals = !process._noBrowserGlobals;
diff --git a/lib/internal/process/worker_thread_only.js b/lib/internal/process/worker_thread_only.js
index f05d5e932b..2cc52cbf01 100644
--- a/lib/internal/process/worker_thread_only.js
+++ b/lib/internal/process/worker_thread_only.js
@@ -45,7 +45,17 @@ function wrapProcessMethods(binding) {
   return { umask };
 }
 
+function unavailable(name) {
+  function unavailableInWorker() {
+    throw new ERR_WORKER_UNSUPPORTED_OPERATION(name);
+  }
+
+  unavailableInWorker.disabled = true;
+  return unavailableInWorker;
+}
+
 module.exports = {
   initializeWorkerStdio,
+  unavailable,
   wrapProcessMethods
 };