apparenty this fixed it. thanks!
I'm gonna let it run with this patch for a few days and
report back to the list if the problem still persists.
Regards,
-Oscar
On Thu, Apr 25, 2002 at 02:51:34PM +0200, Claudio Jeker wrote:
> On Thu, Apr 25, 2002 at 08:47:45AM +0200, Claudio Jeker wrote:
> > I will try to set up a test machine to trigger the problem (there was
> > already someone complaining about strange things in qmail-lspawn).
> >
> OK, I think I have found the bug.
> The probelm is the "new" log interface. If in one read a log entry and a
> delivery status notification was delivered the status notification was
> dropped and therefor qmail-send did not close the job.
> The following patch should fix the problem.
>
> --
> :wq Claudio
> Index: qmail-send.c
> ===================================================================
> RCS file: /home/cvs-qmail-ldap/CVS/qmail-ldap/qmail-send.c,v
> retrieving revision 1.12
> diff -u -r1.12 qmail-send.c
> --- qmail-send.c 2002/04/22 10:03:16 1.12
> +++ qmail-send.c 2002/04/25 12:27:31
> @@ -975,13 +975,15 @@
> logsafe(dline[c].s + 3);
> log1("\n");
> dline[c].len = 0;
> - return;
> + break;
> default:
> log3("delivery ",strnum3,": report mangled, will defer\n");
> }
> - job_close(d[c][delnum].j);
> - d[c][delnum].used = 0; --concurrencyused[c];
> - del_status();
> + if (dline[c].s[2] != 'L') {
> + job_close(d[c][delnum].j);
> + d[c][delnum].used = 0; --concurrencyused[c];
> + del_status();
> + }
> }
> dline[c].len = 0;
> }
--
pgp fingerprint: BC64 2E7A CAEF 39E1 9544 80CA F7D5 784D FB46 16C1
