clone 339691 -1
reassign -1 maildrop
retitle -1 maildrop: kills processes it shouldn't
severity 339691 normal
thanks

On Fri, Nov 18, 2005 at 10:44:26AM +0100, Vlada Macek wrote:
> [At 18.11.2005 08:23, Steve Langasek kindly sent the following quotation.]

> > On Fri, Nov 18, 2005 at 01:17:43AM +0100, Vlada Macek wrote:

> >> Vacation does not wait for its sendmail child to die in any way and
> >> exits!

> >> Therefore accurate vacation parent (such as maildrop MDA) wipes
> >> forked and executed sendmail before it could send any message...

> > Huh? Why is that a reasonable thing for the MDA to do?

> As I was reading its source, Maildrop just cleans up its own mess, which
> is rather creditable I think. It sets a process group at the beginning
> and does

>     (void)kill( -getprocgroup(), SIGHUP );

> in the cleanup() method. It's probably more than most of the parent
> processes do out there, but at least it reveals such megabugs like that
> of vacation.

No, I think this is a bogus assumption on the part of maildrop, not a
"megabug" in vacation.  I don't see any reason why maildrop should be either
setting a process group, or killing the group, under such circumstances.

I think it's still a bug in vacation to not check the exit value of the
commmand it spawns, but I believe the larger bug is maildrop's.

-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
[EMAIL PROTECTED]                                   http://www.debian.org/

Attachment: signature.asc
Description: Digital signature

Reply via email to