diff options
| author | Martin Storsjö <martin@martin.st> | 2011-02-16 14:35:47 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michaelni@gmx.at> | 2011-02-18 19:52:26 +0100 |
| commit | 09e22efc7e0d98a90e99c9c2047a203c89d9dcd9 (patch) | |
| tree | b05f19b9db3ef9f4d6dca7c5c1f2ab78d1247fa4 | |
| parent | 34c340d49f2e6ca5190f16ced82da32561a7bef2 (diff) | |
| download | ffmpeg-09e22efc7e0d98a90e99c9c2047a203c89d9dcd9.tar.gz | |
rtsp: udp_read_packet returning 0 doesn't mean success
If udp_read_packet returns 0, rtsp_st isn't set and we shouldn't
treat it as a successfully received packet (which is counted and
possibly triggers a RTCP receiver report).
This fixes issue 2612.
(cherry picked from commit 2c35a6bde95a382e2d48570255deb67a7633fa46)
| -rw-r--r-- | libavformat/rtsp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index 41427ae08e..d828c8d09b 100644 --- a/libavformat/rtsp.c +++ b/libavformat/rtsp.c @@ -1681,7 +1681,7 @@ int ff_rtsp_fetch_packet(AVFormatContext *s, AVPacket *pkt) case RTSP_LOWER_TRANSPORT_UDP: case RTSP_LOWER_TRANSPORT_UDP_MULTICAST: len = udp_read_packet(s, &rtsp_st, rt->recvbuf, RECVBUF_SIZE, wait_end); - if (len >=0 && rtsp_st->transport_priv && rt->transport == RTSP_TRANSPORT_RTP) + if (len > 0 && rtsp_st->transport_priv && rt->transport == RTSP_TRANSPORT_RTP) rtp_check_and_send_back_rr(rtsp_st->transport_priv, len); break; } |