Don't report failure just because MTA is being over-picky

diff --git a/lib/sendmail.c b/lib/sendmail.c
index 12291b8859a6ee0b71f5a0c29234a15d2ba366e5..2dc82d05f3f6a727ef8e8a1d41da5356cf059a82 100644 (file)
--- a/lib/sendmail.c
+++ b/lib/sendmail.c
@@ -271,11 +271,11 @@ int sendmail(const char *sender,
       ;
     if(w < 0)
       fatal(errno, "error calling waitpid");
       ;
     if(w < 0)
       fatal(errno, "error calling waitpid");

-    if(w) {
-      error(0, "%s -bs: %s", config->sendmail, wstat(w));
-      if(!rc)
-        rc = -1;
-    }
+    if(w)
+      info("warning: %s -bs: %s", config->sendmail, wstat(w));
+    /* Not fatal - we determine success/failure from the SMTP conversation.
+     * Some MTAs exit nonzero if you don't QUIT, which is just stupidly
+     * picky. */

   }
   return rc;
 }
   }
   return rc;
 }