route is already set to "INVALID_ROUTE" when fill_missing is true, but
setting invalid_syntax was not done yet.
---
src/lib-mail/message-address.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/lib-mail/message-address.c b/src/lib-mail/message-address.c
index 29cac2b..96584e2 100644
--- a/src/lib-mail/message-address.c
+++ b/src/lib-mail/message-address.c
@@ -153,6 +153,7 @@ static int parse_angle_addr(struct
message_address_parser_context *ctx)
if (parse_domain_list(ctx) <= 0 || *ctx->parser.data != ':') {
if (ctx->fill_missing)
ctx->addr.route = "INVALID_ROUTE";
+ ctx->addr.invalid_syntax = TRUE;
if (ctx->parser.data == ctx->parser.end)
return -1;
/* try to continue anyway */
--
1.7.9.5
On Saturday 12 August 2017 14:36:31 Pali Roh?r wrote:> route is already set to "INVALID_ROUTE" when fill_missing is true, but > setting invalid_syntax was not done yet. > --- > src/lib-mail/message-address.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/lib-mail/message-address.c b/src/lib-mail/message-address.c > index 29cac2b..96584e2 100644 > --- a/src/lib-mail/message-address.c > +++ b/src/lib-mail/message-address.c > @@ -153,6 +153,7 @@ static int parse_angle_addr(struct message_address_parser_context *ctx) > if (parse_domain_list(ctx) <= 0 || *ctx->parser.data != ':') { > if (ctx->fill_missing) > ctx->addr.route = "INVALID_ROUTE"; > + ctx->addr.invalid_syntax = TRUE; > if (ctx->parser.data == ctx->parser.end) > return -1; > /* try to continue anyway */Hi! I would like to remind this patch. -- Pali Roh?r pali.rohar at gmail.com
Possibly Parallel Threads
- another assertion failure in current 1.1 hg (1.1.3 was working fine) - file message-address.c: line 43 (parse_local_part): assertion failed: (ctx->parser.data != ctx->parser.end)
- v2.2.27 Panic: file rfc822-parser.h: line 23 (rfc822_parser_deinit): assertion failed: (ctx->data <= ctx->end)
- v2.2.27 Panic: file rfc822-parser.h: line 23 (rfc822_parser_deinit): assertion failed: (ctx->data <= ctx->end)
- 1.1.5 abort with mal-formed address in header