Package: mailman
Version: 2.1.5-8

The Mailman package has a Debian patch ("21_newlist_help.dpatch") that
includes the following three lines:

+    gid = grp.getgrnam(mm_cfg.MAILMAN_GROUP)[2]
+    if os.getgid() != mm_cfg.MAILMAN_GROUP:
+        os.setgid(gid)

os.getgid() returns a number, but mm_cfg.MAILMAN_GROUP is a string.
These lines should be:

+    gid = grp.getgrnam(mm_cfg.MAILMAN_GROUP)[2]
+    if os.getgid() != gid:
+        os.setgid(gid)

Also, the os.setgid() call fails if the current user is not root or does
not have a primary group of mm_cfg.MAILMAN_GROUP. I guess this function
should therefore not be called if uid != 0? (Maybe the user should be
directed to the "sg" or "newgrp" commands to be more friendly, or maybe
these three lines don't need to be there at all?)

-- 
Matthew Newton <[EMAIL PROTECTED]>

UNIX and e-mail Systems Administrator, Network Support Section,
Computer Centre, University of Leicester,
Leicester LE1 7RH, United Kingdom


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to