Merge pull request #326 from tschaub/uid-cleanup

chore: use 0 internally for uid/gid on Windows

Author: tschaub

lib/item.js

@@ -19,13 +19,13 @@ const permissions = {
 };
 
 function getUid() {
-  // force NaN on windows.
-  return process.getuid ? process.getuid() : NaN;
+  // force 0 on windows.
+  return process.getuid ? process.getuid() : 0;
 }
 
 function getGid() {
-  // force NaN on windows.
-  return process.getgid ? process.getgid() : NaN;
+  // force 0 on windows.
+  return process.getgid ? process.getgid() : 0;
 }
 
 /**
@@ -108,10 +108,9 @@ Item.prototype.canRead = function() {
   const uid = getUid();
   const gid = getGid();
   let can = false;
-  if (uid === 0) {
+  if (process.getuid && uid === 0) {
     can = true;
-  } else if (uid === this._uid || uid !== uid) {
-    // (uid !== uid) means uid is NaN, only for windows
+  } else if (uid === this._uid) {
     can = (permissions.USER_READ & this._mode) === permissions.USER_READ;
   } else if (gid === this._gid) {
     can = (permissions.GROUP_READ & this._mode) === permissions.GROUP_READ;
@@ -129,10 +128,9 @@ Item.prototype.canWrite = function() {
   const uid = getUid();
   const gid = getGid();
   let can = false;
-  if (uid === 0) {
+  if (process.getuid && uid === 0) {
     can = true;
-  } else if (uid === this._uid || uid !== uid) {
-    // (uid !== uid) means uid is NaN, only for windows
+  } else if (uid === this._uid) {
     can = (permissions.USER_WRITE & this._mode) === permissions.USER_WRITE;
   } else if (gid === this._gid) {
     can = (permissions.GROUP_WRITE & this._mode) === permissions.GROUP_WRITE;
@@ -150,10 +148,9 @@ Item.prototype.canExecute = function() {
   const uid = getUid();
   const gid = getGid();
   let can = false;
-  if (uid === 0) {
+  if (process.getuid && uid === 0) {
     can = true;
-  } else if (uid === this._uid || isNaN(uid)) {
-    // NaN occurs on windows
+  } else if (uid === this._uid) {
     can = (permissions.USER_EXEC & this._mode) === permissions.USER_EXEC;
   } else if (gid === this._gid) {
     can = (permissions.GROUP_EXEC & this._mode) === permissions.GROUP_EXEC;
@@ -289,19 +286,8 @@ Item.prototype.getStats = function(bigint) {
   stats[0] = convert(8675309); // dev
   // [1] is mode
   stats[2] = convert(this.links); // nlink
-
-  const uid = this.getUid();
-  if (!isNaN(uid)) {
-    // uid is NaN on windows
-    stats[3] = convert(uid); // uid
-  }
-
-  const gid = this.getGid();
-  if (!isNaN(gid)) {
-    // gid is NaN on windows
-    stats[4] = convert(gid); // gid
-  }
-
+  stats[3] = convert(this.getUid()); // uid
+  stats[4] = convert(this.getGid()); // gid
   stats[5] = convert(0); // rdev
   stats[6] = convert(4096); // blksize
   stats[7] = convert(this._id); // ino