[FFmpeg-devel] [PATCH 2/4] avformat/concatdec: Check filename length before use

Nicolas George george at nsup.org
Thu Oct 29 17:09:57 EET 2020


Michael Niedermayer (12020-10-28):
> Fixes: out array read
> Fixes: 26610/clusterfuzz-testcase-minimized-ffmpeg_dem_CONCAT_fuzzer-5631838049271808
> 
> Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
>  libavformat/concatdec.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

This is not the first time such a fix is proposed. If strcmp() does not
behave intelligently, then we need an extra function for the task, but
duplicating the test all over the place is not a good idea.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20201029/a41825eb/attachment.sig>


More information about the ffmpeg-devel mailing list