[FFmpeg-devel] [aacdec] Parse and drop gain control data, so that SSR packets decode.

Dale Curtis dalecurtis at chromium.org
Wed Feb 21 02:56:55 EET 2018

On Tue, Feb 20, 2018 at 4:18 PM, Alex Converse <alex.converse at gmail.com>

> I would make this uint8_t


> I would drop this stack struck and replace the leaf get_bits() with
> skip bits. It makes the code that much harder to exploit and there is
> no point in storing data we don't plan on decoding


> This block seems funny. decode_gain_control() always returns zero.
> Maybe make this warn once per stream when present like some of the
> other AAC warn
> cases.

Done. I've added an AACContext::warn_gain_control member to do this.

This patch set also changes the attribution from Robert Swain to Maxim
Gavrilov based on svn blame of the SoC repository after discussion at
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ssr_drop_final_fixed_attrib.patch
Type: text/x-patch
Size: 3704 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180220/5f298654/attachment.bin>

More information about the ffmpeg-devel mailing list