qmail-ldap-1.03-20010501.patch (and old patch) has something incorrect.
It swap condition between line 12486 and line 12488


Line : 12486
+      if (!addr.s[0] || !str_diff("#@[]", addr.s)) /* if (!addr.s[0]) */
<--- This should be line 12488
+      {
+         if (!str_diff("NOBOUNCE", denymail))
<--- This should be line 12486
+         {
+            why = "refused to accept RFC821 bounce from remote";
+            flagbarf=1;
+         }
+      }
+      else


The correct code as following.

Line : 12486
+      if (!str_diff("NOBOUNCE", denymail))
+      {
+         if (!addr.s[0] || !str_diff("#@[]", addr.s)) /* if (!addr.s[0]) */
+         {
+            why = "refused to accept RFC821 bounce from remote";
+            flagbarf=1;
+         }
+      }
+      else


Metee Khanongnuch



Reply via email to