BUG/MEDIUM: tools: fix bad character handling in str2sa_range()

Commit d4448bc8 brought support for parsing port ranges, but invalid
characters are not properly handled and can result in a crash while
parsing the configuration if an invalid character is present in the
port, because the return value is set to NULL then dereferenced.
This commit is contained in:
Willy Tarreau 2013-03-06 15:28:17 +01:00
parent 0dbbf317e6
commit e4c58c89f6

View File

@ -679,8 +679,10 @@ struct sockaddr_storage *str2sa_range(const char *str, int *low, int *high)
porth = atoi(port1 + 1);
porta = porth;
}
else if (*port1) /* other any unexpected char */
else if (*port1) { /* other any unexpected char */
ret = NULL;
goto out;
}
set_host_port(ret, porta);