[FFmpeg-devel] [PATCH] avcodec/hevcdec: do not let updated extradata corrupt state

[Hendrik Leppkes]

On Tue, Jul 4, 2017 at 10:53 PM, Derek Buitenhuis
<derek.buitenhuis at gmail.com> wrote:
> On 7/4/2017 9:33 PM, Michael Niedermayer wrote:
>> Fixes: out of array access
>> Fixes: 2451/clusterfuzz-testcase-minimized-4781613957251072
>>
>> Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
>> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
>> ---
>>  libavcodec/hevcdec.c | 8 ++++----
>>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> Is it possible this will cause any regressions with mid-stream parameter change
> handling?
>

The patch seems fine to me, mid-stream extradata should be populated
to avctx when its used by a frame, not when its parsed.
The export during parsing should only be done on codec open so initial
values are populated to the context right away.

- Hendrik