[FFmpeg-cvslog] lavf/utils: add support for special characters encoding in URL

Senthilnathan M git at videolan.org
Thu Feb 28 00:41:14 CET 2013


ffmpeg | branch: master | Senthilnathan M <senthilnathan.maadasamy at gmail.com> | Sun Feb 10 23:26:52 2013 +0530| [4a8fc1d83b1b55e1ac533644168018ebeec0c732] | committer: Stefano Sabatini

lavf/utils: add support for special characters encoding in URL

In particular, fix trac ticket #2031.

Signed-off-by: Senthilnathan M <senthilnathan.maadasamy at gmail.com>
Signed-off-by: Stefano Sabatini <stefasab at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=4a8fc1d83b1b55e1ac533644168018ebeec0c732
---

 libavformat/utils.c |   47 +++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 47 insertions(+)

diff --git a/libavformat/utils.c b/libavformat/utils.c
index 9bd2d0c..47637b9 100644
--- a/libavformat/utils.c
+++ b/libavformat/utils.c
@@ -3767,6 +3767,50 @@ void av_pkt_dump_log2(void *avcl, int level, AVPacket *pkt, int dump_payload,
     pkt_dump_internal(avcl, NULL, level, pkt, dump_payload, st->time_base);
 }
 
+/**
+ * Percent encode part of an URL string according to RFC 3986.
+ *
+ * @param component      portion of an URL (e.g. protocol, hostname, path) to
+ *                       percent-encode. This will be percent-encoded in place.
+ * @param allowed        string containing the allowed characters which must not be
+ *                       encoded. It may be NULL if there are no such characters.
+ * @param component_size size in bytes of the component buffer
+ */
+static void percent_encode_url(char *component, size_t component_size,
+                               const char *allowed)
+{
+    char enc[MAX_URL_SIZE], c;
+    int enc_len = 0;
+    char *src = component;
+
+    while (c = *src) {
+        if (isalnum(c) || strchr("-._~%", c) ||
+            (allowed && strchr(allowed, c))) {
+            if (enc_len+1 < MAX_URL_SIZE)
+                enc[enc_len] = c;
+            else
+                break;
+            enc_len++;
+        } else {
+            if (enc_len+3 < MAX_URL_SIZE)
+                snprintf(&enc[enc_len], 4, "%%%02x", c);
+            else
+                break;
+            enc_len += 3;
+        }
+        src++;
+    }
+
+    enc[enc_len++] = '\0';
+    if (enc_len <= component_size) {
+        av_strlcpy(component, enc, component_size);
+    } else {
+        av_log(NULL, AV_LOG_ERROR,
+               "Skipping percent-encoding for string '%s' since buffer is too small\n",
+               component);
+    }
+}
+
 void av_url_split(char *proto, int proto_size,
                   char *authorization, int authorization_size,
                   char *hostname, int hostname_size,
@@ -3830,6 +3874,9 @@ void av_url_split(char *proto, int proto_size,
             av_strlcpy(hostname, p,
                        FFMIN(ls + 1 - p, hostname_size));
     }
+
+    percent_encode_url(hostname, hostname_size, NULL);
+    percent_encode_url(path, path_size, "/?");
 }
 
 char *ff_data_to_hex(char *buff, const uint8_t *src, int s, int lowercase)



More information about the ffmpeg-cvslog mailing list