[vlc-devel] commit: rtsp: fix RTP-Info header syntax (Pierre Ynard )
git version control
git at videolan.org
Wed May 13 20:09:34 CEST 2009
vlc | branch: master | Pierre Ynard <linkfanel at yahoo.fr> | Wed May 13 18:26:01 2009 +0200| [42043990d3418add07088b8e2e1117c9caee5412] | committer: Rémi Denis-Courmont
rtsp: fix RTP-Info header syntax
As per RFC2326, the correct syntax includes a "url=" that is missing
here. Tested (and bug found) with a Real Player client.
Signed-off-by: Rémi Denis-Courmont <remi at remlab.net>
> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=42043990d3418add07088b8e2e1117c9caee5412
---
modules/stream_out/rtsp.c | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/modules/stream_out/rtsp.c b/modules/stream_out/rtsp.c
index 30e3552..523a1d4 100644
--- a/modules/stream_out/rtsp.c
+++ b/modules/stream_out/rtsp.c
@@ -635,7 +635,7 @@ static int RtspHandler( rtsp_stream_t *rtsp, rtsp_stream_id_t *id,
{
/* FIXME: we really need to limit the number of tracks... */
char info[ses->trackc * ( strlen( control )
- + sizeof("/trackID=123;seq=65535, ") ) + 1];
+ + sizeof("url=/trackID=123;seq=65535, ") ) + 1];
size_t infolen = 0;
for( int i = 0; i < ses->trackc; i++ )
@@ -649,7 +649,8 @@ static int RtspHandler( rtsp_stream_t *rtsp, rtsp_stream_id_t *id,
rtp_add_sink( tr->id, tr->fd, false );
}
infolen += sprintf( info + infolen,
- "%s/trackID=%u;seq=%u, ", control,
+ "url=%s/trackID=%u;seq=%u, ",
+ control,
rtp_get_num( tr->id ),
rtp_get_seq( tr->id ) );
}
More information about the vlc-devel
mailing list