1
0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2025-01-29 22:00:58 +02:00

avutil/frame: use the maximum compile time supported alignment for strides

This puts lavu frame buffer allocator helpers in sync with lavc's decoder frame
buffer allocator's STRIDE_ALIGN define.

Remove the comment about av_cpu_max_align() while at it as using it is not
ideal when CPU flags can be changed mid process.

Should fix ticket #11116.

Signed-off-by: James Almer <jamrial@gmail.com>
This commit is contained in:
James Almer 2024-07-31 09:54:02 -03:00
parent 792a9979eb
commit 210740b4ed

View File

@ -166,6 +166,8 @@ void av_frame_free(AVFrame **frame)
av_freep(frame);
}
#define ALIGN (HAVE_SIMD_ALIGN_64 ? 64 : 32)
static int get_video_buffer(AVFrame *frame, int align)
{
const AVPixFmtDescriptor *desc = av_pix_fmt_desc_get(frame->format);
@ -182,7 +184,7 @@ static int get_video_buffer(AVFrame *frame, int align)
if (!frame->linesize[0]) {
if (align <= 0)
align = 32; /* STRIDE_ALIGN. Should be av_cpu_max_align() */
align = ALIGN;
for (int i = 1; i <= align; i += i) {
ret = av_image_fill_linesizes(frame->linesize, frame->format,