From: Joerg Jaspert <joerg@debian.org>
Date: Sun, 16 Sep 2012 16:37:30 +0000 (+0200)
Subject: Merge remote-tracking branch 'ansgar/fix-unpriv' into merge
X-Git-Url: https://git.decadent.org.uk/gitweb/?a=commitdiff_plain;h=45790364d23e259a8ce10438d7243ada87536a3b;hp=e571c7b55a0fb16fa23c27c13346db30edf2f444;p=dak.git

Merge remote-tracking branch 'ansgar/fix-unpriv' into merge

* ansgar/fix-unpriv:
  daklib/utils.py: lookup gid for group.
  daklib/checks.py: typo: unpribgroup → unprivgroup

Signed-off-by: Joerg Jaspert <joerg@debian.org>
---

diff --git a/daklib/checks.py b/daklib/checks.py
index 25cbb8f0..8111ef75 100644
--- a/daklib/checks.py
+++ b/daklib/checks.py
@@ -604,7 +604,7 @@ class LintianCheck(Check):
 
         changespath = os.path.join(upload.directory, changes.filename)
         try:
-            if cnf.unpribgroup:
+            if cnf.unprivgroup:
                 cmd = "sudo -H -u {0} -- /usr/bin/lintian --show-overrides --tags-from-file {1} {2}".format(cnf.unprivgroup, temp_filename, changespath)
             else:
                 cmd = "/usr/bin/lintian --show-overrides --tags-from-file {0} {1}".format(temp_filename, changespath)
diff --git a/daklib/utils.py b/daklib/utils.py
index 8a160356..03cd5894 100755
--- a/daklib/utils.py
+++ b/daklib/utils.py
@@ -27,6 +27,7 @@ import datetime
 import email.Header
 import os
 import pwd
+import grp
 import select
 import socket
 import shutil
@@ -1468,7 +1469,8 @@ def temp_filename(directory=None, prefix="dak", suffix="", mode=None, group=None
     if mode:
         os.chmod(tfname, mode)
     if group:
-        os.chown(tfname, -1, group)
+        gid = grp.getgrnam(group).gr_gid
+        os.chown(tfname, -1, gid)
     return (tfd, tfname)
 
 ################################################################################