X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=src%2Fsrc%2Fchild.c;h=d3cd88201d46c5422588e11e894a10d4996af718;hb=8c2a478b1f6f8c3fb43317c1e6729b23a3b972b7;hp=085f2660063a686cb559649bb162a2e4f8bf860f;hpb=c2a1bba0d1fe5e19f93c92544422036814695c45;p=exim.git diff --git a/src/src/child.c b/src/src/child.c index 085f26600..d3cd88201 100644 --- a/src/src/child.c +++ b/src/src/child.c @@ -10,10 +10,6 @@ static void (*oldsignal)(int); -#if defined(SUPPORT_TLS) && defined(EXPERIMENTAL_REQUIRETLS) -static uschar tls_requiretls_copy = 0; -#endif - /************************************************* * Ensure an fd has a given value * @@ -79,11 +75,7 @@ int n = 0; int extra = pcount ? *pcount : 0; uschar **argv; -#if defined(SUPPORT_TLS) && defined(EXPERIMENTAL_REQUIRETLS) -if (tls_requiretls) extra++; -#endif - -argv = store_get((extra + acount + MAX_CLMACROS + 18) * sizeof(char *)); +argv = store_get((extra + acount + MAX_CLMACROS + 18) * sizeof(char *), FALSE); /* In all case, the list starts out with the path, any macros, and a changed config file. */ @@ -129,11 +121,6 @@ if (!minimal) } } -#if defined(SUPPORT_TLS) && defined(EXPERIMENTAL_REQUIRETLS) -if (tls_requiretls_copy & REQUIRETLS_MSG) - argv[n++] = US"-MS"; -#endif - /* Now add in any others that are in the call. Remember which they were, for more helpful diagnosis on failure. */ @@ -243,9 +230,6 @@ occur. */ if (pid == 0) { -#if defined(SUPPORT_TLS) && defined(EXPERIMENTAL_REQUIRETLS) - tls_requiretls_copy = tls_requiretls; -#endif force_fd(pfd[pipe_read], 0); (void)close(pfd[pipe_write]); if (debug_fd > 0) force_fd(debug_fd, 2); @@ -517,13 +501,18 @@ for(;;) if (rc == pid) { int lowbyte = status & 255; - if (lowbyte == 0) yield = (status >> 8) & 255; - else yield = -lowbyte; + yield = lowbyte == 0 ? (status >> 8) & 255 : -lowbyte; break; } if (rc < 0) { - yield = (errno == EINTR && sigalrm_seen)? -256 : -257; + /* This "shouldn't happen" test does happen on MacOS: for some reason + I do not understand we seems to get an alarm signal despite not having + an active alarm set. There seems to be only one, so just go round again. */ + + if (errno == EINTR && sigalrm_seen && timeout <= 0) continue; + + yield = (errno == EINTR && sigalrm_seen) ? -256 : -257; break; } }