nodejs
diff --git a/‎lib/fs.js‎
Lines changed: 10 additions & 3 deletions b/‎lib/fs.js‎
Lines changed: 10 additions & 3 deletions
diff --git a/‎test/sequential/test-fs-readdir-recursive.js‎
Lines changed: 2 additions & 0 deletions b/‎test/sequential/test-fs-readdir-recursive.js‎
Lines changed: 2 additions & 0 deletions
@@ -1430,14 +1430,21 @@ function readdirSyncRecursive(basePath, options) {
     );
     handleErrorFromBinding(ctx);
 
-    for (let i = 0; i < readdirResult.length; i++) {
-      if (withFileTypes) {
+    if (withFileTypes) {
+      // Calling `readdir` with `withFileTypes=true`, the result is an array of arrays.
+      // The first array is the names, and the second array is the types.
+      // They are guaranteed to be the same length; hence, setting `length` to the length
+      // of the first array within the result.
+      const length = readdirResult[0].length;
+      for (let i = 0; i < length; i++) {
         const dirent = getDirent(path, readdirResult[0][i], readdirResult[1][i]);
         ArrayPrototypePush(readdirResults, dirent);
         if (dirent.isDirectory()) {
           ArrayPrototypePush(pathsQueue, pathModule.join(dirent.path, dirent.name));
         }
-      } else {
+      }
+    } else {
+      for (let i = 0; i < readdirResult.length; i++) {
         const resultPath = pathModule.join(path, readdirResult[i]);
         const relativeResultPath = pathModule.relative(basePath, resultPath);
         const stat = binding.internalModuleStat(resultPath);
 
@@ -131,9 +131,11 @@ function getDirentPath(dirent) {
 }
 
 function assertDirents(dirents) {
+  assert.strictEqual(dirents.length, expected.length);
   dirents.sort((a, b) => (getDirentPath(a) < getDirentPath(b) ? -1 : 1));
   for (const [i, dirent] of dirents.entries()) {
     assert(dirent instanceof fs.Dirent);
+    assert.notStrictEqual(dirent.name, undefined);
     assert.strictEqual(getDirentPath(dirent), expected[i]);
   }
 }
Original file line number	Diff line number	Diff line change
@@ -131,9 +131,11 @@ function getDirentPath(dirent) {
131	131	}
132	132
133	133	function assertDirents(dirents) {
	134	+ assert.strictEqual(dirents.length, expected.length);
134	135	dirents.sort((a, b) => (getDirentPath(a) < getDirentPath(b) ? -1 : 1));
135	136	for (const [i, dirent] of dirents.entries()) {
136	137	assert(dirent instanceof fs.Dirent);
	138	+ assert.notStrictEqual(dirent.name, undefined);
137	139	assert.strictEqual(getDirentPath(dirent), expected[i]);
138	140	}
139	141	}