[vlc-devel] commit: rtsp: fix RTP-Info header syntax (Pierre Ynard )

git version control git at videolan.org
Wed May 13 19:29:05 CEST 2009


vlc | branch: 1.0-bugfix | 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