rtpdec: Handle more received packets than expected when sending RR
authorMartin Storsjö <martin@martin.st>
Mon, 14 Jan 2013 09:34:19 +0000 (11:34 +0200)
committerMartin Storsjö <martin@martin.st>
Mon, 14 Jan 2013 15:52:02 +0000 (17:52 +0200)
Without this, we'd signal a huge loss rate (due to unsigned
wraparound) if we had received one packet more than expected (that
is, one seq number sent twice). The code has a check for lost_interval
<= 0, but that doesn't do what was intended as long as the variable is
unsigned.

Signed-off-by: Martin Storsjö <martin@martin.st>
libavformat/rtpdec.c

index 1ccc0f6..73d0206 100644 (file)
@@ -258,7 +258,7 @@ int ff_rtp_check_and_send_back_rr(RTPDemuxContext *s, URLContext *fd,
     uint32_t extended_max;
     uint32_t expected_interval;
     uint32_t received_interval;
-    uint32_t lost_interval;
+    int32_t  lost_interval;
     uint32_t expected;
     uint32_t fraction;