On Wed, Jan 23, 2019 at 08:11:48AM +0000, f...@gmx.ch wrote: > From: Frederic Pillonel <f...@gmx.ch> > > --- > libavformat/rtsp.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c > index ceb770a..3cc9012f 100644 > --- a/libavformat/rtsp.c > +++ b/libavformat/rtsp.c > @@ -548,9 +548,10 @@ static void sdp_parse_line(AVFormatContext *s, > SDPParseState *s1, > NULL, NULL, 0, p); > if (proto[0] == '\0') { > /* relative control URL */ > - if > (rtsp_st->control_url[strlen(rtsp_st->control_url)-1]!='/') > - av_strlcat(rtsp_st->control_url, "/", > - sizeof(rtsp_st->control_url)); > + if > (rtsp_st->control_url[strlen(rtsp_st->control_url)-1]!='/' && > + (strlen(p) > 0 && p[0]!='/')) > + av_strlcat(rtsp_st->control_url, "/", > + sizeof(rtsp_st->control_url));
is this actually correct ? if the added path starts with a / shouldnt it be an absolute path ? IIUC https://tools.ietf.org/html/rfc1808 needs to be followed here thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The worst form of inequality is to try to make unequal things equal. -- Aristotle
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel