[FFmpeg-devel] [PATCH] lavfi/zscale: fix tmp ptr alignment for zimg_filter_graph_process

[James Almer]

On 11/9/2024 1:37 PM, Pavel Koshevoy wrote:
> ---
>   libavfilter/vf_zscale.c | 8 ++++++--
>   1 file changed, 6 insertions(+), 2 deletions(-)
> 
> diff --git a/libavfilter/vf_zscale.c b/libavfilter/vf_zscale.c
> index 4ba059064b..87fe691cfd 100644
> --- a/libavfilter/vf_zscale.c
> +++ b/libavfilter/vf_zscale.c
> @@ -750,7 +750,9 @@ static int filter_slice(AVFilterContext *ctx, void *data, int job_nr, int n_jobs
>       }
>       if (!s->graph[job_nr])
>           return AVERROR(EINVAL);
> -    ret = zimg_filter_graph_process(s->graph[job_nr], &src_buf, &dst_buf, s->tmp[job_nr], 0, 0, 0, 0);
> +    ret = zimg_filter_graph_process(s->graph[job_nr], &src_buf, &dst_buf,
> +                                    (uint8_t *)FFALIGN((uintptr_t)s->tmp[job_nr], ZIMG_ALIGNMENT),

Does s->tmp[job_nr] have enough extra bytes for this?

> +                                    0, 0, 0, 0);
>       if (ret)
>           return print_zimg_error(ctx);
>   
> @@ -765,7 +767,9 @@ static int filter_slice(AVFilterContext *ctx, void *data, int job_nr, int n_jobs
>   
>           if (!s->alpha_graph[job_nr])
>               return AVERROR(EINVAL);
> -        ret = zimg_filter_graph_process(s->alpha_graph[job_nr], &src_buf, &dst_buf, s->tmp[job_nr], 0, 0, 0, 0);
> +        ret = zimg_filter_graph_process(s->alpha_graph[job_nr], &src_buf, &dst_buf,
> +                                        (uint8_t *)FFALIGN((uintptr_t)s->tmp[job_nr], ZIMG_ALIGNMENT),
> +                                        0, 0, 0, 0);
>           if (ret)
>               return print_zimg_error(ctx);
>       }

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 495 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20241109/6f7d7154/attachment.sig>