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

Michael Niedermayer michael at niedermayer.cc
Wed Jul 5 05:08:12 EEST 2017


On Wed, Jul 05, 2017 at 01:15:39AM +0200, Hendrik Leppkes wrote:
> 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.

applied

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170705/2e5c291f/attachment.sig>


More information about the ffmpeg-devel mailing list