Fixed issue with MatchDomainName. Fixes issue #1606. This is a valid and confirmed bug report in v3.15.0. Applies to ./configure --enable-sni case with wolfSSL_CTX_UseSNI where common name has wildcards. Pushing fix for visibility now and will add test case.

This commit is contained in:
David Garske 2018-06-08 10:09:53 -07:00
parent b63d3173a1
commit e99fc3026d

View File

@ -7644,6 +7644,7 @@ static int BuildFinished(WOLFSSL* ssl, Hashes* hashes, const byte* sender)
return 1 on success */
int MatchDomainName(const char* pattern, int len, const char* str)
{
int ret = 0;
char p, s;
if (pattern == NULL || str == NULL || len <= 0)
@ -7676,11 +7677,17 @@ int MatchDomainName(const char* pattern, int len, const char* str)
return 0;
}
if (len > 0)
if (len > 0) {
str++;
len--;
}
}
return *str == '\0';
if (*str == '\0')
ret = 1; /* success */
return ret;
}