main(): Write the master-qrunner.pid file with 660 permissions.

1 files changed, 9 insertions, 4 deletions

diff --git a/bin/mailmanctl b/bin/mailmanctl
index 673deb6f1..8d9b90750 100644
--- a/bin/mailmanctl
+++ b/bin/mailmanctl
@@ -365,10 +365,15 @@ def main():
             return
         # child
         lock._take_possession()
-        # First, save our pid in a file for "mailmanctl stop" rendezvous
-        fp = open(mm_cfg.PIDFILE, 'w')
-        print >> fp, os.getpid()
-        fp.close()
+        # First, save our pid in a file for "mailmanctl stop" rendezvous.  We
+        # want the perms on the .pid file to be rw-rw----
+        omask = os.umask(6)
+        try:
+            fp = open(mm_cfg.PIDFILE, 'w')
+            print >> fp, os.getpid()
+            fp.close()
+        finally:
+            os.umask(omask)
         # Create a new session and become the session leader, but since we
         # won't be opening any terminal devices, don't do the ultra-paranoid
         # suggestion of doing a second fork after the setsid() call.