process: fix default env for process.execve

The `env` parameter for `process.execve` is documented to default
to `process.env`.

PR-URL: #60029
Refs: nodejs/build#4156
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>

Author: richardlau

lib/internal/process/per_thread.js

@@ -279,7 +279,7 @@ function wrapProcessMethods(binding) {
     return true;
   }
 
-  function execve(execPath, args = [], env) {
+  function execve(execPath, args = [], env = process.env) {
     emitExperimentalWarning('process.execve');
 
     const { isMainThread } = require('internal/worker');
@@ -301,22 +301,20 @@ function wrapProcessMethods(binding) {
     }
 
     const envArray = [];
-    if (env !== undefined) {
-      validateObject(env, 'env');
-
-      for (const { 0: key, 1: value } of ObjectEntries(env)) {
-        if (
-          typeof key !== 'string' ||
-          typeof value !== 'string' ||
-          StringPrototypeIncludes(key, '\u0000') ||
-          StringPrototypeIncludes(value, '\u0000')
-        ) {
-          throw new ERR_INVALID_ARG_VALUE(
-            'env', env, 'must be an object with string keys and values without null bytes',
-          );
-        } else {
-          ArrayPrototypePush(envArray, `${key}=${value}`);
-        }
+    validateObject(env, 'env');
+
+    for (const { 0: key, 1: value } of ObjectEntries(env)) {
+      if (
+        typeof key !== 'string' ||
+        typeof value !== 'string' ||
+        StringPrototypeIncludes(key, '\u0000') ||
+        StringPrototypeIncludes(value, '\u0000')
+      ) {
+        throw new ERR_INVALID_ARG_VALUE(
+          'env', env, 'must be an object with string keys and values without null bytes',
+        );
+      } else {
+        ArrayPrototypePush(envArray, `${key}=${value}`);
       }
     }