ffmpeg | branch: master | Martin Storsjö <mar...@martin.st> | Tue Dec  9 
16:05:14 2014 +0200| [95d880fa6436f3b557a9c060428a04b9e4d552df] | committer: 
Martin Storsjö

rtpproto: Fix the input RTP data format check

Only the upper 2 bits of the first byte are known to be
a fixed value.

The lower bits in the first byte of a RTP packet could be set
if the input is from another RTP packetizers than libavformat's,
but for RTCP packets, they would also be set when sending RTCP RR
packets, triggering false warnings about incorrect input format
to the protocol.

Signed-off-by: Martin Storsjö <mar...@martin.st>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=95d880fa6436f3b557a9c060428a04b9e4d552df
---

 libavformat/rtpproto.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/rtpproto.c b/libavformat/rtpproto.c
index 1171fc2..5bff00e 100644
--- a/libavformat/rtpproto.c
+++ b/libavformat/rtpproto.c
@@ -411,7 +411,7 @@ static int rtp_write(URLContext *h, const uint8_t *buf, int 
size)
     if (size < 2)
         return AVERROR(EINVAL);
 
-    if (buf[0] != (RTP_VERSION << 6))
+    if ((buf[0] & 0xc0) != (RTP_VERSION << 6))
         av_log(h, AV_LOG_WARNING, "Data doesn't look like RTP packets, "
                                   "make sure the RTP muxer is used\n");
 

_______________________________________________
ffmpeg-cvslog mailing list
ffmpeg-cvslog@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-cvslog

Reply via email to