lib: use cache fs internals against path traversal

PR-URL: nodejs-private/node-private#516
Fixes: https://hackerone.com/bugs?subject=nodejs&report_id=2259914
Reviewed-By: Moshe Atlow <[email protected]>
CVE-ID: CVE-2024-21891

Author: RafaelGSS

lib/internal/process/pre_execution.js

@@ -11,6 +11,7 @@ const {
   DatePrototypeGetSeconds,
   NumberParseInt,
   ObjectDefineProperty,
+  ObjectFreeze,
   ObjectGetOwnPropertyDescriptor,
   SafeMap,
   String,
@@ -573,6 +574,8 @@ function initializePermission() {
     process.binding = function binding(_module) {
       throw new ERR_ACCESS_DENIED('process.binding');
     };
+    // Guarantee path module isn't monkey-patched to bypass permission model
+    ObjectFreeze(require('path'));
     process.emitWarning('Permission is an experimental feature',
                         'ExperimentalWarning');
     const { has, deny } = require('internal/process/permission');

test/fixtures/permission/fs-traversal.js

@@ -6,9 +6,12 @@ const assert = require('assert');
 const fs = require('fs');
 const path = require('path');
 
-// This should not affect how the permission model resolves paths.
 const { resolve } = path;
-path.resolve = (s) => s;
+// This should not affect how the permission model resolves paths.
+try {
+  path.resolve = (s) => s;
+  assert.fail('should not be called');
+} catch {}
 
 const blockedFolder = process.env.BLOCKEDFOLDER;
 const allowedFolder = process.env.ALLOWEDFOLDER;
@@ -96,6 +99,26 @@ const uint8ArrayTraversalPath = new TextEncoder().encode(traversalPath);
   }));
 }
 
+// Monkey-patching path module should also not allow path traversal.
+{
+  const fs = require('fs');
+  const path = require('path');
+
+  const cwd = Buffer.from('.');
+  try {
+    path.toNamespacedPath = (path) => { return traversalPath; };
+    assert.fail('should throw error when pacthing');
+  } catch { }
+
+  assert.throws(() => {
+    fs.readFile(cwd, common.mustNotCall());
+  }, common.expectsError({
+    code: 'ERR_ACCESS_DENIED',
+    permission: 'FileSystemRead',
+    resource: resolve(cwd.toString()),
+  }));
+}
+
 // Monkey-patching Buffer internals should also not allow path traversal.
 {
   const extraChars = '.'.repeat(40);