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 DC46C4B8A0 for ; Tue, 18 Jun 2024 00:49:42 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id C762A68D7AC; Tue, 18 Jun 2024 03:49:39 +0300 (EEST) Received: from mail-pf1-f171.google.com (mail-pf1-f171.google.com [209.85.210.171]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 891BC68D30C for ; Tue, 18 Jun 2024 03:49:33 +0300 (EEST) Received: by mail-pf1-f171.google.com with SMTP id d2e1a72fcca58-70417a6c328so3760298b3a.1 for ; Mon, 17 Jun 2024 17:49:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1718671770; x=1719276570; darn=ffmpeg.org; h=content-transfer-encoding:mime-version:message-id:date:subject:to :from:from:to:cc:subject:date:message-id:reply-to; bh=acghzN+vnjlb6Eu3EdQJFKOuX4Fxl3BjManfTm5LqxQ=; b=FS/jks+QVH9ZcX6vGrGCtLWSrtSefpWuaiDBs71VbY5nqTnzoaX29u7tFgI4UXtNtl FBroe5Nfw5g2bo0BBOnHHERftmQcK9OwP3YGRaa3N30ya5+BjWSTWUiLmhuuTI3wBLmc eM7334/33Im/TtAB9Mq5hDlDPjVLzZd48miuBsB12m6Bh/J48FB6ku7P7ECSG03fBsxN TBmaqTE5IDz2nPVZfNdLLgOCGObXeQTtiBZl3fzgQ1qHPU3BPk8e0uqUmBo9bJgoQ1RC LsepanlRWh/+WVMJhw7gjNV8R3Rn4Gtu1+cx2VsGsCWMyYF2LgrI3L8q5wP4CJNDZuKf Kcog== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1718671770; x=1719276570; 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=acghzN+vnjlb6Eu3EdQJFKOuX4Fxl3BjManfTm5LqxQ=; b=BCP8SMiJsyaKsU4B6yzmQt31UDzEeBQ3t2x5nPGJXhyBj94cg+lIxDyv9LSpqJu6cc WDUinSm7f0yD7gWig1zp6s9eixzbkltC2xzk/OgrB8Tmzo7Vu2JGk31UtplaoZwwllPL xXYs2JiRBHJHdTl1Lq82BCJa+JLS+glsiSUKVCcDMoFrLRRpRVwaYDSBLU19VX7rcvdm OZsZdJ+OlS7AzOLzUY6McD5b4LXxeY0L1kwxGm4OS3B5Oa4Fmahz5Tzxzji6139sy3Xj i2FxzG9Aw+BehwbrqEsQHpjRTP6AiJpWMEmFLnvaglG7bGuBLUfloibN7/cnxUokVCrV eexA== X-Gm-Message-State: AOJu0YyAHyqclISBGifI7jAxQPSgDwcsbzchO0pAzgXtIlxL/iNBXYkJ ZgTPr1PIrzkuZwgdKCi+yb8Dz9/f2rGLluRXNLB1SWfGb8476Zx+XjziEA== X-Google-Smtp-Source: AGHT+IGgU6TEy3vpukx/91IBFBrILUuFBurfLiZsCSZ1q5zodCOftaOlB+ydxmzSI3f4ehKGV3VIYw== X-Received: by 2002:a05:6a20:3caa:b0:1b7:406c:107d with SMTP id adf61e73a8af0-1bae7f14310mr11181173637.27.1718671770226; Mon, 17 Jun 2024 17:49:30 -0700 (PDT) Received: from localhost.localdomain ([190.194.167.233]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-1f860bd8f40sm72509785ad.154.2024.06.17.17.49.29 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 17 Jun 2024 17:49:29 -0700 (PDT) From: James Almer To: ffmpeg-devel@ffmpeg.org Date: Mon, 17 Jun 2024 21:49:01 -0300 Message-ID: <20240618004901.4835-1-jamrial@gmail.com> X-Mailer: git-send-email 2.45.2 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH] avformat/iamf_parse: add missing padding to AAC extradata 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: Fixes: out of array access Fixes: 68863/clusterfuzz-testcase-minimized-ffmpeg_dem_IAMF_fuzzer-4833546039525376 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: James Almer --- libavformat/iamf_parse.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavformat/iamf_parse.c b/libavformat/iamf_parse.c index 312090b247..ff1b6cc75b 100644 --- a/libavformat/iamf_parse.c +++ b/libavformat/iamf_parse.c @@ -92,13 +92,16 @@ static int aac_decoder_config(IAMFCodecConfig *codec_config, if (left <= 0) return AVERROR_INVALIDDATA; - codec_config->extradata = av_malloc(left); + // We pad extradata here because avpriv_mpeg4audio_get_config2() needs it. + codec_config->extradata = av_malloc(left + AV_INPUT_BUFFER_PADDING_SIZE); if (!codec_config->extradata) return AVERROR(ENOMEM); codec_config->extradata_size = avio_read(pb, codec_config->extradata, left); if (codec_config->extradata_size < left) return AVERROR_INVALIDDATA; + memset(codec_config->extradata + codec_config->extradata_size, 0, + AV_INPUT_BUFFER_PADDING_SIZE); ret = avpriv_mpeg4audio_get_config2(&cfg, codec_config->extradata, codec_config->extradata_size, 1, logctx); -- 2.45.2 _______________________________________________ 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".