On Sun, May 24, 2015 at 12:04:23AM +0200, Andreas Cadhalpun wrote:
> Without this the mov demuxer can return a large number of empty packets.
>
> Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com>
> ---
> libavformat/mov.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/libavformat/mov.c b/libavformat/mov.c
> index f6f07be..a6290d1 100644
> --- a/libavformat/mov.c
> +++ b/libavformat/mov.c
> @@ -4260,6 +4260,11 @@ static int mov_read_packet(AVFormatContext *s,
> AVPacket *pkt)
> sample->size = FFMIN(sample->size, (mov->next_root_atom -
> sample->pos));
> }
>
> + if (!sample->size) {
> + av_log(s, AV_LOG_ERROR, "sample size is zero\n");
> + return AVERROR_INVALIDDATA;
> + }this would cause failure to demux future packets, i think its better to skip this but continue without failing [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Republics decline into democracies and democracies degenerate into despotisms. -- Aristotle
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
