Hi,

there are some more uninitialized values when running with the ->id
stuff. This one happens also in -forkserver, and can only be fixed by
resetting the _connection_ after queueing the mail. See attached diff.
Is this what we want?

        Hanno
Index: qpsmtpd-prefork
===================================================================
--- qpsmtpd-prefork	(revision 804)
+++ qpsmtpd-prefork	(working copy)
@@ -555,6 +555,7 @@
     my %children;
     shmem_opt(\%children, undef, $$, $iaddr);
 
+    $qpsmtpd->reset_connection;
     my ($rc, @msg) =
       $qpsmtpd->run_hooks(
                           "pre-connection",
Index: lib/Qpsmtpd/SMTP.pm
===================================================================
--- lib/Qpsmtpd/SMTP.pm	(revision 804)
+++ lib/Qpsmtpd/SMTP.pm	(working copy)
@@ -802,6 +802,7 @@
   my ($self, $rc, $msg, $args) = @_;
   
   # reset transaction if we queued the mail
+  $self->reset_connection;
   $self->reset_transaction;
   
   if ($rc == DONE) {

Reply via email to