Conroy's fix to make the code match with the RTP documentation regarding

writing out the whole header. The docs say it writes the whole header,
but the code (before this patch) did not write out the leading $.

diff --git a/lib/rtsp.c b/lib/rtsp.c
index fd02018fef50395caa1c8331e9441fae2581b91d..873b15f5f9f820c1bc570ac5c2d4754d193815e8 100644 (file)
--- a/lib/rtsp.c
+++ b/lib/rtsp.c
@@ -562,10 +562,10 @@ CURLcode Curl_rtsp_rtp_readwrite(struct SessionHandle *data,
       }
       else {
         /* We have the full RTP interleaved packet
-         * Write out the header but strip the leading '$' */
+         * Write out the header including the leading '$' */
         DEBUGF(infof(data, "RTP write channel %d rtp_length %d\n",
               rtspc->rtp_channel, rtp_length));
-        result = rtp_client_write(conn, &rtp[1], rtp_length + 3);
+        result = rtp_client_write(conn, &rtp[0], rtp_length + 4);
         if(result) {
           failf(data, "Got an error writing an RTP packet");
           *readmore = FALSE;