[FFmpeg-devel] [PATCH] lavf/avio: check for : in filenames for protocols.

Nicolas George nicolas.george at normalesup.org
Wed Feb 27 19:23:12 CET 2013


If the first "special" character in a filename is a comma,
it can introduce protocol options, but only if there is a
colon at the end. Otherwise, it is just a filename with a
comma.

Fix trac ticket #2303.

Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
---
 libavformat/avio.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/libavformat/avio.c b/libavformat/avio.c
index b00cc9d..f6af0cb 100644
--- a/libavformat/avio.c
+++ b/libavformat/avio.c
@@ -220,7 +220,9 @@ int ffurl_alloc(URLContext **puc, const char *filename, int flags,
                                      "Missing call to av_register_all()?\n");
     }
 
-    if (filename[proto_len] != ':' &&  filename[proto_len] != ',' || is_dos_path(filename))
+    if (filename[proto_len] != ':' &&
+        (filename[proto_len] != ',' || !strchr(filename + proto_len + 1, ':')) ||
+        is_dos_path(filename))
         strcpy(proto_str, "file");
     else
         av_strlcpy(proto_str, filename, FFMIN(proto_len+1, sizeof(proto_str)));
-- 
1.7.10.4



More information about the ffmpeg-devel mailing list