postfix-2.4-20060727

diff --git a/postfix/HISTORY b/postfix/HISTORY
index ba0653d2792af3f2ceb6769a30016e0eebdaa604..246e2a462bc72ae655de3e438a1480c00e2c1ce8 100644 (file)
--- a/postfix/HISTORY
+++ b/postfix/HISTORY
@@ -12632,6 +12632,12 @@ Apologies for any names omitted.
        = no", the SMTP server would panic as it generated spurious
        Milter requests for unrecognized commands.  File: smtpd/smtpd.c.
 
+20060727
+
+       Cleanup: change redundant milter_abort() and milter_disc_event()
+       calls into NO-OPs.  This avoids unnecessary panic() events
+       for completely harmless conditions.  File: milter/milter8.c.
+
 Wish list:
 
        The type of var_message_limit should be changed from int

diff --git a/postfix/README_FILES/DSN_README b/postfix/README_FILES/DSN_README
index b7bc3fd10685f61c78c9c0eeb95737c011472e5b..a422a3f6fd67bccd28c8a0131ae23d8acca1b4e5 100644 (file)
--- a/postfix/README_FILES/DSN_README
+++ b/postfix/README_FILES/DSN_README
@@ -11,7 +11,7 @@ delivery notifications.
 Specifically, DSN support gives an email sender the ability to specify:
 
   * What notifications are sent: success, failure, delay, or none. Normally,
-    Postfix informs the sender only mail when delivery is delayed or when
+    Postfix informs the sender only when mail delivery is delayed or when
     delivery fails.
 
   * What content is returned in case of failure: only the message headers, or

diff --git a/postfix/html/DSN_README.html b/postfix/html/DSN_README.html
index 38d1efe71aa642ca47c0437d89e3594a7f65d329..8a389498b38813ebc2ccc4b9609be53d917933b2 100644 (file)
--- a/postfix/html/DSN_README.html
+++ b/postfix/html/DSN_README.html
@@ -30,7 +30,7 @@ specify: </p>
 <ul>
 
 <li> <p> What notifications are sent: success, failure, delay, or
-none. Normally, Postfix informs the sender only mail when delivery
+none. Normally, Postfix informs the sender only when mail delivery
 is delayed or when delivery fails.  </p>
 
 <li> <p> What content is returned in case of failure: only the
@@ -83,7 +83,7 @@ strangers (see below for how to turn this off for all clients):
 
 <blockquote>
 <pre>
-/etc/postfix/main.cf:
+/etc/postfix/<a href="postconf.5.html">main.cf</a>:
     <a href="postconf.5.html#smtpd_discard_ehlo_keyword_address_maps">smtpd_discard_ehlo_keyword_address_maps</a> = 
         <a href="cidr_table.5.html">cidr</a>:/etc/postfix/esmtp_access
 
@@ -100,7 +100,7 @@ use the <a href="postconf.5.html#smtpd_discard_ehlo_keywords">smtpd_discard_ehlo
 
 <blockquote>
 <pre>
-/etc/postfix/main.cf:
+/etc/postfix/<a href="postconf.5.html">main.cf</a>:
     <a href="postconf.5.html#smtpd_discard_ehlo_keywords">smtpd_discard_ehlo_keywords</a> = silent-discard, dsn
 </pre>
 </blockquote>

diff --git a/postfix/proto/DSN_README.html b/postfix/proto/DSN_README.html
index ec219ca459c9105ab971aa959524b52a9ad00b5e..7933172759af8f395e98a2514ade41eff8d8272f 100644 (file)
--- a/postfix/proto/DSN_README.html
+++ b/postfix/proto/DSN_README.html
@@ -30,7 +30,7 @@ specify: </p>
 <ul>
 
 <li> <p> What notifications are sent: success, failure, delay, or
-none. Normally, Postfix informs the sender only mail when delivery
+none. Normally, Postfix informs the sender only when mail delivery
 is delayed or when delivery fails.  </p>
 
 <li> <p> What content is returned in case of failure: only the

diff --git a/postfix/src/global/mail_version.h b/postfix/src/global/mail_version.h
index 90cc673b6445d9bd0d425c32743c18ed066a25ad..6a457703c38d79b91251f1f3a78bcccc67120d9c 100644 (file)
--- a/postfix/src/global/mail_version.h
+++ b/postfix/src/global/mail_version.h
@@ -20,7 +20,7 @@
   * Patches change both the patchlevel and the release date. Snapshots have no
   * patchlevel; they change the release date only.
   */
-#define MAIL_RELEASE_DATE      "20060726"
+#define MAIL_RELEASE_DATE      "20060727"
 #define MAIL_VERSION_NUMBER    "2.4"
 
 #ifdef SNAPSHOT

diff --git a/postfix/src/milter/milter.c b/postfix/src/milter/milter.c
index fef14721e1504dbecd9b7ee7a919888b2d18c93b..f006ffc71a595f36187282eb970bae35c9d20915 100644 (file)
--- a/postfix/src/milter/milter.c
+++ b/postfix/src/milter/milter.c
@@ -141,7 +141,9 @@
 /*
 /*     milter_disc_event() reports an SMTP client disconnection
 /*     event to the specified milter instances. No events can
-/*     reported after this call, not even abort() events.
+/*     reported after this call. To simplify usage, redundant calls
+/*     of this function are NO-OPs and don't raise a run-time
+/*     error.
 /*
 /*     milter_helo_event() reports a HELO or EHLO event to the
 /*     specified milter instances, after sending the macros that
@@ -178,9 +180,9 @@
 /*     by a preceding milter. This function must be called with
 /*     as argument an open Postfix queue file.
 /*
-/*     milter_abort() cancels a mail transaction in progress. This
-/*     function is safe to call anywhere between connect and
-/*     disconnect events.
+/*     milter_abort() cancels a mail transaction in progress.  To
+/*     simplify usage, redundant calls of this function are NO-OPs
+/*     and don't raise a run-time error.
 /*
 /*     milter_send() sends a list of mail filters over the specified
 /*     stream. When given a null list pointer, a "no filter"

diff --git a/postfix/src/milter/milter8.c b/postfix/src/milter/milter8.c
index 711f1015a9ebee536c4e83923505a58a96349b5c..b4105c0cb3004781a87042877e466876a87a4c76 100644 (file)
--- a/postfix/src/milter/milter8.c
+++ b/postfix/src/milter/milter8.c
@@ -1835,6 +1835,9 @@ static void milter8_abort(MILTER *m)
      * has to open a new MTA-to-filter socket for each SMTP client.
      */
     switch (milter->state) {
+    case MILTER8_STAT_CLOSED:
+    case MILTER8_STAT_READY:
+       return;
     case MILTER8_STAT_ERROR:
     case MILTER8_STAT_ACCEPT_CON:
     case MILTER8_STAT_REJECT_CON:
@@ -1869,6 +1872,9 @@ static void milter8_disc_event(MILTER *m)
      * has to open a new MTA-to-filter socket for each SMTP client.
      */
     switch (milter->state) {
+    case MILTER8_STAT_CLOSED:
+    case MILTER8_STAT_READY:
+       return;
     case MILTER8_STAT_ERROR:
 #ifdef LIBMILTER_AUTO_DISCONNECT
     case MILTER8_STAT_ACCEPT_CON: