[webkit-changes] [WebKit/WebKit] f98a7b: Don't allow form-feed (U+000C) as a WebVTT signatu...

Anne van Kesteren noreply at github.com
Fri Oct 20 23:06:00 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: f98a7bee4cc3cd9f75b1e4e2f99d7414117c23d3
      https://github.com/WebKit/WebKit/commit/f98a7bee4cc3cd9f75b1e4e2f99d7414117c23d3
  Author: Anne van Kesteren <annevk at annevk.nl>
  Date:   2023-10-20 (Fri, 20 Oct 2023)

  Changed paths:
    M LayoutTests/imported/w3c/web-platform-tests/webvtt/parsing/file-parsing/signature-invalid-expected.txt
    M Source/WebCore/html/track/WebVTTParser.cpp

  Log Message:
  -----------
  Don't allow form-feed (U+000C) as a WebVTT signature separator
https://bugs.webkit.org/show_bug.cgi?id=260390
rdar://114118340

Reviewed by Eric Carlson.

As U+000A and U+000D are already taken care of by BufferedLineReader
we can switch this isASCIIWhitespace to isTabOrSpace and thereby avoid
ignoring an incorrect U+000C.

(Note that other identifiers in WebVTT, such as "STYLE", do allow a
trailing U+000C, so we are not changing those, even though they are also
redundantly checking for U+000A and U+000D.)

* LayoutTests/imported/w3c/web-platform-tests/webvtt/parsing/file-parsing/signature-invalid-expected.txt:
* Source/WebCore/html/track/WebVTTParser.cpp:
(WebCore::WebVTTParser::hasRequiredFileIdentifier):

Canonical link: https://commits.webkit.org/269619@main




More information about the webkit-changes mailing list