Any feedback on this patch?
I'm loath to include patches in the FreeBSD port for Exim until I know a
few people are happy with them.
Ciao,
Sheldon.
----- Forwarded message from Philip Hazel <ph10@???> -----
Date: Tue, 15 Jul 2003 15:29:27 +0100 (BST)
From: Philip Hazel <ph10@???>
To: Richard Gilbert <R.Gilbert@???>,
Ian Jackson <ijackson@???>, gordonb@???
cc: exim-users@???, postmaster@???
Subject: Re: [Exim] Unexpected callout failure
On Wed, 9 Jul 2003, Richard Gilbert wrote:
> I came across this message while searching the archive to answer the
> question as to why callout sender verification was occasionally failing
> with an apparently illegal response to the RSET command.
The patch below should fix this problem. Feedback appreciated...
Philip
--
Philip Hazel University of Cambridge Computing Service,
ph10@??? Cambridge, England. Phone: +44 1223 334714.
*** exim-4.20/src/verify.c Mon May 12 14:39:23 2003
--- verify.c Tue Jul 15 15:12:09 2003
***************
*** 463,471 ****
state to send RCPT. Unless there's some problem such as a dropped
connection, we expect to succeed, because the commands succeeded above. */
! else
{
! if (errno == 0 && randombuffer[0] == '5')
new_domain_record.random_result = ccache_reject;
done =
--- 463,471 ----
state to send RCPT. Unless there's some problem such as a dropped
connection, we expect to succeed, because the commands succeeded above. */
! else if (errno == 0)
{
! if (randombuffer[0] == '5')
new_domain_record.random_result = ccache_reject;
done =
***************
*** 477,483 ****
smtp_read_response(&inblock, responsebuffer, sizeof(responsebuffer),
'2', callout);
}
! } /* Random check */
/* If the host is accepting all local parts, as determined by the "random"
check, we don't need to waste time doing any further checking. */
--- 477,484 ----
smtp_read_response(&inblock, responsebuffer, sizeof(responsebuffer),
'2', callout);
}
! else done = FALSE; /* Some timeout/connection problem */
! } /* Random check */
/* If the host is accepting all local parts, as determined by the "random"
check, we don't need to waste time doing any further checking. */
***************
*** 531,557 ****
don't give the IP address because this may be an internal host whose identity
is not to be widely broadcast. */
! if (!done && errno == 0)
{
! if (*responsebuffer == 0) Ustrcpy(responsebuffer, US"connection dropped");
! addr->message =
! string_sprintf("response to \"%s\" from %s [%s] was: %s",
! big_buffer, host->name, host->address,
! string_printing(responsebuffer));
! addr->user_message = is_recipient?
! string_sprintf("Callout verification failed:\n%s", responsebuffer)
! :
! string_sprintf("Called: %s\nSent: %s\nResponse: %s",
! host->address, big_buffer, responsebuffer);
! /* Hard rejection ends the process */
! if (responsebuffer[0] == '5') /* Address rejected */
! {
! yield = FAIL;
! done = TRUE;
}
}
--- 532,565 ----
don't give the IP address because this may be an internal host whose identity
is not to be widely broadcast. */
! if (!done)
{
! if (errno == ETIMEDOUT)
! {
! DEBUG(D_verify) debug_printf("SMTP timeout\n");
! }
! else if (errno == 0)
! {
! if (*responsebuffer == 0) Ustrcpy(responsebuffer, US"connection dropped");
! addr->message =
! string_sprintf("response to \"%s\" from %s [%s] was: %s",
! big_buffer, host->name, host->address,
! string_printing(responsebuffer));
! addr->user_message = is_recipient?
! string_sprintf("Callout verification failed:\n%s", responsebuffer)
! :
! string_sprintf("Called: %s\nSent: %s\nResponse: %s",
! host->address, big_buffer, responsebuffer);
! /* Hard rejection ends the process */
! if (responsebuffer[0] == '5') /* Address rejected */
! {
! yield = FAIL;
! done = TRUE;
! }
}
}
--
## List details at
http://www.exim.org/mailman/listinfo/exim-users Exim details at
http://www.exim.org/ ##
----- End forwarded message -----