[Pkg-mailman-hackers] Bug#309339: marked as done (newlist has broken "if" statement in patch 21)

Debian Bug Tracking System owner at bugs.debian.org
Sun Jan 29 23:03:19 UTC 2006


Your message dated Sun, 29 Jan 2006 14:32:24 -0800
with message-id <E1F3L5s-0002z5-ML at spohr.debian.org>
and subject line Bug#309339: fixed in mailman 2.1.7-1
has caused the attached Bug report to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere.  Please contact me immediately.)

Debian bug tracking system administrator
(administrator, Debian Bugs database)

--------------------------------------
Received: (at submit) by bugs.debian.org; 16 May 2005 14:55:34 +0000
>From mcn4 at leicester.ac.uk Mon May 16 07:55:34 2005
Return-path: <mcn4 at leicester.ac.uk>
Received: from artemis.le.ac.uk [143.210.4.129] 
	by spohr.debian.org with esmtp (Exim 3.35 1 (Debian))
	id 1DXh0I-0006yR-00; Mon, 16 May 2005 07:55:34 -0700
Received: from [143.210.8.57] (helo=falcon.le.ac.uk)
	by artemis.le.ac.uk with esmtp (Exim 4.44)
	id 1DXgzm-0006HF-Qr
	for submit at bugs.debian.org; Mon, 16 May 2005 15:55:03 +0100
Received: from mcn4 by falcon.le.ac.uk with local (Exim 3.35 #1 (Debian))
	id 1DXgzm-0003uj-00
	for <submit at bugs.debian.org>; Mon, 16 May 2005 15:55:02 +0100
Date: Mon, 16 May 2005 15:55:01 +0100
To: submit at bugs.debian.org
Subject: newlist has broken "if" statement in patch 21
Message-ID: <20050516145501.GB30877 at ultra.le.ac.uk>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.3.28i
From: Matthew Newton <mcn4 at leicester.ac.uk>
X-UoL-Id: 969b77883c2a465e597889a9aa7ea69a at 1DXgzm-0006HF-Qr@artemis.le.ac.uk
Delivered-To: submit at bugs.debian.org
X-Spam-Checker-Version: SpamAssassin 2.60-bugs.debian.org_2005_01_02 
	(1.212-2003-09-23-exp) on spohr.debian.org
X-Spam-Status: No, hits=-7.0 required=4.0 tests=BAYES_01,HAS_PACKAGE 
	autolearn=no version=2.60-bugs.debian.org_2005_01_02
X-Spam-Level: 

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 <mcn4 at le.ac.uk>

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

---------------------------------------
Received: (at 309339-close) by bugs.debian.org; 29 Jan 2006 22:42:01 +0000
>From katie at ftp-master.debian.org Sun Jan 29 14:42:01 2006
Return-path: <katie at ftp-master.debian.org>
Received: from katie by spohr.debian.org with local (Exim 4.50)
	id 1F3L5s-0002z5-ML; Sun, 29 Jan 2006 14:32:24 -0800
From: Lionel Elie Mamane <lmamane at debian.org>
To: 309339-close at bugs.debian.org
X-Katie: $Revision: 1.65 $
Subject: Bug#309339: fixed in mailman 2.1.7-1
Message-Id: <E1F3L5s-0002z5-ML at spohr.debian.org>
Sender: Archive Administrator <katie at ftp-master.debian.org>
Date: Sun, 29 Jan 2006 14:32:24 -0800
X-Spam-Checker-Version: SpamAssassin 2.60-bugs.debian.org_2005_01_02 
	(1.212-2003-09-23-exp) on spohr.debian.org
X-Spam-Level: 
X-Spam-Status: No, hits=-6.0 required=4.0 tests=BAYES_00,HAS_BUG_NUMBER 
	autolearn=no version=2.60-bugs.debian.org_2005_01_02

Source: mailman
Source-Version: 2.1.7-1

We believe that the bug you reported is fixed in the latest version of
mailman, which is due to be installed in the Debian FTP archive:

mailman_2.1.7-1.diff.gz
  to pool/main/m/mailman/mailman_2.1.7-1.diff.gz
mailman_2.1.7-1.dsc
  to pool/main/m/mailman/mailman_2.1.7-1.dsc
mailman_2.1.7-1_sparc.deb
  to pool/main/m/mailman/mailman_2.1.7-1_sparc.deb
mailman_2.1.7.orig.tar.gz
  to pool/main/m/mailman/mailman_2.1.7.orig.tar.gz



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 309339 at bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Lionel Elie Mamane <lmamane at debian.org> (supplier of updated mailman package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster at debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: RIPEMD160

Format: 1.7
Date: Sun, 29 Jan 2006 21:57:45 +0100
Source: mailman
Binary: mailman
Architecture: source sparc
Version: 2.1.7-1
Distribution: unstable
Urgency: low
Maintainer: Mailman for Debian <pkg-mailman-hackers at lists.alioth.debian.org>
Changed-By: Lionel Elie Mamane <lmamane at debian.org>
Description: 
 mailman    - Powerful, web-based mailing list manager
Closes: 268117 309339 309870 341699 344907 345234 346180 346297 347607
Changes: 
 mailman (2.1.7-1) unstable; urgency=low
 .
   * New upstream release:
     - Update patches:
       71_date_overflows.dpatch
       76_fix-private-redirect.dpatch
       99_js_templates.dpatch
       53_disable_addons.dpatch
       68_translation_update_nl.dpatch
     - Drop patches merged upstream:
       60_french_archtocentry
       76_fix-private-redirect
     - Bugs fixed upstream:
       - Mailman loses header spam rules (closes: #309870)
   * Pull fixes from various bugs from upstream CVS
   * Ship documentation source (closes: #346180)
   * Document queue-is-not-empty upgrade condition better
     (closes: #268117)
   * Updated debconf template translations:
     - it (closes: #345234)
     - fr (closes: #346297)
     - sv (closes: #347607)
   * Tweak mailman/update_passwords debconf note to address ambiguity in
     English version discovered by French translation.
 .
 mailman (2.1.6-2) UNRELEASED; urgency=low
 .
   * Tweak 21_newlist_help.dpatch:
     The "automagically setgid to right group" part was slightly
     suboptimal. (closes: #309339)
   * Update German translation of debconf templates (closes: #344907)
   * Shut up DeprecationWarning with Python 2.4 in Scrubber.py (closes: #341699)
Files: 
 2cba239d1af2f72fcc20dc7d54b83450 738 mail optional mailman_2.1.7-1.dsc
 81ea139ecd24fbd2a85a9185a37df402 6736536 mail optional mailman_2.1.7.orig.tar.gz
 4bf14dd7ba35cad69fde4d4e72eabe9f 229456 mail optional mailman_2.1.7-1.diff.gz
 d68af159191ffb6d092dc38632233ea8 8008202 mail optional mailman_2.1.7-1_sparc.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (GNU/Linux)

iEYEAREDAAYFAkPdPxsACgkQscRzFz57S3P54gCgxMsUW9Vsa6evnbfavtc59eYl
v6kAoJFNKXnNLAvHsad7GIgZjzHb7LPP
=fubV
-----END PGP SIGNATURE-----




More information about the Pkg-mailman-hackers mailing list