On 2005-11-14 22:31:16 +0100, Peter J. Holzer wrote:
> On 2005-11-12 19:23:38 +0100, Hanno Hecker wrote:
> > diff -urN qpsmtpd-0.31-dev/qpsmtpd-forkserver
> > qpsmtpd-0.31-dev+pre+post/qpsmtpd-forkserver
> > --- qpsmtpd-0.31-dev/qpsmtpd-forkserver 2005-11-01 13:34:53.000000000
> > +0100
> > +++ qpsmtpd-0.31-dev+pre+post/qpsmtpd-forkserver 2005-11-12
> > 18:13:49.000000000 +0100
> > @@ -209,6 +209,7 @@
> > next;
> > }
> > }
> > + $qpsmtpd->run_hooks("pre-connection");
> > my $pid = safe_fork();
> > if ($pid) {
> > # parent
>
> That looks too simple to me. I've submitted a patch for this (twice ;-))
> some time ago, and I had to move around some code lib/Qpsmtpd/* to get
> useful behaviour. Of course quite a lot has changed in 0.31, so that may
> not be necessary anymore, but I doubt it.
I just downloaded rc3.
Looks like the main problem (plugins were loaded in the child process
again) has been fixed in the meantime.
So the only advantage my patch still has over yours is that with mine
the pre-connection hook can return DENYSOFT or DENY to prevent the
forking of a child.
hp
--
_ | Peter J. Holzer | Ich sehe nun ein, dass Computer wenig
|_|_) | Sysadmin WSR | geeignet sind, um sich was zu merken.
| | | [EMAIL PROTECTED] |
__/ | http://www.hjp.at/ | -- Holger Lembke in dan-am
pgp9QfoWUizHL.pgp
Description: PGP signature
