From patchwork Thu Dec 23 07:19:12 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Gyan Doshi X-Patchwork-Id: 32849 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a6b:cd86:0:0:0:0:0 with SMTP id d128csp7041728iog; Wed, 22 Dec 2021 23:19:56 -0800 (PST) X-Google-Smtp-Source: ABdhPJwvK242Jxx4/P9yYY5BpqRMy6OaYmNWRC7Mgq4yzKOoNfX2Y0sW0wdXdFo4LZeER8xVs+ve X-Received: by 2002:a17:906:3e09:: with SMTP id k9mr995160eji.104.1640243995849; Wed, 22 Dec 2021 23:19:55 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1640243995; cv=none; d=google.com; s=arc-20160816; b=bAqXaGiGPuFtbLv7jeWeCrqGv/p7LMdswFhUlz6Zc5rdFhW46V+HGpAYVlm+M0Cnvg 78WEYWumrVfUCPAPmAjiksdm5oUc3Gt9KOooLW1/ld5YfiyI/+N2Pax5xzgiph6x7haN vfNmlv+w3ONsHWgUtKiKIKv3cUhoRyYYxmecMafq1hQc59SB4Xhycy3w8ZYw6npu9+58 x62QB8luTO+h4/47LXoTF4UqjmLrAY4ayWoQmJ9fnG5bgZbnfUjAgUHRqV1G8S4VQcAK sl8F4A2jXJigoSf7asPqt37AitlrviTMFqg/1Gc+yz2NXAGiOsoNIKVO+/8xz1u49ptL DBaw== 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:message-id:date:to:from :delivered-to; bh=K+YEGC477ggyNfupuTLJza4rtPCJYUIcy6fC0CTgAUc=; b=EYar+c3SJvBsVt5haIt3GxRCOqfK9U/3gdQXfSk4zWXGxE/2pYEI2FZ7If9AXonrk4 XdxSzfNid7bPjgI1duwmUGpnj/BygNQZBJd8x2ELgOz/Gm/ED+IbTOoNOECCQtosl3Dt Og2DDutykGMqfAd4SERaSzmmRZokp3/qaPDq2/QFE7lOHL4KDkOK7XUYxQipncGqtw5y ih8/KepMjj/zeBV36Ic2FlsonrZbLsw+X74+Unl6s0ZEmCdQ3xEIwcEE4znz26B9Hw7E D5SN7STKehnUNFoaJaUgGQOsDBHscVDyfm2nxTpZ+lHVL0lcy0h2lA0cvU3qsfjJhh2x oBLw== 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 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id a19si1864845edr.373.2021.12.22.23.19.55; Wed, 22 Dec 2021 23:19:55 -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 Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 35ECC68B111; Thu, 23 Dec 2021 09:19:52 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mout-p-103.mailbox.org (mout-p-103.mailbox.org [80.241.56.161]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 3E8C868ADF2 for ; Thu, 23 Dec 2021 09:19:45 +0200 (EET) Received: from smtp2.mailbox.org (smtp2.mailbox.org [IPv6:2001:67c:2050:105:465:1:2:0]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mout-p-103.mailbox.org (Postfix) with ESMTPS id 4JKM5X39p7zQk9s for ; Thu, 23 Dec 2021 08:19:44 +0100 (CET) X-Virus-Scanned: amavisd-new at heinlein-support.de From: Gyan Doshi To: ffmpeg-devel@ffmpeg.org Date: Thu, 23 Dec 2021 12:49:12 +0530 Message-Id: <20211223071913.50635-1-ffmpeg@gyani.pro> MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH v3 1/2] avformat/mov: add validate_box_size 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: H/QzRgAJ3F0L Helper function to check if stored box size is correct and looks to be fully available. --- libavformat/mov.c | 33 +++++++++++++++++++++++++++++++++ 1 file changed, 33 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 2aed6e80ef..e223e95e2e 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -80,6 +80,39 @@ static int mov_read_mfra(MOVContext *c, AVIOContext *f); static int64_t add_ctts_entry(MOVCtts** ctts_data, unsigned int* ctts_count, unsigned int* allocated_size, int count, int duration); +/** Check if the box size meets the requirements passed in limit and constraint_type. + * If input avio_size is valid, it checks if box size appears to be available. + * + * constraint_type may be + * 0 if the box size has to be exactly equal to limit + * -1 if the box size has to be at most limit + * 1 if the box size has to be at least limit + * + * Returns 0 if size meets requirements. + */ +static int validate_box_size(MOVContext *c, MOVAtom atom, AVIOContext *pb, + int64_t pos, int64_t limit, int constraint_type) +{ + int size_fit; + int64_t input_size = avio_size(pb); + + if (input_size > 0 && + input_size - pos < atom.size) { + av_log(c->fc, AV_LOG_ERROR, "Box %s is truncated\n", av_fourcc2str(atom.type)); + return AVERROR_INVALIDDATA; + } + + av_assert0(FFABS(constraint_type) <= 1); + + switch(constraint_type) { + case 0: size_fit = atom.size == limit; break; + case -1: size_fit = atom.size <= limit; break; + case 1: size_fit = atom.size >= limit; break; + } + + return !size_fit; +} + static int mov_metadata_track_or_disc_number(MOVContext *c, AVIOContext *pb, unsigned len, const char *key) {