unpack: don't call process.getgid() unnecessarily

Author: isaacs

lib/unpack.js

@@ -234,7 +234,7 @@ class Unpack extends Parser {
           fs.utimes(entry.absolute, entry.atime || new Date(), entry.mtime, cb))
       if (this[DOCHOWN](entry))
         queue.push(cb =>
-          fs.chown(entry.absolute, entry.uid, entry.gid || process.getgid(), cb))
+          fs.chown(entry.absolute, entry.uid, entry.gid || this.processGid, cb))
 
       processQueue()
     })
@@ -373,7 +373,7 @@ class UnpackSync extends Unpack {
         }
         if (this[DOCHOWN](entry)) {
           try {
-            fs.fchownSync(fd, entry.uid, entry.gid || process.getgid())
+            fs.fchownSync(fd, entry.uid, entry.gid || this.processGid)
           } catch (er) {}
         }
         try { fs.closeSync(fd) } catch (er) { this[ONERROR](er, entry) }
@@ -393,7 +393,7 @@ class UnpackSync extends Unpack {
     }
     if (this[DOCHOWN](entry)) {
       try {
-        fs.chownSync(entry.absolute, entry.uid, entry.gid || process.getgid())
+        fs.chownSync(entry.absolute, entry.uid, entry.gid || this.processGid)
       } catch (er) {}
     }
     entry.resume()