From patchwork Tue Apr 2 03:17:59 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: James Almer X-Patchwork-Id: 47724 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6a20:9f96:b0:1a3:b6bb:3029 with SMTP id mm22csp1113319pzb; Mon, 1 Apr 2024 20:18:21 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCUXtNze2YYal/y/eAh+LNRKVS9cYrRPDyXxHO+wyDfcbTH3D2ldR7uoDBbQFGzRT9DfUSA83yG1fA1k8lOZvJFww4vAeQqzXI+tNA== X-Google-Smtp-Source: AGHT+IE10X4yZFSLu9BRY+tNYNGUON+A6kNJfhGgMABJOX3DTGqgST/AQPasLJ0t+iywY5hzykdo X-Received: by 2002:a17:906:250c:b0:a4e:4ed4:5efb with SMTP id i12-20020a170906250c00b00a4e4ed45efbmr4708593ejb.3.1712027901627; Mon, 01 Apr 2024 20:18:21 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1712027901; cv=none; d=google.com; s=arc-20160816; b=hGbkXoLvTIONIQgMG8gpInTc4iksYWi2AFP7RiaB7+MNHpbyKZEO4t/yy+JxvObXfZ UzHKu2n0zcGDbs2gNRqGFEQbyiFyBDcq/iaLIugh3eQRATFHgrOGHIhmkUxmPhfY5YLR 8saFJS+8MR6PjVjal5ImpF6kjKfzWEYCkLn4vvCHGfeDRY4/AulTRAI3UK2OJ4V7yd8q FZeGLfuEkht/HVE3TLokdEnnWs9DqxakvuQTXINGdH7Dd5X4L8P8OvheSWKTh9Z8eCEJ GbwknepKH2CSceLZ4ABA5DW+PiR/EkFEc1/myfro5GuvXHfuWS6RHgg03Jc6pQqfJ31P RHOg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:reply-to:list-subscribe :list-help:list-post:list-archive:list-unsubscribe:list-id :precedence:subject:mime-version:references:in-reply-to:message-id :date:to:from:dkim-signature:delivered-to; bh=gv5JpmFZWVLkD5hlGpgeZmiMTLHM418yFu5fErXtEY8=; fh=YOA8vD9MJZuwZ71F/05pj6KdCjf6jQRmzLS+CATXUQk=; b=AGjE5PBaPpfRTRDtkgQ/4EOQqpTG5+mpYz6P/Mn1zPZAciOD1WH8irvtX+AXM3xWrv ggqnC03hVENHdsA23QmEsRKoUwb4Wm69dTDf3/xm7RY3WFZn+9FJGwa1ddYUFZGetL8E 2fGt32eKACOK/pNxLHgM7bFUXycQoKjnoEw40m/msguSrdfuWSEkEZ3j1DWY7pgoKf4V bD1/6tvWeTQiodlHLDN7wxgXkTdgy4anKJ/y/UsmPu1+sRFppLy3KhlYw6UHCvXNw1s/ LoFEyA2jjnnkm/03MwPeFh2264jN04I9+VkuZpXoOXBx9Iu4Xja15E0GNajhoyNCyn3k 1UWQ==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com header.s=20230601 header.b=ByCUXQXM; 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=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id ko10-20020a170907986a00b00a4e513652a0si2888485ejc.516.2024.04.01.20.18.21; Mon, 01 Apr 2024 20:18:21 -0700 (PDT) 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; dkim=neutral (body hash did not verify) header.i=@gmail.com header.s=20230601 header.b=ByCUXQXM; 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=QUARANTINE dis=NONE) header.from=gmail.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 2F56F68D0CE; Tue, 2 Apr 2024 06:18:11 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-pl1-f173.google.com (mail-pl1-f173.google.com [209.85.214.173]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 444DE68D0A6 for ; Tue, 2 Apr 2024 06:18:05 +0300 (EEST) Received: by mail-pl1-f173.google.com with SMTP id d9443c01a7336-1e0edd0340fso42001305ad.2 for ; Mon, 01 Apr 2024 20:18:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1712027883; x=1712632683; darn=ffmpeg.org; 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=QOtJAnhNuq1t+a0g+EZpL3rBqoyqNx8iYzruPICtfLE=; b=ByCUXQXM+hi9LVzwvkAC1HiRT8p0/6kWWHw9fW5F15RuHWd/VzpfNv/Q5S2vOG58wp nQ9dN4E98A9DP2iRzUugzvbgjpC0oYmDU26pR66bJtU4wr2zlDqwcIfQrAa6r+hqc7Do VmZYlXVgrAc0O+UaCTRtoLfgG7Q4o/a0P1FSuYkQfKXgl92i/NMCnIz2NmC3vL7LhQBE bbXPKxZAmyiL6tzP8fQchtJHBWk3JQgdoEEriKNK/cDkDUQgEpOW++3c+9TYfFhfUoUj +RVwkY1dkTw/4NGwKFO/vWbVgNEJqc9+VN2w9Acvmu0NTHLcGrvsOT6D+AN5ya77EZYM 3JLg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1712027883; x=1712632683; 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=QOtJAnhNuq1t+a0g+EZpL3rBqoyqNx8iYzruPICtfLE=; b=Usuo0Cw9NBtELsqjogSa6Z+wTq/REvevoW5bHl6/f/ni7kfJ8AP74hE0RdxJ1B1kX7 pUcasXYdaunrajsU4WYM6xIbbvifkmXDWfDbVH56S1lcQV8RbtSOZ1qSd2oxggjpc4Nu 912RLu3iHDeGeoK2iUfhJ8OSyGuUOzfyg8meZWNJpq2VKF9PN/yLVx0aYSLQuC+AcOaE 2abYDS0rvsC9heTMXN0DvpTGkjg3QX8ceyCwagDH25sd6Eh/9gyAphRp+VbIXudlYz0W hIAS2YWTRpku+KbS1vWE+pylujyBUGiUg02GfhfaLPO1sWG7ztNuQfv74CV03RKZGXDx eHnQ== X-Gm-Message-State: AOJu0YylBIoUhnM87PkQO3XEzSipQqTpj+SR0uvoCmMHlD4ja9I3w9IE iq6R1a4Nky5rJMh8VHR1W5nO7GJhdcUQbKfMSsClU4SBhMgElV+fWWwKLiJN X-Received: by 2002:a17:902:ce84:b0:1e0:e85b:3389 with SMTP id f4-20020a170902ce8400b001e0e85b3389mr14809820plg.3.1712027882772; Mon, 01 Apr 2024 20:18:02 -0700 (PDT) Received: from localhost.localdomain ([190.194.167.233]) by smtp.gmail.com with ESMTPSA id g12-20020a170902fe0c00b001e26ba8882fsm408652plj.287.2024.04.01.20.18.01 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 01 Apr 2024 20:18:02 -0700 (PDT) From: James Almer To: ffmpeg-devel@ffmpeg.org Date: Tue, 2 Apr 2024 00:17:59 -0300 Message-ID: <20240402031800.7159-2-jamrial@gmail.com> X-Mailer: git-send-email 2.44.0 In-Reply-To: <20240402031800.7159-1-jamrial@gmail.com> References: <20240402031800.7159-1-jamrial@gmail.com> MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH 2/3] avformat/mov: don't read key_size bytes twice in the keys atom 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 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: v0OjhQ7Q3Nnp We only support mdta as type, yet we were not skipping other types, but rather reading key_size worth of bytes twice per entry. Signed-off-by: James Almer --- libavformat/mov.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 3273e2e89b..a935ef7326 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -5049,6 +5049,7 @@ static int mov_read_keys(MOVContext *c, AVIOContext *pb, MOVAtom atom) key_size -= 8; if (type != MKTAG('m','d','t','a')) { avio_skip(pb, key_size); + continue; } c->meta_keys[i] = av_mallocz(key_size + 1); if (!c->meta_keys[i])