On Fri, Nov 06, 2020 at 12:10:21AM +0100, Michael Niedermayer wrote:
The reverted code split at block_align boundaries, but there was already code which splits at a hardcoded 4096 sample boundary.
reverting this seemed like the easiest fix but this is a RFC in case another solution is preferred
Fixes: out of array write Fixes: 26821/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ADPCM_SWF_fuzzer-5764465137811456
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg This reverts commit e9dd73d30d09043446ac6dd7b8ad31e557873852. --- libavcodec/adpcm.c | 15 ++------------- 1 file changed, 2 insertions(+), 13 deletions(-)
please disregard this patchset, this was sent by mistake to the wrong mailing list, this was intended all for ffmpeg-devel [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The worst form of inequality is to try to make unequal things equal. -- Aristotle