fs: revert "change statSync to accessSync..."

Trott · Michael Scovetta · commit d9ae28b18f9a · 2016-04-02T06:07:15.000Z
This reverts commit 809bf5e ("change statSync to accessSync in realpathSync"). It was causing tests to fail on Windows CI. Ref: nodejs#4575 PR-URL: nodejs#4679 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
diff --git a/lib/fs.js b/lib/fs.js
@@ -1550,7 +1550,7 @@ fs.realpathSync = function realpathSync(p, cache) {
         }
       }
       if (linkTarget === null) {
-        fs.accessSync(base, fs.F_OK);  // Throws ELOOP on cyclic links.
+        fs.statSync(base);
         linkTarget = fs.readlinkSync(base);
       }
       resolvedLink = pathModule.resolve(previous, linkTarget);

Original file line number	Diff line number	Diff line change
`@@ -1550,7 +1550,7 @@ fs.realpathSync = function realpathSync(p, cache) {`
`1550`	`1550`	`}`
`1551`	`1551`	`}`
`1552`	`1552`	`if (linkTarget === null) {`
`1553`		`- fs.accessSync(base, fs.F_OK); // Throws ELOOP on cyclic links.`
	`1553`	`+ fs.statSync(base);`
`1554`	`1554`	`linkTarget = fs.readlinkSync(base);`
`1555`	`1555`	`}`
`1556`	`1556`	`resolvedLink = pathModule.resolve(previous, linkTarget);`