From: Andreas Öman <andreas@lonelycoder.com>
Date: Fri, 7 Dec 2007 08:05:02 +0000 (+0000)
Subject: improve HTTP URL de-escaper
X-Git-Tag: 2.12~1297
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=fc024a0099b4a8904603128ef65f5b1df9b05034;p=thirdparty%2Ftvheadend.git

improve HTTP URL de-escaper
---

diff --git a/http.c b/http.c
index dd81ecdc1..58b75bad4 100644
--- a/http.c
+++ b/http.c
@@ -622,13 +622,67 @@ http_path_add(const char *path, void *opaque, http_callback_t *callback)
 }
 
 
+/**
+ * De-escape HTTP URL
+ */
+static void
+http_deescape(char *s)
+{
+  char v, *d = s;
+
+  while(*s) {
+    if(*s == '+') {
+      *d++ = ' ';
+      s++;
+    } else if(*s == '%') {
+      s++;
+      switch(*s) {
+      case '0' ... '9':
+	v = (*s - '0') << 4;
+	break;
+      case 'a' ... 'f':
+	v = (*s - 'a' + 10) << 4;
+	break;
+      case 'A' ... 'F':
+	v = (*s - 'A' + 10) << 4;
+	break;
+      default:
+	*d = 0;
+	return;
+      }
+      s++;
+      switch(*s) {
+      case '0' ... '9':
+	v |= (*s - '0');
+	break;
+      case 'a' ... 'f':
+	v |= (*s - 'a' + 10);
+	break;
+      case 'A' ... 'F':
+	v |= (*s - 'A' + 10);
+	break;
+      default:
+	*d = 0;
+	return;
+      }
+      s++;
+
+      *d++ = v;
+    } else {
+      *d++ = *s++;
+    }
+  }
+  *d = 0;
+}
+
+
 /**
  * Parse arguments of a HTTP GET url, not perfect, but works for us
  */
 static void
 http_parse_get_args(http_connection_t *hc, char *args)
 {
-  char *k, *v, *s;
+  char *k, *v;
 
   while(args) {
     k = args;
@@ -641,13 +695,8 @@ http_parse_get_args(http_connection_t *hc, char *args)
     if(args != NULL)
       *args++ = 0;
 
-    s = v;
-    while(*s) {
-      if(*s == '+')
-	*s = ' ';
-      s++;
-    }
-
+    http_deescape(k);
+    http_deescape(v);
     http_arg_set(&hc->hc_url_args, k, v);
   }
 }