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 61B1B43F8C for ; Sun, 23 Oct 2022 23:07:10 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id DC91868BC4F; Mon, 24 Oct 2022 02:07:07 +0300 (EEST) Received: from mail-oi1-f180.google.com (mail-oi1-f180.google.com [209.85.167.180]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id D5F8A68BA4A for ; Mon, 24 Oct 2022 02:07:01 +0300 (EEST) Received: by mail-oi1-f180.google.com with SMTP id p127so9252007oih.9 for ; Sun, 23 Oct 2022 16:07:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:to:from:from:to:cc:subject:date:message-id :reply-to; bh=V4alIhIEQe+8aQK/o7PkkVan7mBGKML+ABgmvQeEQBg=; b=I3Teed8+TQUAfKH8Jm5YYPwpwLhTKxg4OBA46mrOLvxzTubxdb84B1RRzacQSfIB8Z 8gNtA70Ny6icbjjpKrJPTTf9v4H8ZzYPWqY9DvZwUzVfS/7jn3Q8dhSA8BCFmzkgsGsF SqYDnWPAwUOjCV9vf5lU60mYb3yHK/DDPPctw7TdFbHZV2YxVU7mzm05H7gSyRsYA7L2 Dil0qVQotbbN54zOYq1T5fZQobbOPkBqE6gu0InIYVJ3G3ZOckUbMZ7/h3vQCgCsGY9A YDq2EbJ1ajr7I4ZSyW+z6c0xgof1JlX0mS2R5fnF45Ao661S82G09t9NhFuTIeLGN/Z9 MLgQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=V4alIhIEQe+8aQK/o7PkkVan7mBGKML+ABgmvQeEQBg=; b=w7F36OR/HrTfYq7TLcUPjOlAETN7nIXEuMN6qxcUwNLA9l6e0IOhqlE/rWoDJNKTER UCt10r2qImlbkh7JkMpx3iQw3jVBRayZtVrXB5cGsp3tiW3IG11ntrwfN65fV7VJvkfa gLlfeYxNrSF35NN+COpV/x0w497FzsLuyvhgJfK+r3Wlnqbcxe3pxC88E3JlMFMkppMy X7KoiQjLlU7n8eIChPqVJPWAeL9AXinEJWdybEvRwUFL1VFtDUb9yD4MUiIRnngU32XE azOEYxZdhgzwSEeclqJ9SO3Se8nR5q07ILlK53gY+3LoC59TpYr9HVhqNrnd0O5SyrSb 2feA== X-Gm-Message-State: ACrzQf32GJyke77C1KgpydXsF8JA77c+zriHrZ61JKXGOlkphyKBXgtv CoI0XTHRX9p6Gj1xB51QVuV+D3ph5rg= X-Google-Smtp-Source: AMsMyM610NEbSkBCV/TodW9K4q9QGQK7ixiKUESAXluiJrARZSxfr2qaH8g3uEa75qCiFQ8KtZgxuw== X-Received: by 2002:a05:6808:1404:b0:355:4cd4:b10b with SMTP id w4-20020a056808140400b003554cd4b10bmr16312569oiv.207.1666566419997; Sun, 23 Oct 2022 16:06:59 -0700 (PDT) Received: from localhost.localdomain ([191.97.187.183]) by smtp.gmail.com with ESMTPSA id z36-20020a056870d6a400b0012b298699dbsm13550191oap.1.2022.10.23.16.06.59 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 23 Oct 2022 16:06:59 -0700 (PDT) From: James Almer To: ffmpeg-devel@ffmpeg.org Date: Sun, 23 Oct 2022 20:04:19 -0300 Message-Id: <20221023230419.1278-1-jamrial@gmail.com> X-Mailer: git-send-email 2.37.3 In-Reply-To: <20221022210226.2200-1-jamrial@gmail.com> References: <20221022210226.2200-1-jamrial@gmail.com> MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH 2/6] avcodec/adts_parsed: allow passing a pre allocated AACADTSHeaderInfo to avpriv_adts_header_parse() 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: Code freeing the struct on failure is kept for backwards compatibility, but should ideally be removed in the next major bump, and the existing lavf user adapted. Signed-off-by: James Almer --- libavcodec/adts_header.c | 2 ++ libavcodec/adts_parser.c | 12 +++++++++--- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/libavcodec/adts_header.c b/libavcodec/adts_header.c index ff4efafbf7..00fa0a5a99 100644 --- a/libavcodec/adts_header.c +++ b/libavcodec/adts_header.c @@ -32,6 +32,8 @@ int ff_adts_header_parse(GetBitContext *gbc, AACADTSHeaderInfo *hdr) int size, rdb, ch, sr; int aot, crc_abs; + memset(hdr, 0, sizeof(*hdr)); + if (get_bits(gbc, 12) != 0xfff) return AAC_AC3_PARSE_ERROR_SYNC; diff --git a/libavcodec/adts_parser.c b/libavcodec/adts_parser.c index 4a1a8fd5f4..f2e155fc99 100644 --- a/libavcodec/adts_parser.c +++ b/libavcodec/adts_parser.c @@ -47,24 +47,30 @@ int avpriv_adts_header_parse(AACADTSHeaderInfo **phdr, const uint8_t *buf, size_ { #if CONFIG_ADTS_HEADER int ret = 0; + int allocated = 0; GetBitContext gb; if (!phdr || !buf || size < AV_AAC_ADTS_HEADER_SIZE) return AVERROR_INVALIDDATA; - *phdr = av_mallocz(sizeof(AACADTSHeaderInfo)); + if (!*phdr) { + allocated = 1; + *phdr = av_mallocz(sizeof(AACADTSHeaderInfo)); + } if (!*phdr) return AVERROR(ENOMEM); ret = init_get_bits8(&gb, buf, AV_AAC_ADTS_HEADER_SIZE); if (ret < 0) { - av_freep(phdr); + if (allocated) + av_freep(phdr); return ret; } ret = ff_adts_header_parse(&gb, *phdr); if (ret < 0) { - av_freep(phdr); + if (allocated) + av_freep(phdr); return ret; } -- 2.37.3 _______________________________________________ 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".