From patchwork Wed Nov 17 03:04:15 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Wu Jianhua X-Patchwork-Id: 31449 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a6b:d206:0:0:0:0:0 with SMTP id q6csp105634iob; Tue, 16 Nov 2021 19:04:38 -0800 (PST) X-Google-Smtp-Source: ABdhPJx8hqiccmMqFYbvkKlY68b1tnKEW2H8MC9z8W+Dn0JZM7cfjW1SEDzissW+bnxHKq77L6nI X-Received: by 2002:a17:907:90c3:: with SMTP id gk3mr17279536ejb.282.1637118278530; Tue, 16 Nov 2021 19:04:38 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1637118278; cv=none; d=google.com; s=arc-20160816; b=MxiF7It2Ymt5LcHBL+32nTGQi5OM/JeFRIBlTeJGF9Fz5Zls3fB7pTZZpES62zYhiD 9BL7WvyrXdb8gfaR7P1jhhxMf6uHKKe3C3qbcTqpmCXOSs4Tlp54Npf4TEbJ97MdZ30X cbEjoEVDEAbyME5Nb93f0SbjmiamsbjId3jgWSM0J4Sq4/48CyMLKdozCz/Fhpo63sQ9 hST5g43MDfX/bDrrayb1qIKdGH77lHpjzDSXqo8gz/i2bfoqlXLr3HlLjwdHaMsM4c6Q cQayRs+ULX+s+Yxc3DaAaD7jq7DRM92zDTnbVDgkO4+T4ZEqfIiqQb32ardNevtc7yCK Nz+g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:mime-version:cc:reply-to :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:subject:message-id:date:to:from:delivered-to; bh=IIBfzW/Cz3PqIhAOGldbVgJTqkCN5IkhBjVIkzQhdnM=; b=SYB0E8yR2DcRV/s8kXqi7evZEaQZcdQtOVyNuCCh2ddjLIMEm/RZ6zazv0rttH83ud /u/DXdjeJs7vHxFx3KsCfPwS53rX5ML5ZIVO3QrgOphq5ZQRf8EQqAY5NXgo0ijVnMro ec0ZQW1X7pBfYGb21u1fOxU0d2yv6omBoQPcpBrYxVjB+Kou1ByOvWqBeTvIC/F0xMCd NLnc60wux97zqCkrIfzsF6QjrN+BELe+aMWPJBQzcssn4W3v8BKqvUCLaQI3GzRBPUWf jBhxPMQRkoguM7tq1r8c7uHYYYAloaWEBpUMOW5s7rRm63gWw+No/lXNgq/G8jl3OOH8 R4Bw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id zo7si36042710ejb.448.2021.11.16.19.04.34; Tue, 16 Nov 2021 19:04:38 -0800 (PST) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 205C768ACE7; Wed, 17 Nov 2021 05:04:30 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id D536D680BAD for ; Wed, 17 Nov 2021 05:04:22 +0200 (EET) X-IronPort-AV: E=McAfee;i="6200,9189,10170"; a="297292947" X-IronPort-AV: E=Sophos;i="5.87,240,1631602800"; d="scan'208";a="297292947" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Nov 2021 19:04:19 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.87,240,1631602800"; d="scan'208";a="506725077" Received: from otc-skl-e5-server.sh.intel.com ([10.239.43.106]) by orsmga008.jf.intel.com with ESMTP; 16 Nov 2021 19:04:18 -0800 From: Wu Jianhua To: ffmpeg-devel@ffmpeg.org Date: Wed, 17 Nov 2021 11:04:15 +0800 Message-Id: <20211117030415.81482-1-jianhua.wu@intel.com> X-Mailer: git-send-email 2.17.1 Subject: [FFmpeg-devel] [PATCH] avfilter/gblur_vulkan: fix memory leak 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: , Reply-To: FFmpeg development discussions and patches Cc: Wu Jianhua MIME-Version: 1.0 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: xBZ7FbC0ICyb Whether failed or not, the block of codes labeled with fail should be always run to ensure the av_free(kernel_def) is executed. Signed-off-by: Wu Jianhua --- libavfilter/vf_gblur_vulkan.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavfilter/vf_gblur_vulkan.c b/libavfilter/vf_gblur_vulkan.c index b9e2c4fbc7..d4f391b25d 100644 --- a/libavfilter/vf_gblur_vulkan.c +++ b/libavfilter/vf_gblur_vulkan.c @@ -296,8 +296,6 @@ static av_cold int init_filter(AVFilterContext *ctx, AVFrame *in) s->initialized = 1; - return 0; - fail: av_free(kernel_def); return err;