Hi Cosmin,
Cosmin Stejerean via ffmpeg-devel <ffmpeg-devel@ffmpeg.org> schrieb am Sa.,
25. Nov. 2023, 21:39:
> Fixes #10688
>
> Signed-off-by: Cosmin Stejerean <cos...@cosmin.at>
> ---
> libavfilter/vf_bwdif.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/libavfilter/vf_bwdif.c b/libavfilter/vf_bwdif.c
> index 137cd5ef13..bce11c39f7 100644
> --- a/libavfilter/vf_bwdif.c
> +++ b/libavfilter/vf_bwdif.c
> @@ -197,6 +197,18 @@ static int config_props(AVFilterLink *link)
> }
>
> yadif->csp = av_pix_fmt_desc_get(link->format);
> +
> + if (yadif->csp->nb_components > 1) {
> + int w_chroma, h_chroma;
> + h_chroma = AV_CEIL_RSHIFT(link->h, yadif->csp->log2_chroma_h);
> + w_chroma = AV_CEIL_RSHIFT(link->w, yadif->csp->log2_chroma_w);
> +
> + if (w_chroma < 3 || h_chroma < 4) {
> + av_log(ctx, AV_LOG_ERROR, "Video with planes less than 3
> columns or 4 lines is not supported\n");
> + return AVERROR(EINVAL);
> + }
> + }
> +
>
Thanks for your quick patch.
Could you please make the size check for all components and remove the old
one to avoid having two size checks in a row?
Regards,
Thomas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
