[FFmpeg-devel] [PATCH 2/2] avcodec/hcadec: Check or bound indexes

[Michael Niedermayer]

On Wed, May 13, 2020 at 10:42:15PM +0200, Michael Niedermayer wrote:
> This causes indexes into scale_conversion_table to wrap around, alternatively they
> could be clipped, the table be enlarged or we can error out. I have not found a document that specifies
> what is the correct way to handle this
> 
> Fixes: out of array access
> Fixes: 21727/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_HCA_fuzzer-5752477891952640.fuzz
> 
> 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/hcadec.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

If someone has hca files which do not originate from a fuzzer, these would be
helpfull to test this

Thanks

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

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200514/fde806bd/attachment.sig>