rtmp: fix url parsing
authorLuca Barbato <lu_zero@gentoo.org>
Tue, 15 May 2012 00:24:27 +0000 (17:24 -0700)
committerLuca Barbato <lu_zero@gentoo.org>
Fri, 25 May 2012 21:20:34 +0000 (14:20 -0700)
The application component can have a subcomponent to specify the
application instance even if it doesn't have a ":" in the playpath.

doc/protocols.texi
libavformat/rtmpproto.c

index d6e12f7..172184e 100644 (file)
@@ -164,7 +164,7 @@ content across a TCP/IP network.
 
 The required syntax is:
 @example
-rtmp://@var{server}[:@var{port}][/@var{app}][/@var{playpath}]
+rtmp://@var{server}[:@var{port}][/@var{app}][/@var{instance}][/@var{playpath}]
 @end example
 
 The accepted parameters are:
index 9af4584..807e899 100644 (file)
@@ -1037,9 +1037,10 @@ static int rtmp_open(URLContext *s, const char *uri, int flags)
             fname = next;
             rt->app[0] = '\0';
         } else {
+            // make sure we do not mismatch a playpath for an application instance
             char *c = strchr(p + 1, ':');
             fname = strchr(p + 1, '/');
-            if (!fname || c < fname) {
+            if (!fname || (c && c < fname)) {
                 fname = p + 1;
                 av_strlcpy(rt->app, path + 1, p - path);
             } else {