[FFmpeg-devel] [PATCH] avcodec/libx265: Don't copy A53 closed captions by default

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Tue Jun 11 11:05:54 EEST 2024


Tomas Härdin:
> tis 2024-06-11 klockan 09:42 +0200 skrev Andreas Rheinhardt:
>> The SEI handling of libx265 is buggy and can easily lead
>> to memory corruption: It reuses certain buffers, but when
>> reusing them it presumes that it is enough for these buffers
>> to exist and does not check whether they are actually large
>> enough to hold what is intended to be stored in them.*
>>
>> Our users are exposed to this because forwarding A53 CC data
>> is enabled by default. Change this to make it disabled
>> by default.
>>
>> "Fixes" tickets #9666, #10411, #11052 and (presumably) #10906.
> 
> Shouldn't users use non-buggy versions of libx26? I've had people ask
> about CC, and I'm sure many users would be annoyed at them suddenly
> breaking. I suggest complaining loudly at compile time and/or when
> loading libx265 instead

Non-buggy versions of libx265? People use what they have because it exists.

- Andreas



More information about the ffmpeg-devel mailing list