On Wed, 12 Feb 2020, gautamr...@gmail.com wrote:
From: Gautam Ramakrishnan <gautamr...@gmail.com>
This commit fixes bug #8466 wherein URLs with query string
not preceeded by the '/' symbol in a URL was failing to get
parsed. The av_url_split() function now checks if the path
starts with a '/' and prepends it if it does not.
---
libavformat/utils.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/libavformat/utils.c b/libavformat/utils.c
index 81ea239a66..a14d4ef90d 100644
--- a/libavformat/utils.c
+++ b/libavformat/utils.c
@@ -4819,6 +4819,13 @@ void av_url_split(char *proto, int proto_size,
av_strlcpy(path, ls, path_size);
else
ls = &p[strlen(p)]; // XXX
+ /* if path does not start with a /, prepend a / */
+ if (path[0] != '/') {
+ char temp[1024];
+ av_strlcpy(temp, path, path_size);
+ path[0] = '/';
+ av_strlcpy(path + 1, temp, path_size - 1);
+ }
I made a patch which fixes this in the HTTP protocol, as modifying the
empty path to / is a HTTP requirement, other protocols might not need
this.
http://mplayerhq.hu/pipermail/ffmpeg-devel/2020-February/256902.html
Regards,
Marton
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
