[FFmpeg-devel] [PATCH] lavf/mp3enc: make sure the header is valid before writing audio pkt.

Clément Bœsch ubitux at gmail.com
Fri Nov 16 14:12:08 CET 2012


On Fri, Nov 16, 2012 at 01:49:37PM +0100, Michael Niedermayer wrote:
> On Fri, Nov 16, 2012 at 09:24:47AM +0100, Clément Bœsch wrote:
> > From: Clément Bœsch <clement.boesch at smartjog.com>
> > 
> > An invalid header can lead avpriv_mpegaudio_decode_header() to overread
> > and/or div by zero.
> > ---
> 
> > Should we write the packet anyway instead of dropping it completely?
> 
> i suspect it would be better to write them and print a warning
> 
> patch LGTM otherwise
> 

Added a warning, packet written, and pushed. Thanks.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121116/47fd82f5/attachment.asc>


More information about the ffmpeg-devel mailing list