Enforce minimum recommended SMTP timeouts, apply to EHLO/LHLO as well.

Short timeouts could cause fetchmail to not wait long enough for the
"250 Ok" after shipping a long message, particularly with synchronous
mailers and extensive spam filtering. This caused fetchmail to re-fetch
long messages.

While the actual fix is making sure that the timeout is no shorter than
the time the SMTP server takes to process the message, we now enforce
the minimum RFC-5321 recommended timeouts even if the user configures a
lower timeout.

This is to fix Berlios Bug #10972, reported by Viktor Binzberger.

NOTE: it is untested whether we will properly delete the message from
the POP3/IMAP server or mark it as seen, as the upstream server may
close the connection sooner.

svn path=/branches/BRANCH_6-3/; revision=5338

1 files changed, 1 insertions, 1 deletions

diff --git a/odmr.c b/odmr.c
index cb0d559a..a20d89f1 100644
--- a/odmr.c
+++ b/odmr.c
@@ -36,7 +36,7 @@ static int odmr_ok (int sock, char *argbuf)
     int ok;
 
     (void)argbuf;
-    ok = SMTP_ok(sock, SMTP_MODE);
+    ok = SMTP_ok(sock, SMTP_MODE, TIMEOUT_DEFAULT);
     if (ok == SM_UNRECOVERABLE)
 	return(PS_PROTOCOL);
     else