fs.access is not reliable in windows7 #7700

Activity

1. bnoordhuis commented on Jul 13, 2016

   

   bnoordhuis

   on Jul 13, 2016

   Member

   More actions

   From the documentation:

   fs.F_OK - File is visible to the calling process.

   You probably want:

   fs.W_OK - File can be written by the calling process.

   Good luck.
2. bnoordhuis

   closed this as completedon Jul 13, 2016
3. bnoordhuis

   added

   questionIssues asking questions about Node.js.Issues asking questions about Node.js.

   fsIssues and PRs related to file-system APIs and the fs module.Issues and PRs related to file-system APIs and the fs module.

   on Jul 13, 2016
4. leftstick commented on Jul 13, 2016

   

   leftstick

   on Jul 13, 2016

   Author

   More actions

   @bnoordhuis , when switch to fs.W_OK, still have nothing to err

   fs.access('C:/Windows/System32/drivers/etc/hosts', fs.W_OK, function(err) {
       console.log('err', err); //err null
   });

   but according to following screenshot, i don't have right to write this file

   
5. bnoordhuis commented on Jul 13, 2016

   

   bnoordhuis

   on Jul 13, 2016

   Member

   More actions

   Node isn't aware of ACLs, it just looks at the file attributes. If the file doesn't have the read-only attribute set, it's considered writable.
6. leftstick commented on Jul 13, 2016

   

   leftstick

   on Jul 13, 2016

   Author

   More actions

   @bnoordhuis So, you mean there is no way to check ACL on windows? A stable way?
7. leftstick commented on Jul 13, 2016

   

   leftstick

   on Jul 13, 2016

   Author

   More actions

   Cause when i trying to write something into the hosts file, error thrown:

   var fs = require('fs');
   
   fs.access('C:/Windows/System32/drivers/etc/hosts', fs.W_OK, function(err) {
       console.log('err', err); //err null
   });
   
   fs.writeFile('C:/Windows/System32/drivers/etc/hosts', 'test', function(err) {
       console.log(err);//error occurs
   });

   
8. bnoordhuis commented on Jul 13, 2016

   

   bnoordhuis

   on Jul 13, 2016

   Member

   More actions

   you mean there is no way to check ACL on windows?

   Not with node core. Maybe with a npm module.

   Cause when i trying to write something into the hosts file, error thrown

   That's how it should be. Look up 'TOCTOU'. Check-then-write is an anti-pattern.
9. leftstick commented on Jul 13, 2016

   

   leftstick

   on Jul 13, 2016

   Author

   More actions

   Thanks for your time