lib-mail: message-address: Add support for parsing RFC5322 "path" syntax. This is either a single angle-addr or just <>. This path syntax differs from the RFC5321 "Path" syntax in that it allows whitespace, which is very important when it is parsed from a header.

Updated copyright notices to include the year 2018.

lib-mail: message-address: Dropped message_detail_address_parse(), since it is no longer used.

lib-mail: message-address: Added convenience functions that convert (single) addresses to string.

lib-mail: message-address: Added functions to initialize a struct message_address.

lib-mail: Add message_detail_address_parse Parses e.g. username+foo@domain into username@domain with detail foo, when delimiter is set to +

lib-mail: message_address_parse() - Fix fill_missing==TRUE handling Mainly MISSING_DOMAIN wasn't set in all situations. Also added unit tests.

lib-mail: message_address_parse() - Handle invalid standalone phrases better We'll treat "local-part" (without quotes) as a mailbox even without @domain, but if it continues with anything or if it's a quoted-string, we'll treat it as a display-name instead. This is probably better than just converting everything to display-name, since there are likely to exist headers like "To: localuser"

lib-mail: message_address_parse() - don't stop at <> when parsing address lists And similarly don't stop at <@domain>

lib-mail: message_address_parse() - don't stop at invalid addresses when parsing group Also the end-group's message_address shouldn't have invalid_syntax=TRUE, except when we didn't actually find the ending ';'

lib-mail: message_address_parse() - Fix reading out-of-bounds read on invalid input

lib-mail: message_address_parse() - Remove unnecessary NULL checks message_address_parse() always initialized rfc822_parser_init() with last_comment string.

lib-mail: message_address_parse() - Add INVALID_ROUTE/SYNTAX_ERROR only if fill_missing==TRUE This allows message_address_write() to write the parsed invalid address in a way that at least somewhat resembles the original input.

lib-mail: message_address_write() - Write empy username as "" We can't output <@domain> because that conflicts with the <@route:...> syntax. Also ""@domain seems to be a valid address.

lib-mail: message_address_write() - Always write <user@domain> as output This is the standard format nowadays, so don't convert <user@domain> into plain user@domain.

lib-mail: message_address_write() - Don't write empty <>

lib-mail: message_address_write() - Don't add '@' if domain is missing

Updated copyright notices to include the year 2017.

global: Replaced all instances of memset(p, 0, sizeof(*p)) with the new i_zero() macro. Used the following script: C_FILES=`git ls-files *.c` H_FILES=`git ls-files *.h` for F in "$C_FILES $H_FILES"; do echo "$F" perl -p -i -e 's/safe_memset\(&\(?([^,]*)\)?,\s*0,\s*sizeof\(\g1\)\)/i_zero_safe(&$1)/g' $F perl -p -i -e 's/safe_memset\(([^,]*),\s*0,\s*sizeof\(\*\g1\)\)/i_zero_safe($1)/g' $F perl -p -i -e 's/memset\(&\(?([^,]*)\)?,\s*0,\s*sizeof\(\g1\)\)/i_zero(&$1)/g' $F perl -p -i -e 's/memset\(([^,]*),\s*0,\s*sizeof\(\*\g1\)\)/i_zero($1)/g' $F done

global: Avoid loops unnecessarily decreasing below zero. Avoids complains from clang -fsanitize=integer

-Wstrict-bool warning fixes

lib-mail: Fixed to 91fdb25b5 Caused by my manual edits to avoid the -Wstrict-bool warnings.

global: Use only explicit int -> bool conversions These were checked with a patched clang.

lib-mail: message_address_write: Quote and escape strings if needed ATEXT characters must be properly quoted when are in phrase. Test case: { name = "test\"test", mailbox = "user", domain = "host" } converts to: "test\"test" <user@host>

lib-mail: parse_mailbox: Set display name instead mailbox when parsing failed It does not make sense to set mailbox without domain on incorrect input. Rather set display name which is more likely useable value. Test case: test is parsed as: { name = "test", mailbox = NULL, domain = NULL }

lib-mail: parse_addr_spec: Email address without local-part is invalid Add explicit invalid_syntax flag also when end of input occure because address is without domain invalid and in this case it was not correctly propagated.

lib-mail: parse_addr_spec: Like in rfc822_skip_comment() check if last_comment is not NULL This will fix possible NULL pointer dereference when caller does not set last_comment.

lib-mail: message_address_write: Fix generating group list with empty name Empty name for group list must be quoted. Test case: { { name = NULL, mailbox = "", domain = NULL }, { name = NULL, mailbox = NULL, domain = NULL } } converts to: "":;

lib-mail: message_address_write: Fix generating empty group list Empty group list ends with ": " not with ", ". Test case: { { name = NULL, mailbox = "group", domain = NULL }, { name = NULL, mailbox = NULL, domain = NULL } } converts to: group:;

global: freshen copyright git ls-files | xargs perl -p -i -e 's/(\d+)-201[0-5]/$1-2016/g;s/ (201[0-5]) Dovecot/ $1-2016 Dovecot/'

global: freshen copyright Robomatically: git ls-files | xargs perl -p -i -e 's/(\d+)-201[0-4]/$1-2015/g;s/ (201[0-4]) Dovecot/ $1-2015 Dovecot/' Happy 2015 everyone! Signed-off-by: Phil Carmody <phil@dovecot.fi>

Updated copyright notices to include year 2014.

Oops :) Update copyrights to 2013 without breaking all .c files.

Updated copyright notices to include year 2013.