fs: fix realpath inode link caching

The `fs.realpath` / `fs.realpathSync` cache already seen symbolic links
using the inode number which may be longer that max supported
JS number (2**53) and will therefore be incorrectly handled by possibly
entering infinite loop of calling stat on the same node.

This PR changes those functions (where appropriate) to use
bigint for inode numbers.

Fixes: #33936

PR-URL: #33945
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>

Author: lundibundi

lib/fs.js

@@ -27,6 +27,7 @@
 const {
   Map,
   MathMax,
+  Number,
   NumberIsSafeInteger,
   ObjectCreate,
   ObjectDefineProperties,
@@ -174,7 +175,10 @@ const isFd = isUint32;
 function isFileType(stats, fileType) {
   // Use stats array directly to avoid creating an fs.Stats instance just for
   // our internal use.
-  return (stats[1/* mode */] & S_IFMT) === fileType;
+  let mode = stats[1];
+  if (typeof mode === 'bigint')
+    mode = Number(mode);
+  return (mode & S_IFMT) === fileType;
 }
 
 function access(path, mode, callback) {
@@ -1614,7 +1618,7 @@ function realpathSync(p, options) {
 
       const baseLong = pathModule.toNamespacedPath(base);
       const ctx = { path: base };
-      const stats = binding.lstat(baseLong, false, undefined, ctx);
+      const stats = binding.lstat(baseLong, true, undefined, ctx);
       handleErrorFromBinding(ctx);
 
       if (!isFileType(stats, S_IFLNK)) {
@@ -1747,7 +1751,7 @@ function realpath(p, options, callback) {
       return process.nextTick(LOOP);
     }
 
-    return fs.lstat(base, gotStat);
+    return fs.lstat(base, { bigint: true }, gotStat);
   }
 
   function gotStat(err, stats) {