From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 1420D4E4A9 for ; Mon, 9 Jun 2025 23:51:16 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 7E35468CFA9; Tue, 10 Jun 2025 02:51:13 +0300 (EEST) Received: from smtpout1.mo533.mail-out.ovh.net (smtpout1.mo533.mail-out.ovh.net [51.210.94.138]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id D56F8688001 for ; Tue, 10 Jun 2025 02:51:06 +0300 (EEST) Received: from director1.derp.mail-out.ovh.net (director1.derp.mail-out.ovh.net [51.68.80.175]) by mo533.mail-out.ovh.net (Postfix) with ESMTPS id 4bGTG15tWtz1Tf9; Mon, 9 Jun 2025 23:51:05 +0000 (UTC) Received: from director1.derp.mail-out.ovh.net (director1.derp.mail-out.ovh.net. [127.0.0.1]) by director1.derp.mail-out.ovh.net (inspect_sender_mail_agent) with SMTP for ; Mon, 9 Jun 2025 23:51:05 +0000 (UTC) Received: from mta6.priv.ovhmail-u1.ea.mail.ovh.net (unknown [10.110.113.54]) by director1.derp.mail-out.ovh.net (Postfix) with ESMTPS id 4bGTG11snTz5xPW; Mon, 9 Jun 2025 23:51:05 +0000 (UTC) Received: from mailstore2.priv.ovhmail-u1.ea.mail.ovh.net (unknown [10.2.8.2]) by mta6.priv.ovhmail-u1.ea.mail.ovh.net (Postfix) with ESMTP id DCE1AD4402F; Mon, 9 Jun 2025 23:51:04 +0000 (UTC) Date: Mon, 9 Jun 2025 23:51:04 +0000 (UTC) From: Marcos Del Sol To: Tomas =?utf-8?Q?H=C3=A4rdin?= Message-ID: <711117349.249383674.1749513064741.JavaMail.zimbra@orca.pet> In-Reply-To: <5d3bd41e77e388369dd3a69469a57f1a1131e00e.camel@haerdin.se> References: <20250527102811.369474-1-marcos@orca.pet> <1210481741.184664378.1748342436436.JavaMail.zimbra@orca.pet> <204e8be99e97c2a9d56c5fbd0f51c4540452a436.camel@haerdin.se> <5d3bd41e77e388369dd3a69469a57f1a1131e00e.camel@haerdin.se> MIME-Version: 1.0 X-Originating-IP: [79.117.18.167] Thread-Topic: avformat/webvttdec: improve WebVTT parsing Thread-Index: gOCgaFlB7osCQuJByAA8OTGjLvW6ng== X-Ovh-Tracer-Id: 8316178189504239275 X-VR-SPAMSTATE: OK X-VR-SPAMSCORE: -100 X-VR-SPAMCAUSE: gggruggvucftvghtrhhoucdtuddrgeeffedrtddugdelleekucetufdoteggodetrfdotffvucfrrhhofhhilhgvmecuqfggjfdpvefjgfevmfevgfenuceurghilhhouhhtmecuhedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmnecujfgurhepfffhvfevkfgjfhfugggtgfhithesthejtgdttddtjeenucfhrhhomhepofgrrhgtohhsucffvghlucfuohhluceomhgrrhgtohhssehorhgtrgdrphgvtheqnecuggftrfgrthhtvghrnhephfdtteeigeegtdfgtdeufefhieettdekuedvgfehjeehvdejveegiefftdehheelnecukfhppeduvdejrddtrddtrddupdejledruddujedrudekrdduieejnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehinhgvthepuddvjedrtddrtddruddpmhgrihhlfhhrohhmpehmrghrtghoshesohhrtggrrdhpvghtpdhnsggprhgtphhtthhopedvpdhrtghpthhtohepfhhfmhhpvghgqdguvghvvghlsehffhhmphgvghdrohhrghdprhgtphhtthhopehgihhtsehhrggvrhguihhnrdhsvgdpoffvtefjohhsthepmhhoheeffegmpdhmohguvgepshhmthhpohhuth Subject: Re: [FFmpeg-devel] [PATCH] avformat/webvttdec: improve WebVTT parsing 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: 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: > The linked file does not follow the syntax specified in section 4 of > the WebVTT standard. Therefore it is not WebVTT. The section 6.1 on file parsing, says explicitely that processing should be aborted if, and only if: - The file is too small (point 4) - The file does not start with "WEBVTT" (point 5) - The character after "WEBVTT" is not a whitespace (point 6) - There is no data after "WEBVTT" (point 8 and 10) Note on the point 14, the main block handling loop, it not even once talks about aborting parsing. If an extraneous block is found, no action should be taken. It is just ignored and keeps processing the next ones. WebVTT is supposed to be an extensible format. Limiting to a small set of known values and silently aborting when anything new is introduced does not seem like the best option to me. Web browsers do not stop rendering pages when they see a new, unknown HTML tag or CSS option. _______________________________________________ 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".