From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id F02AC4720B for ; Thu, 30 Nov 2023 00:23:34 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 16E7B68CE33; Thu, 30 Nov 2023 02:23:31 +0200 (EET) Received: from a27-44.smtp-out.us-west-2.amazonses.com (a27-44.smtp-out.us-west-2.amazonses.com [54.240.27.44]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 930C9688158 for ; Thu, 30 Nov 2023 02:23:23 +0200 (EET) To: =?UTF-8?Q?ffmpeg-devel=40ffmpeg=2Eorg?= Date: Thu, 30 Nov 2023 00:23:20 +0000 Mime-Version: 1.0 References: <20231130002316.73504-1-cosmin@cosmin.at> X-Mailer: Amazon WorkMail Thread-Index: AQHaIyNrRPKJ9BURSnGRNaEm4d8hOw== Thread-Topic: [PATCH v2 0/3] consider chroma subsampling for bwdif (including CUDA and Vulkan) X-Original-Mailer: git-send-email 2.42.1 X-Wm-Sent-Timestamp: 1701303799 Message-ID: <0101018c1d9bd369-81383354-8caa-4736-ba47-6a70b92d3f4d-000000@us-west-2.amazonses.com> Feedback-ID: 1.us-west-2.An468LAV0jCjQDrDLvlZjeAthld7qrhZr+vow8irkvU=:AmazonSES X-SES-Outgoing: 2023.11.30-54.240.27.44 Subject: [FFmpeg-devel] [PATCH v2 0/3] consider chroma subsampling for bwdif (including CUDA and Vulkan) X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Cosmin Stejerean via ffmpeg-devel Reply-To: FFmpeg development discussions and patches Cc: =?UTF-8?Q?Cosmin_Stejerean?= Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: From: Cosmin Stejerean This fixes the issue reported in #10688. In v2 the checks for chroma plane are combined together and the fixes are extended to cover the Vulkan and CUDA implementations of bwdif. I have not had a chance to replicate the issue on CUDA or Vulkan but based on the dimension checks in those filters I believe they would have the same out of bounds issue. I kept the changes in separate patches since I've only properly tested the first one. I'll work on testing the CUDA and Vulkan implementations as well but meanwhile wanted to get some feedback on the overall approach. Cosmin Stejerean (3): avfilter/vf_bwdif: consider chroma subsampling when enforcing minimum dimensions avfilter/vf_bwdif_cuda: consider chroma subsampling when enforcing minimum dimensions avfilter/vf_bwdif_vulkan: consider chroma subsampling when enforcing minimum dimensions libavfilter/vf_bwdif.c | 13 ++++++++++--- libavfilter/vf_bwdif_cuda.c | 13 ++++++++++--- libavfilter/vf_bwdif_vulkan.c | 14 ++++++++++---- 3 files changed, 30 insertions(+), 10 deletions(-) -- 2.42.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".