inn-2.2.1/innd/inndstart.c:92 makes wrong assumption

[Russ Allbery]

Benedict Lofstedt <benedict at daimi.au.dk> writes:

> The test

>     } else if (pwd->pw_gid != Sb.st_gid) {
> 	syslog(L_FATAL, "inndstart %s must have group %s", innconf->pathrun, NEWSGRP);
> 	exit(1);

> does not test that innconf->pathrun has group ownership NEWSGRP.

> What it does test is, that the directory is owned by the default group
> of user NEWSUSER, which may or may not be the group NEWSGRP.

Thanks for the report.  I've rewritten inndstart almost completely for 2.3
and fixed this bug in the process; I don't think it's worth going back and
adding in the getgrnam() stuff into 2.2.1 to fix this, since NEWSGRP
really should be NEWSUSER's default group, but if someone else wants to do
the work that's cool too.

-- 
Russ Allbery (rra at stanford.edu)         <URL:http://www.eyrie.org/~eagle/>