lib-mail: message-address: Make the parser allow paths that omit `<' …

…and `>'.

This is a syntax violation, but we allow it to account for a rather wide
selection of software that does not follow the standards.

src/lib-mail/message-address.c

@@ -393,10 +393,19 @@ static int parse_path(struct message_address_parser_context *ctx)
 
 	if (rfc822_skip_lwsp(&ctx->parser) <= 0)
 		return -1;
-	if (*ctx->parser.data != '<')
-		return -1;
-	if ((ret=parse_angle_addr(ctx, TRUE)) < 0 ||
-	    (ret=rfc822_skip_lwsp(&ctx->parser)) < 0 ||
+	if (*ctx->parser.data != '<') {
+		/* Cope with paths that omit < and >. This is a syntax
+		   violation, but we allow it to account for a rather wide
+		   selection of software that does not follow the standards.
+		 */
+		if ((ret=parse_local_part(ctx)) > 0 &&
+		    *ctx->parser.data == '@') {
+			ret = parse_domain(ctx);
+		}
+	} else {
+		ret = parse_angle_addr(ctx, TRUE);
+	}
+	if (ret < 0 || (ret=rfc822_skip_lwsp(&ctx->parser)) < 0 ||
 	    ctx->parser.data != ctx->parser.end ||
 	    (ctx->addr.mailbox != NULL &&
 	     (ctx->addr.domain == NULL || *ctx->addr.domain == '\0')) ||

src/lib-mail/test-message-address.c

@@ -355,6 +355,10 @@ static void test_message_address_path(void)
 		  { NULL, NULL, NULL, "user", "domain", FALSE } },
 		{ "  <user@domain>  ", "<user@domain>",
 		  { NULL, NULL, NULL, "user", "domain", FALSE } },
+		{ "user@domain", "<user@domain>",
+		  { NULL, NULL, NULL, "user", "domain", FALSE } },
+		{ "  user@domain  ", "<user@domain>",
+		  { NULL, NULL, NULL, "user", "domain", FALSE } },
 		{ "<\"user\"@domain>", "<user@domain>",
 		  { NULL, NULL, NULL, "user", "domain", FALSE } },
 		{ "<\"user name\"@domain>", NULL,
@@ -405,17 +409,20 @@ static void test_message_address_path_invalid(void)
 		" < ",
 		">",
 		" > ",
-		"user@domain",
-		"  user@domain  ",
 		"<user@domain",
 		"  <user@domain  ",
 		"user@domain>",
 		"  user@domain>  ",
 		"<user>",
 		"<@route@route2:user>",
 		"<@domain>",
+		"@domain",
+		"  @domain  ",
 		"<user@>",
+		"user@",
+		"  user@  ",
 		"<user@domain>bladiebla",
+		"user@domain@"
 	};
 	const struct message_address *addr;
 	unsigned int i;