* apt-pkg/deb/dpkgpm.cc:

  - set permissions of term.log to root.adm and 644 (LP: #404724)

diff --git a/apt-pkg/deb/dpkgpm.cc b/apt-pkg/deb/dpkgpm.cc
index b37980b7ecbbdae96d210bb6ffca6a700c94d23b..ca544e778f7e5fbedbdc8a5cef6c04cab1771e28 100644 (file)
--- a/apt-pkg/deb/dpkgpm.cc
+++ b/apt-pkg/deb/dpkgpm.cc
@@ -32,6 +32,8 @@
 #include <algorithm>
 #include <sstream>
 #include <map>
+#include <pwd.h>
+#include <grp.h>
 
 #include <termios.h>
 #include <unistd.h>
@@ -667,7 +669,13 @@ bool pkgDPkgPM::OpenLog()
         return _error->WarningE("OpenLog", _("Could not open file '%s'"), logfile_name.c_str());
       setvbuf(term_out, NULL, _IONBF, 0);
       SetCloseExec(fileno(term_out), true);
-      chmod(logfile_name.c_str(), 0600);
+      struct passwd *pw;
+      struct group *gr;
+      pw = getpwnam("root");
+      gr = getgrnam("adm");
+      if (pw != NULL && gr != NULL)
+         chown(logfile_name.c_str(), pw->pw_uid, gr->gr_gid);
+      chmod(logfile_name.c_str(), 0644);
       fprintf(term_out, "\nLog started: %s\n", timestr);
    }
 

diff --git a/debian/changelog b/debian/changelog
index 2c296b597381c428af3e06eca8f55166e343cc26..16c0990d9d81825e35e11463d1fa226822c3cca4 100644 (file)
--- a/debian/changelog
+++ b/debian/changelog
@@ -86,6 +86,10 @@ apt (0.8.15) UNRELEASED; urgency=low
   * apt-pkg/deb/debindexfile.cc:
     - remove some no longer valid checks for "TranslationsAvailable()"
 
+  [ Kenneth Solbø Andersen ]
+  * apt-pkg/deb/dpkgpm.cc:
+    - set permissions of term.log to root.adm and 644 (LP: #404724)
+
  -- Michael Vogt <mvo@debian.org>  Mon, 16 May 2011 14:57:52 +0200
 
 apt (0.8.14.1) unstable; urgency=low