[FFmpeg-devel] [PATCH] lavu/get_video_buffer: also align data pointers (v2)

Anton Khirnov anton at khirnov.net
Sat Nov 16 06:25:40 EET 2024


Quoting Pavel Koshevoy (2024-11-15 20:32:21)
> This avoids unpleasant surprises to av_frame_get_buffer callers
> that explicitly specified 64-byte alignment and didn't get
> AVFrame.data pointers that are 64-byte aligned.
> 
> For example, see https://github.com/sekrit-twc/zimg/issues/212
> 
> Although the zscale issue has already been resolved by other means
> it would still be prudent to improve the behavior of av_frame_get_buffer
> to fix any unknown and future instances of similar issues.
> ---
>  libavutil/frame.c | 4 +++-
>  libavutil/frame.h | 7 ++++---
>  2 files changed, 7 insertions(+), 4 deletions(-)

I think this warrants a micro bump and an APIchanges entry, so external
callers can reliably depend on this.

Otherwise looks good.

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list