From: Philip Hazel Date: Thu, 15 Dec 2005 15:44:46 +0000 (+0000) Subject: Moved a debug statement to avoid a testing race. X-Git-Tag: exim-4_61~94 X-Git-Url: https://git.exim.org/exim.git/commitdiff_plain/1921d2ea1648c90d45da23f26cf8855465521426 Moved a debug statement to avoid a testing race. --- diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog index 27c74dabb..7ac300ce2 100644 --- a/doc/doc-txt/ChangeLog +++ b/doc/doc-txt/ChangeLog @@ -1,4 +1,4 @@ -$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.276 2005/12/15 11:18:21 ph10 Exp $ +$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.277 2005/12/15 15:44:46 ph10 Exp $ Change log file for Exim from version 4.21 ------------------------------------------- @@ -43,6 +43,9 @@ PH/07 Two small changes when running in the test harness: increase delay when PH/08 Changed debug output of dbfn_open() flags from numbers to names, so as to be the same on different OS. +PH/09 Moved a debug statement in filter processing to avoid a race problem when + testing. + Exim version 4.60 ----------------- diff --git a/src/src/rda.c b/src/src/rda.c index 1c45f7bb3..1acc75288 100644 --- a/src/src/rda.c +++ b/src/src/rda.c @@ -1,4 +1,4 @@ -/* $Cambridge: exim/src/src/rda.c,v 1.10 2005/08/08 13:21:46 ph10 Exp $ */ +/* $Cambridge: exim/src/src/rda.c,v 1.11 2005/12/15 15:44:46 ph10 Exp $ */ /************************************************* * Exim - an Internet mail transport agent * @@ -770,14 +770,6 @@ if (read(fd, filtertype, sizeof(int)) != sizeof(int) || read(fd, &yield, sizeof(int)) != sizeof(int) || !rda_read_string(fd, error)) goto DISASTER; -/* In the test harness, give the subprocess time to finish off and write -any debugging output. */ - -if (running_in_test_harness) millisleep(250); - -DEBUG(D_route) - debug_printf("rda_interpret: subprocess yield=%d error=%s\n", yield, *error); - /* Read the contents of any syntax error blocks if we have a pointer */ if (eblockp != NULL) @@ -934,6 +926,9 @@ while ((rc = wait(&status)) != pid) } } +DEBUG(D_route) + debug_printf("rda_interpret: subprocess yield=%d error=%s\n", yield, *error); + if (had_disaster) { *error = string_sprintf("internal problem in %s: failure to transfer "