rewrite: revert to unchecked result of parse_extract_address()
authorHeiko Schlittermann (HS12-RIPE) <hs@schlittermann.de>
Mon, 26 Apr 2021 14:16:49 +0000 (16:16 +0200)
committerHeiko Schlittermann (HS12-RIPE) <hs@schlittermann.de>
Thu, 27 May 2021 19:31:00 +0000 (21:31 +0200)
Now it breaks 471, and overlong addresses won't make it into the rewrite
process, as they are handled as empty.

(cherry picked from commit 506286c62b8786a926dafb5bb05d3103492b86bc)

src/src/rewrite.c

index f40c73c5120e18cc49f45d2b3c71fc1d48915335..6a6da47dfa0c70d4e3265fb85a033efe5a4f0a23 100644 (file)
@@ -488,18 +488,34 @@ while (*s)
 
   /* There isn't much we can do for syntactic disasters at this stage.
   Pro tem (possibly for ever) ignore them.
 
   /* There isn't much we can do for syntactic disasters at this stage.
   Pro tem (possibly for ever) ignore them.
-  If we got nothing, they there was any sort of error: non-parsable address,
+  If we got nothing, then there was any sort of error: non-parsable address,
   empty address, overlong addres. Sometimes the result matters, sometimes not.
   It seems this function is called for *any* header we see. */
 
 
   if (!recipient)
     {
   empty address, overlong addres. Sometimes the result matters, sometimes not.
   It seems this function is called for *any* header we see. */
 
 
   if (!recipient)
     {
+#if 0
+    /* FIXME:
+    This was(!) an attempt tho handle empty rewrits, but seemingly it
+    needs more effort to decide if the returned empty address matters.
+    Now this will now break test 471 again.
+
+    471 fails now because it uses an overlong address, for wich parse_extract_address()
+    returns an empty address (which was not expected).
+
+    Checking the output and exit if rewrite_rules or routed_old are present
+    isn't a good idea either: It's enough to have *any* rewrite rule
+    in the configuration plus "To: undisclosed recpients:;" to exit(), which
+    is not what we want.
+    */
+
     if (rewrite_rules || routed_old)
       {
       log_write(0, LOG_MAIN, "rewrite: %s", errmess);
       exim_exit(EXIT_FAILURE);
       }
     if (rewrite_rules || routed_old)
       {
       log_write(0, LOG_MAIN, "rewrite: %s", errmess);
       exim_exit(EXIT_FAILURE);
       }
+#endif
     loop_reset_point = store_reset(loop_reset_point);
     continue;
     }
     loop_reset_point = store_reset(loop_reset_point);
     continue;
     }