[Pkg-mailman-hackers] Bug#309339: Mailman Debian bug: newlist has broken "if" statement in patch 21

[Lionel Elie Mamane]

tags 309339 +pending
thanks

Thank you for your bug report. You wrote:

> os.getgid() returns a number, but mm_cfg.MAILMAN_GROUP is a string.

Yup. Corrected in svn repo; will be part of next upload (2.1.6-2 or
2.1.7-1).

> 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?

newlist shouldn't be called at all if you are neither root nor the
right user/group combination.


-- 
Lionel