On Sun, Jul 21, 2019 at 11:27:44AM +0200, Paul B Mahol wrote: > On 7/21/19, Michael Niedermayer <[email protected]> wrote: > > On Mon, Jul 08, 2019 at 01:18:04AM +0200, Michael Niedermayer wrote: > >> Fixes: out of array access > >> Fixes: > >> 15649/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_VORBIS_fuzzer-5729191309344768 > >> > >> Found-by: continuous fuzzing process > >> https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > >> Signed-off-by: Michael Niedermayer <[email protected]> > >> --- > >> libavcodec/vorbisdec.c | 6 ++++-- > >> 1 file changed, 4 insertions(+), 2 deletions(-) > > > > will apply > > Are you sure returning that value is correct approach? > > Correct value to return is usually INVALIDDATA.
the only call of this function i see does not use the value if it is
negative and uses INVALIDDATA instead.
ret = floor->decode(vc, &floor->data, floor_ptr[i]);
if (ret < 0) {
av_log(vc->avctx, AV_LOG_ERROR, "Invalid codebook in
vorbis_floor_decode.\n");
return AVERROR_INVALIDDATA;
}
but ill replace the code in this patch by an explicit AVERROR_INVALIDDATA
before pushing
Thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
