diff options
| author | Jeremy Harris <jgh146exb@wizmail.org> | 2019-03-14 12:26:34 +0000 |
|---|---|---|
| committer | Jeremy Harris <jgh146exb@wizmail.org> | 2019-03-14 12:29:06 +0000 |
| commit | 14bc9cf085aff7bd5147881e5b7068769a29b026 (patch) | |
| tree | a26147ed32c7e7b8fb5b49fb51abdfa443a3fb04 /src | |
| parent | a23acfd5c4366f1c4d97e87ac61ee841f39b819a (diff) | |
Fix crash from SRV lookup hitting a CNAME
Diffstat (limited to 'src')
| -rw-r--r-- | src/src/dns.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/src/src/dns.c b/src/src/dns.c index dd929d49f..6ef6b7784 100644 --- a/src/src/dns.c +++ b/src/src/dns.c @@ -710,7 +710,11 @@ lookup, which constructs the names itself, so they should be OK. Besides, bitstring labels don't conform to normal name syntax. (But the aren't used any more.) -For SRV records, we omit the initial _smtp._tcp. components at the start. */ +For SRV records, we omit the initial _smtp._tcp. components at the start. +The check has been seen to bite on the destination of a SRV lookup that +initiall hit a CNAME, for which the next name had only two components. +RFC2782 makes no mention of the possibiility of CNAMES, but the Wikipedia +article on SRV says they are not a valid configuration. */ #ifndef STAND_ALONE /* Omit this for stand-alone tests */ @@ -726,8 +730,8 @@ if (check_dns_names_pattern[0] != 0 && type != T_PTR && type != T_TXT) if (type == T_SRV || type == T_TLSA) { - while (*checkname++ != '.'); - while (*checkname++ != '.'); + while (*checkname && *checkname++ != '.') ; + while (*checkname && *checkname++ != '.') ; } if (pcre_exec(regex_check_dns_names, NULL, CCS checkname, Ustrlen(checkname), |