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 491444B1D9 for ; Mon, 1 Jul 2024 20:49:54 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id F0DB068D84E; Mon, 1 Jul 2024 23:49:51 +0300 (EEST) Received: from mail-lj1-f181.google.com (mail-lj1-f181.google.com [209.85.208.181]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 81E3C68D741 for ; Mon, 1 Jul 2024 23:49:46 +0300 (EEST) Received: by mail-lj1-f181.google.com with SMTP id 38308e7fff4ca-2ebe6495aedso36725021fa.0 for ; Mon, 01 Jul 2024 13:49:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1719866985; x=1720471785; darn=ffmpeg.org; h=user-agent:in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date:from:to:cc :subject:date:message-id:reply-to; bh=kFky3kWx+hCIHHZGHJv7GvVhzF2cuMuOoiqapd3Wxbw=; b=USe2tBBl1TCWtwrhkzrgAe4XdFT7adpUQ33ZjnelxlWX8qUUZdQKBdmBWjZh4HTSB2 buuY/PmkPfYY4/DbEH1lHzr9vBorA0MkK8otSrvXxoRz+824ZnRYmprEpBmN51ROaPPo Ej/sjHbnC/r7dnySHPw+1vbYZOq8CDV9hDeKQ9jw2KXpnDL2Xy3fyVKta1N+ayg5V7H5 oWtIenh5TH8iNn7Fmawq6ELc11ReSyPNsFCgZ20tE/4Le+SButIkbuon4hCcMgJGbAUA kx8TB3ayGUvoIRPRSFVVXQb40t0D1kl3pFadnTTvpyUoqaT9NeBYQgJj+am5MUdV+1e4 ay6g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1719866985; x=1720471785; h=user-agent:in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=kFky3kWx+hCIHHZGHJv7GvVhzF2cuMuOoiqapd3Wxbw=; b=n02wE+/Ft2if53hNo6vweEDWFFNPf2sQSvvAmXBXgql6RAdjSv/q8lzdnQyEjlbCfg 4Kye5Qo2/lPhJ2q+0WZT/PglYjMdmeKR4dbvsN8PPxbF87qrwIVJUxQhplREK2UCB/2B EUTfM1zEmXHnWbXLn6hJ7YvbOLRy5xM6O6DOAuA9Np0NE2STOIGzTrWwmkzGZQQEFHob xPNVptGXwTJllLJLDSphEq8TscmqyIa73yx8yPC3s9NbH7aYp1u/Zm7dwlyw21Wc6Fj4 jguv2iTjb0FI0reKPAlvfqpbgV3g9jHTlbxH0DewjBvhBLoLPgKUqw+DrU0kJBhc7SE3 yJog== X-Gm-Message-State: AOJu0YyumnOhSTkSsUqUxr0Np7vzXw6jK16L1MjWot8UMSOYuK3KI+2f CErWFTNnP2UeeBFtqfYmI2+R5zcMg6IEXA36nWDYJKQHG2hxYJCZMqVyUs0O+J4= X-Google-Smtp-Source: AGHT+IG3wRScWuZDViZFlrJdcxUsJJD4ih2CiAtlHuPw4K0W/1BxOuWPZCwGkZqKaxUuu7FnyJvANQ== X-Received: by 2002:a05:651c:508:b0:2ec:5200:a935 with SMTP id 38308e7fff4ca-2ee5e6e01cfmr47451301fa.40.1719866984606; Mon, 01 Jul 2024 13:49:44 -0700 (PDT) Received: from mariano (dynamic-adsl-84-220-189-10.clienti.tiscali.it. [84.220.189.10]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-58614f3d74dsm4781345a12.90.2024.07.01.13.49.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 01 Jul 2024 13:49:44 -0700 (PDT) Received: by mariano (Postfix, from userid 1000) id 3A2E0BFCE8; Mon, 1 Jul 2024 22:49:42 +0200 (CEST) Date: Mon, 1 Jul 2024 22:49:42 +0200 From: Stefano Sabatini To: FFmpeg development discussions and patches Message-ID: Mail-Followup-To: FFmpeg development discussions and patches , Marth64 References: <20240627030546.143368-1-marth64@proxyid.net> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20240627030546.143368-1-marth64@proxyid.net> User-Agent: Mutt/2.1.4 (2021-12-11) Subject: Re: [FFmpeg-devel] [PATCH] avformat/dvdvideodec: Fix duration logic with 1 chapter and validate chapter range 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 Cc: Marth64 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: On date Wednesday 2024-06-26 22:05:46 -0500, Marth64 wrote: > Chapters and duration are calculated together in dvdvideo demuxer. > Previous chapter calculation logic treated extraction of 1 chapter > using chapter_start and chapter_end switches incorrectly, returning > the duration of the entire title instead of just the segment. > > Fix the logic so that it calculates and returns the duration of the > chapter segment instead. Additionally, validate that chapter_end > exceeds chapter_start (except in the special case of 0). What is the meaning of the 0 special case? > > Signed-off-by: Marth64 > --- > libavformat/dvdvideodec.c | 15 ++++++++++++--- > 1 file changed, 12 insertions(+), 3 deletions(-) [...] Looks good to me. _______________________________________________ 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".