src,permission: fix permission.has on empty param

nodejs · nodejs-github-bot · Nov 16, 2025 · Nov 10, 2025 · Nov 11, 2025 · Nov 10, 2025
commit 1535bdc950e81ff9af23217bfca94cb7dd409195
diff --git a/src/permission/fs_permission.cc b/src/permission/fs_permission.cc
@@ -169,12 +169,18 @@ bool FSPermission::is_granted(Environment* env,
     case PermissionScope::kFileSystem:
       return allow_all_in_ && allow_all_out_;
     case PermissionScope::kFileSystemRead:
+      if (param.empty()) {
+        return allow_all_in_;
+      }
       return !deny_all_in_ &&
-             ((param.empty() && allow_all_in_) || allow_all_in_ ||
+             (allow_all_in_ ||
               is_tree_granted(env, &granted_in_fs_, param));
     case PermissionScope::kFileSystemWrite:
+      if (param.empty()) {
+        return allow_all_out_;
+      }
       return !deny_all_out_ &&
-             ((param.empty() && allow_all_out_) || allow_all_out_ ||
+             (allow_all_out_ ||
               is_tree_granted(env, &granted_out_fs_, param));
     default:
       return false;

diff --git a/test/parallel/test-permission-has.js b/test/parallel/test-permission-has.js
@@ -1,4 +1,4 @@
-// Flags: --permission --allow-fs-read=*
+// Flags: --permission --allow-fs-read=* --allow-fs-write=.
 'use strict';
 
 const common = require('../common');