From adc03a34069efe13857d580e3d973121b7b8a8d3 Mon Sep 17 00:00:00 2001 From: Josh Allmann Date: Tue, 27 Jul 2010 08:16:03 +0000 Subject: [PATCH] rtpdec_xiph: Drop RTP packets that come in without a prior fragment start marker. This can avoid segfaults in some cases. Patch by Josh Allmann, joshua dot allmann at gmail Originally committed as revision 24537 to svn://svn.ffmpeg.org/ffmpeg/trunk --- libavformat/rtpdec_xiph.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/libavformat/rtpdec_xiph.c b/libavformat/rtpdec_xiph.c index 4c9cad2..bcf583f 100644 --- a/libavformat/rtpdec_xiph.c +++ b/libavformat/rtpdec_xiph.c @@ -172,6 +172,11 @@ static int xiph_handle_packet(AVFormatContext * ctx, av_log(ctx, AV_LOG_ERROR, "RTP timestamps don't match!\n"); return AVERROR_INVALIDDATA; } + if (!data->fragment) { + av_log(ctx, AV_LOG_WARNING, + "Received packet without a start fragment; dropping.\n"); + return AVERROR(EAGAIN); + } // copy data to fragment buffer put_buffer(data->fragment, buf, pkt_len); -- 2.7.4