Skip to content
Permalink
Browse files

urlapi: require a non-zero host name length when parsing URL

Updated test 1560 to verify.

Closes #3880
  • Loading branch information...
bagder committed May 13, 2019
1 parent c60042f commit 9f9ec7da570bf1468eafe611023627c9a2b0b5d2
Showing with 5 additions and 0 deletions.
  1. +2 −0 lib/urlapi.c
  2. +3 −0 tests/libtest/lib1560.c
@@ -636,6 +636,8 @@ static CURLUcode hostname_check(struct Curl_URL *u, char *hostname)
/* hostname with bad content */
return CURLUE_MALFORMED_INPUT;
}
if(!hostname[0])
return CURLUE_NO_HOST;
return CURLUE_OK;
}

@@ -140,6 +140,9 @@ static struct testcase get_parts_list[] ={
"file | [11] | [12] | [13] | [14] | [15] | C:\\programs\\foo | [16] | [17]",
CURLU_DEFAULT_SCHEME, 0, CURLUE_OK},
#endif
/* URL without host name */
{"http://a:b@/x", "",
CURLU_DEFAULT_SCHEME, 0, CURLUE_NO_HOST},
{"boing:80",
"https | [11] | [12] | [13] | boing | 80 | / | [16] | [17]",
CURLU_DEFAULT_SCHEME, 0, CURLUE_OK},

0 comments on commit 9f9ec7d

Please sign in to comment.
You can’t perform that action at this time.