[FFmpeg-devel] [PATCH] [h264] Use small padding with the checked bitstream reader.

Michael Niedermayer michael at niedermayer.cc
Sat Aug 17 21:04:49 EEST 2024


On Wed, Aug 14, 2024 at 04:32:36PM -0700, Dale Curtis wrote:
> MAX_MBPAIR_SIZE was added in 23f5cff92cdcfa55a735c458fcb5f95c0e0f3b1f
> to prevent CABAC/CAVLC overread issues. It adds 256kb of padding to
> RBSP allocations. AFAICT it seems unnecessary with the checked
> bitstream reader. Dropping this padding is a substantial memory
> improvement for constrained devices.
> 
> 782865bf3094e36cbb4bd9cfacda252307e6589d removed the small padding
> when AV_CODEC_FLAG2_FAST was set, but I don't have access to that
> fuzzer test case to check this patch. Does anyone have this for testing?

20978/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_H264_fuzzer-5746381832847360 sent privately

thx

[...]

-- 
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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240817/adfc5a00/attachment.sig>


More information about the ffmpeg-devel mailing list