[SCM] libav/experimental: Don't return 0 if buffer setup failed. That signals the RTSP demuxer that the packet was filled in, leading to virtually random behaviour in the decoder later on. Instead, return a negative value.

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Jun 30 17:00:13 UTC 2013


The following commit has been merged in the experimental branch:
commit 298d2fd304becb82c106a5938b95e5d29cf68c37
Author: Ronald S. Bultje <rsbultje at gmail.com>
Date:   Tue Feb 16 18:19:28 2010 +0000

    Don't return 0 if buffer setup failed. That signals the RTSP demuxer that
    the packet was filled in, leading to virtually random behaviour in the
    decoder later on. Instead, return a negative value.
    
    Originally committed as revision 21851 to svn://svn.ffmpeg.org/ffmpeg/trunk

diff --git a/libavformat/rtp_asf.c b/libavformat/rtp_asf.c
index d44ff5a..2b05b3e 100644
--- a/libavformat/rtp_asf.c
+++ b/libavformat/rtp_asf.c
@@ -197,7 +197,7 @@ static int asfrtp_parse_packet(AVFormatContext *s, PayloadContext *asf,
             }
             if (!len_off && !asf->pktbuf &&
                 !(res = url_open_dyn_packet_buf(&asf->pktbuf, rt->asf_ctx->packet_size)))
-                return res;
+                return AVERROR(EIO);
             if (!asf->pktbuf)
                 return AVERROR(EIO);
 

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list