urlapi: Fix port parsing of eol colon

A URL with a single colon without a portnumber should use the default
port, discarding the colon. Fix, add a testcase and also do little bit
of comment wordsmithing.

Closes #3365
Reviewed-by: Daniel Stenberg <daniel@haxx.se>

1 files changed, 8 insertions, 0 deletions

diff --git a/tests/unit/unit1653.c b/tests/unit/unit1653.c
index 9851ee58c..b68d8dc4f 100644
--- a/tests/unit/unit1653.c
+++ b/tests/unit/unit1653.c
@@ -110,6 +110,14 @@ UNITTEST_START
   free(ipv6port);
   curl_url_cleanup(u);
 
+  /* Valid IPv6 with no port after the colon, should use default */
+  u = curl_url();
+  ipv6port = strdup("[fe80::250:56ff:fea7:da15]:");
+  ret = Curl_parse_port(u, ipv6port);
+  fail_unless(ret == CURLUE_OK, "Curl_parse_port returned error");
+  free(ipv6port);
+  curl_url_cleanup(u);
+
   /* Incorrect zone index syntax */
   u = curl_url();
   ipv6port = strdup("[fe80::250:56ff:fea7:da15%!25eth3]:80");