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 0C46E45FEB for ; Fri, 28 Apr 2023 16:37:14 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 4A3C568C000; Fri, 28 Apr 2023 19:37:11 +0300 (EEST) Received: from mail-oa1-f52.google.com (mail-oa1-f52.google.com [209.85.160.52]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 087F868BF06 for ; Fri, 28 Apr 2023 19:37:05 +0300 (EEST) Received: by mail-oa1-f52.google.com with SMTP id 586e51a60fabf-18665c1776dso142439fac.2 for ; Fri, 28 Apr 2023 09:37:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1682699823; x=1685291823; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:from:to:cc:subject:date:message-id:reply-to; bh=dZFo5ReKBwZxjgqrdg+LXFZ4zLszDnCeLqe2U4RltkM=; b=ae5UvBF2QLJFpbVGRsDYppfGw59Vh31xhL1A40EV2zuMECDzxi78RUn/E9gvjXSpVP +N93NzOzLoDI45+Px7f+KZvV1fYYiKkU3QLG/XD40p4kO8aqAtyx3ljle/mOIo8MlB2E +QVpRMD+Q3bepJ7uGbLRDI2FG0tmRvJ3WDT4RZjAPR/CCvD8Iai+GUgCtKXglL2DDEyq 3/4Oyj32VsTfYP2z+CwB28tNfRUeWucNBu8m4tHCjVyYlNpgDgM9Lms1X2xC+GI8Qveb iXLGZ4bgw3tslxC9/tZQw34XngpOOzVOPF8hbKTBDlvUk4CGzsdi2+5tLljIAWaua6PJ aquw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1682699823; x=1685291823; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=dZFo5ReKBwZxjgqrdg+LXFZ4zLszDnCeLqe2U4RltkM=; b=Mr+9ivnmFJvkKCZ2+EuE2r+uAqx2+Y3o82OHNWNsRpDnkXNWXcwTHwQ0MPtW6omEqK OhA80U5/KeYp1AgGUpoG1tekxAkRGcZLouIkjR0LSTpw6CMHlP+C9mRbtIVOfpsQiL4q lpHUmrL6Kql16C9ssk4OAIHLgmpbn9qkgIEEAU/Pni49ocMnzCnZTHIJKStWjUI0y6pW QXJ3QaZ61wDPgqSf8TqD5k8YWvSGQs7j4lcxVlKq65qIXcbcYUZSLYdTAzb4irNEP9WN dHR5Mssz4nwA2i6dOUGwgpP0mZjgrVe7uvP5wyT48l5lw6MX/9dOpvuHPsuFiJvV+NPX 7hiQ== X-Gm-Message-State: AC+VfDw8g0aPUZNfzOLgoZGqaPWKJbLzbkOim1QBapgYH7rY9Ophobqa FlEnloH8u1mp1P/dFOgCW/FEcbXE+9U= X-Google-Smtp-Source: ACHHUZ4pTxQAwY1Kcn8yznwq066KcAn5rpiH5fuzTaMW0/3mo0ntytMaUeDlDzEsVXYfAMBOL5a8Xw== X-Received: by 2002:a05:6870:1a90:b0:18e:c8ee:1186 with SMTP id ef16-20020a0568701a9000b0018ec8ee1186mr3133619oab.19.1682699822888; Fri, 28 Apr 2023 09:37:02 -0700 (PDT) Received: from localhost.localdomain (host197.190-225-105.telecom.net.ar. [190.225.105.197]) by smtp.gmail.com with ESMTPSA id z16-20020a056871039000b0017264f96879sm7627763oaf.17.2023.04.28.09.37.01 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 28 Apr 2023 09:37:02 -0700 (PDT) From: James Almer To: ffmpeg-devel@ffmpeg.org Date: Fri, 28 Apr 2023 13:36:39 -0300 Message-Id: <20230428163639.1573-1-jamrial@gmail.com> X-Mailer: git-send-email 2.40.1 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH] fftools/ffmpeg_filter: fix leak of AVIOContext in read_binary() 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 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: It was only being freed on failure. Signed-off-by: James Almer --- fftools/ffmpeg_filter.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/fftools/ffmpeg_filter.c b/fftools/ffmpeg_filter.c index b26160b375..161ea9c866 100644 --- a/fftools/ffmpeg_filter.c +++ b/fftools/ffmpeg_filter.c @@ -408,11 +408,13 @@ static int read_binary(const char *path, uint8_t **data, int *len) *len = fsize; - return 0; + ret = 0; fail: avio_close(io); - av_freep(data); - *len = 0; + if (ret < 0) { + av_freep(data); + *len = 0; + } return ret; } -- 2.40.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".