fs: fix glob TypeError on restricted dirs

When a directory cannot be read due to permission issues, the async
version of fs.glob() returns null from readdir(), while the sync
version returns an empty array. This causes a TypeError when trying
to access the 'length' property of null.
PR-URL: #58674Fixes: #58670Fixes: #58276
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ethan-Arrowood <ethan@arrowood.dev>
Reviewed-By: Juan José <soyjuanarbol@gmail.com>

Author: Sylphy-0xd3ac

lib/internal/fs/glob.js

@@ -143,7 +143,7 @@ class Cache {
 if(cached){
 returncached;
 }
-constpromise=PromisePrototypeThen(readdir(path,{__proto__: null,withFileTypes: true}),null,()=>null);
+constpromise=PromisePrototypeThen(readdir(path,{__proto__: null,withFileTypes: true}),null,()=>[]);
 this.#readdirCache.set(path,promise);
 returnpromise;
 }

test/parallel/test-fs-glob.mjs

@@ -2,7 +2,7 @@ import * as common from '../common/index.mjs';
 importtmpdirfrom'../common/tmpdir.js';
 import{resolve,dirname,sep,relative,join,isAbsolute}from'node:path';
 import{mkdir,writeFile,symlink,globasasyncGlob}from'node:fs/promises';
-import{glob,globSync,Dirent}from'node:fs';
+import{glob,globSync,Dirent,chmodSync}from'node:fs';
 import{test,describe}from'node:test';
 import{pathToFileURL}from'node:url';
 import{promisify}from'node:util';
@@ -518,3 +518,24 @@ describe('fsPromises glob - exclude', function() {
 });
 }
 });
+
+describe('glob - with restricted directory',function(){
+test('*',async()=>{
+constrestrictedDir=tmpdir.resolve('restricted');
+awaitmkdir(restrictedDir,{recursive: true});
+chmodSync(restrictedDir,0o000);
+try{
+constresults=[];
+forawait(constmatchofasyncGlob('*',{cwd: restrictedDir})){
+results.push(match);
+}
+assert.ok(true,'glob completed without throwing on readdir error');
+}finally{
+try{
+chmodSync(restrictedDir,0o755);
+}catch{
+// ignore
+}
+}
+});
+});