[FFmpeg-devel] [PATCH] lavf/wavdec: skip padding byte

Matthieu Bouron matthieu.bouron at gmail.com
Mon Apr 8 19:44:29 CEST 2013


On Mon, Apr 08, 2013 at 01:33:21PM -0400, Derek Buitenhuis wrote:
> On 2013-04-08 1:28 PM, Matthieu Bouron wrote:
> > Fixes ticket #2417.
> > ---
> >  libavformat/wavdec.c | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> Please provide an actual explanation of what is being fixed.

WAV chunks must be even aligned. This patch skip the extra padding byte
if chunk size is odd.

> 
> > -        avio_skip(pb, size);
> > +        avio_skip(pb, size + (size & 1));
> 
> So um, why?
> 
> > +        /* skip padding byte */
> > +        next_tag_ofs += (next_tag_ofs < INT64_MAX && next_tag_ofs & 1);
> 
> This seems OK.
> 
> - Derek
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


More information about the ffmpeg-devel mailing list