On 24/09/14 11:37, Martin Storsjö wrote:
On Wed, 24 Sep 2014, Luca Barbato wrote:

On 24/09/14 10:33, Martin Storsjö wrote:
This fixes handling redirects in case the server provided a session
id within the redirect reply.
---
 libavformat/rtsp.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c
index 23d7da1..646712e 100644
--- a/libavformat/rtsp.c
+++ b/libavformat/rtsp.c
@@ -1812,6 +1812,7 @@ redirect:
     ff_rtsp_close_connections(s);
     if (reply->status_code >=300 && reply->status_code < 400 &&
s->iformat) {
         av_strlcpy(s->filename, reply->location, sizeof(s->filename));
+        rt->session_id[0] = '\0';
         av_log(s, AV_LOG_INFO, "Status %d: Redirecting to %s\n",
                reply->status_code,
                s->filename);


Should be backported to the release branches?

Was this an implicit ok? :-)

As for backporting, maybe - sure, it's a fix but it's also a behaviour
change (theoretically this might actually break some really odd server),
so I'd wait a little at least to let it live in master for a while first.


Let's do that then =)

lu

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to