From: Steven Schubiger <stsc@member.fsf.org>
Date: Sat, 4 Jul 2009 09:19:57 +0000 (+0200)
Subject: Improve handling of schemeless URLs.
X-Git-Tag: v1.13~320^2
X-Git-Url: http://sjero.net/git/?p=wget;a=commitdiff_plain;h=a0dc7788e5e9e1dad7f64738c32c1373b17de226

Improve handling of schemeless URLs.
---

diff --git a/src/ChangeLog b/src/ChangeLog
index 77466d04..dbfd0b66 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,10 @@
+2009-07-04  Steven Schubiger  <stsc@member.fsf.org>
+
+	* url.c (url_parse): If an URL scheme is invalid, distinguish
+	between an unsupported or missing scheme.
+
+	* url.c: Add a "missing scheme" entry to parse_errors.
+
 2009-07-03  Micah Cowan  <micah@cowan.name>
 
 	* iri.h (iri_dup): Provide macro definition for when IRIs are
diff --git a/src/url.c b/src/url.c
index 6b24e063..1172d77f 100644
--- a/src/url.c
+++ b/src/url.c
@@ -628,18 +628,20 @@ static const char *parse_errors[] = {
 #define PE_NO_ERROR                     0
   N_("No error"),
 #define PE_UNSUPPORTED_SCHEME           1
-  N_("Unsupported scheme %s"),
-#define PE_INVALID_HOST_NAME            2
+  N_("Unsupported scheme %s"), /* support for format token only here */
+#define PE_MISSING_SCHEME               2
+  N_("Scheme missing"),
+#define PE_INVALID_HOST_NAME            3
   N_("Invalid host name"),
-#define PE_BAD_PORT_NUMBER              3
+#define PE_BAD_PORT_NUMBER              4
   N_("Bad port number"),
-#define PE_INVALID_USER_NAME            4
+#define PE_INVALID_USER_NAME            5
   N_("Invalid user name"),
-#define PE_UNTERMINATED_IPV6_ADDRESS    5
+#define PE_UNTERMINATED_IPV6_ADDRESS    6
   N_("Unterminated IPv6 numeric address"),
-#define PE_IPV6_NOT_SUPPORTED           6
+#define PE_IPV6_NOT_SUPPORTED           7
   N_("IPv6 addresses not supported"),
-#define PE_INVALID_IPV6_ADDRESS         7
+#define PE_INVALID_IPV6_ADDRESS         8
   N_("Invalid IPv6 numeric address")
 };
 
@@ -676,7 +678,10 @@ url_parse (const char *url, int *error, struct iri *iri, bool percent_encode)
   scheme = url_scheme (url);
   if (scheme == SCHEME_INVALID)
     {
-      error_code = PE_UNSUPPORTED_SCHEME;
+      if (url_has_scheme (url))
+        error_code = PE_UNSUPPORTED_SCHEME;
+      else
+        error_code = PE_MISSING_SCHEME;
       goto error;
     }