src: fix process.getuid() return value

And process.getgid() too. Commit ed80638 changed fs.chown() and fs.fchown() to only accept unsigned integers. Make process.getuid() and process.getgid() follow suit. This commit should unbreak npm on OS X - it's hitting the new 'uid must be an unsigned int' check when installing as e.g. user 'nobody' (which has an UID of -2 in /etc/passwd or 4294967294 when cast to an uid_t.) Fixes #5904.
nodejs · Jul 25, 2013 · 015ec05 · 015ec05
1 parent 0de5b83
commit 015ec05
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/node.cc b/src/node.cc
@@ -1528,15 +1528,15 @@ static gid_t gid_by_name(Handle<Value> value) {
 
 static Handle<Value> GetUid(const Arguments& args) {
   HandleScope scope;
-  int uid = getuid();
-  return scope.Close(Integer::New(uid));
+  uid_t uid = getuid();
+  return scope.Close(Integer::NewFromUnsigned(uid));
 }
 
 
 static Handle<Value> GetGid(const Arguments& args) {
   HandleScope scope;
-  int gid = getgid();
-  return scope.Close(Integer::New(gid));
+  gid_t gid = getgid();
+  return scope.Close(Integer::NewFromUnsigned(gid));
 }