From: Simon Schubert <corecode@dragonflybsd.org>
Date: Thu, 9 Jul 2009 12:37:15 +0000 (+0200)
Subject: dma: treat a QUIT error as merely a warning
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=83f3cc405c4e2ca4e2ceded20ec54f446ee26e48;p=people%2Fms%2Fdma.git

dma: treat a QUIT error as merely a warning

RFC 2821 only mandates that a QUIT error should abort an unfinished
transaction, and since we've reached this point, the DATA command has
succeeded and the message has been accepted for delivery by the remote
end.  Thus, just warn about it.

Submitted-by: Peter Pentchev <roam@ringlet.net>
---

diff --git a/net.c b/net.c
index e688b10..049baee 100644
--- a/net.c
+++ b/net.c
@@ -462,11 +462,9 @@ deliver_remote(struct qitem *it, const char **errmsg)
 	}
 
 	send_remote_command(fd, "QUIT");
-	if (read_remote(fd, 0, NULL) != 2) {
-		syslog(LOG_ERR, "%s: remote delivery deferred: "
+	if (read_remote(fd, 0, NULL) != 2)
+		syslog(LOG_WARNING, "%s: remote delivery succeeded but "
 		       "QUIT failed: %s", it->queueid, neterr);
-		return (1);
-	}
 out:
 
 	close(fd);