Attachment 340629 Details for Bug 167112 – Patch

[patch] Patch

bug-167112-20180517131335.patch (text/plain), 11.54 KB, created by youenn fablet on 2018-05-17 13:13:36 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: youenn fablet

Created: 2018-05-17 13:13:36 PDT

Size: 11.54 KB

patch

obsolete

>Subversion Revision: 231744
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index b5f815b62318ebf91f8b6085deb8a9cebb91f3d7..126ea44d92f6c938066beeb4c87a40e4af2f1bea 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,21 @@
>+2018-05-17  Youenn Fablet  <youenn@apple.com>
>+
>+        Header values too restricted
>+        https://bugs.webkit.org/show_bug.cgi?id=167112
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Tests: imported/w3c/web-platform-tests/fetch/api/headers/header-values-normalize.html
>+               imported/w3c/web-platform-tests/fetch/api/headers/header-values.html
>+
>+        Moving header values and names from DOMString to ByteString as per spec.
>+        Updating WebCore::isValidHTTPHeaderValue as per https://fetch.spec.whatwg.org/#terminology-headers.
>+
>+        * Modules/fetch/FetchHeaders.idl:
>+        * platform/network/HTTPParsers.cpp:
>+        (WebCore::isValidHTTPHeaderValue):
>+        * xml/XMLHttpRequest.idl:
>+
> 2018-05-14  Youenn Fablet  <youenn@apple.com>
> 
>         readableStreamDefaultControllerError should return early if stream is not readable
>diff --git a/Source/WebCore/platform/network/HTTPParsers.cpp b/Source/WebCore/platform/network/HTTPParsers.cpp
>index 893905f1f6f80e275d1a9a1705e1c935f469d419..9920339072370a2647940a278d1a57e71bccec0b 100644
>--- a/Source/WebCore/platform/network/HTTPParsers.cpp
>+++ b/Source/WebCore/platform/network/HTTPParsers.cpp
>@@ -85,7 +85,7 @@ static inline bool skipEquals(const String& str, unsigned &pos)
>     return skipWhiteSpace(str, pos) && str[pos++] == '=' && skipWhiteSpace(str, pos);
> }
> 
>-// True if a value present, incrementing pos to next space or semicolon, if any.  
>+// True if a value present, incrementing pos to next space or semicolon, if any.
> // Note: might return pos == str.length().
> static inline bool skipValue(const String& str, unsigned& pos)
> {
>@@ -110,7 +110,7 @@ bool isValidReasonPhrase(const String& value)
>     return true;
> }
> 
>-// See RFC 7230, Section 3.2.3.
>+// See https://fetch.spec.whatwg.org/#concept-header until RFC 7230 gets fixed.
> bool isValidHTTPHeaderValue(const String& value)
> {
>     UChar c = value[0];
>@@ -121,7 +121,8 @@ bool isValidHTTPHeaderValue(const String& value)
>         return false;
>     for (unsigned i = 0; i < value.length(); ++i) {
>         c = value[i];
>-        if (c == 0x7F || c > 0xFF || (c < 0x20 && c != '\t'))
>+        ASSERT(c <= 0xFF);
>+        if (c == 0x00 || c == 0x0A || c == 0x0D)
>             return false;
>     }
>     return true;
>diff --git a/LayoutTests/imported/w3c/ChangeLog b/LayoutTests/imported/w3c/ChangeLog
>index 9e28d6190e6d594e05c1343b9b6081f55e3192d9..3aa0389903144ccf3e9a6638b65029037351d743 100644
>--- a/LayoutTests/imported/w3c/ChangeLog
>+++ b/LayoutTests/imported/w3c/ChangeLog
>@@ -1,3 +1,15 @@
>+2018-05-17  Youenn Fablet  <youenn@apple.com>
>+
>+        Header values too restricted
>+        https://bugs.webkit.org/show_bug.cgi?id=167112
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Rebased from WPT repository.
>+
>+        * web-platform-tests/fetch/api/headers/header-values-normalize-exptected.txt: Added.
>+        * web-platform-tests/fetch/api/headers/header-values-expected.txt: Added.
>+
> 2018-05-14  Youenn Fablet  <youenn@apple.com>
> 
>         readableStreamDefaultControllerError should return early if stream is not readable
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/fetch/api/headers/header-values-expected.txt b/LayoutTests/imported/w3c/web-platform-tests/fetch/api/headers/header-values-expected.txt
>index 675a84494ae46acce8129a637d66e1e96469852d..0d5533e25c8735f2c87797717299ef099503df8d 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/fetch/api/headers/header-values-expected.txt
>+++ b/LayoutTests/imported/w3c/web-platform-tests/fetch/api/headers/header-values-expected.txt
>@@ -5,6 +5,6 @@ PASS XMLHttpRequest with value x%0Ax needs to throw
> PASS fetch() with value x%0Ax needs to throw 
> PASS XMLHttpRequest with value x%0Dx needs to throw 
> PASS fetch() with value x%0Dx needs to throw 
>-FAIL XMLHttpRequest with all valid values The string did not match the expected pattern.
>-FAIL fetch() with all valid values Type error
>+PASS XMLHttpRequest with all valid values 
>+PASS fetch() with all valid values 
> 
>diff --git a/LayoutTests/imported/w3c/web-platform-tests/fetch/api/headers/header-values-normalize-expected.txt b/LayoutTests/imported/w3c/web-platform-tests/fetch/api/headers/header-values-normalize-expected.txt
>index 1a63326ce295e59a3cf70f27328d207a2307d2ce..27842294f71f076cd6ef8737dc448d8ca50450c7 100644
>--- a/LayoutTests/imported/w3c/web-platform-tests/fetch/api/headers/header-values-normalize-expected.txt
>+++ b/LayoutTests/imported/w3c/web-platform-tests/fetch/api/headers/header-values-normalize-expected.txt
>@@ -1,64 +1,64 @@
> 
> PASS XMLHttpRequest with value %00 
> PASS fetch() with value %00 
>-FAIL XMLHttpRequest with value %01 The string did not match the expected pattern.
>-FAIL fetch() with value %01 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %02 The string did not match the expected pattern.
>-FAIL fetch() with value %02 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %03 The string did not match the expected pattern.
>-FAIL fetch() with value %03 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %04 The string did not match the expected pattern.
>-FAIL fetch() with value %04 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %05 The string did not match the expected pattern.
>-FAIL fetch() with value %05 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %06 The string did not match the expected pattern.
>-FAIL fetch() with value %06 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %07 The string did not match the expected pattern.
>-FAIL fetch() with value %07 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %08 The string did not match the expected pattern.
>-FAIL fetch() with value %08 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>+PASS XMLHttpRequest with value %01 
>+PASS fetch() with value %01 
>+PASS XMLHttpRequest with value %02 
>+PASS fetch() with value %02 
>+PASS XMLHttpRequest with value %03 
>+PASS fetch() with value %03 
>+PASS XMLHttpRequest with value %04 
>+PASS fetch() with value %04 
>+PASS XMLHttpRequest with value %05 
>+PASS fetch() with value %05 
>+PASS XMLHttpRequest with value %06 
>+PASS fetch() with value %06 
>+PASS XMLHttpRequest with value %07 
>+PASS fetch() with value %07 
>+PASS XMLHttpRequest with value %08 
>+PASS fetch() with value %08 
> PASS XMLHttpRequest with value %09 
> PASS fetch() with value %09 
> PASS XMLHttpRequest with value %0A 
> PASS fetch() with value %0A 
> PASS XMLHttpRequest with value %0D 
> PASS fetch() with value %0D 
>-FAIL XMLHttpRequest with value %0E The string did not match the expected pattern.
>-FAIL fetch() with value %0E promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %0F The string did not match the expected pattern.
>-FAIL fetch() with value %0F promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %10 The string did not match the expected pattern.
>-FAIL fetch() with value %10 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %11 The string did not match the expected pattern.
>-FAIL fetch() with value %11 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %12 The string did not match the expected pattern.
>-FAIL fetch() with value %12 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %13 The string did not match the expected pattern.
>-FAIL fetch() with value %13 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %14 The string did not match the expected pattern.
>-FAIL fetch() with value %14 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %15 The string did not match the expected pattern.
>-FAIL fetch() with value %15 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %16 The string did not match the expected pattern.
>-FAIL fetch() with value %16 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %17 The string did not match the expected pattern.
>-FAIL fetch() with value %17 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %18 The string did not match the expected pattern.
>-FAIL fetch() with value %18 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %19 The string did not match the expected pattern.
>-FAIL fetch() with value %19 promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %1A The string did not match the expected pattern.
>-FAIL fetch() with value %1A promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %1B The string did not match the expected pattern.
>-FAIL fetch() with value %1B promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %1C The string did not match the expected pattern.
>-FAIL fetch() with value %1C promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %1D The string did not match the expected pattern.
>-FAIL fetch() with value %1D promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %1E The string did not match the expected pattern.
>-FAIL fetch() with value %1E promise_test: Unhandled rejection with value: object "TypeError: Type error"
>-FAIL XMLHttpRequest with value %1F The string did not match the expected pattern.
>-FAIL fetch() with value %1F promise_test: Unhandled rejection with value: object "TypeError: Type error"
>+PASS XMLHttpRequest with value %0E 
>+PASS fetch() with value %0E 
>+PASS XMLHttpRequest with value %0F 
>+PASS fetch() with value %0F 
>+PASS XMLHttpRequest with value %10 
>+PASS fetch() with value %10 
>+PASS XMLHttpRequest with value %11 
>+PASS fetch() with value %11 
>+PASS XMLHttpRequest with value %12 
>+PASS fetch() with value %12 
>+PASS XMLHttpRequest with value %13 
>+PASS fetch() with value %13 
>+PASS XMLHttpRequest with value %14 
>+PASS fetch() with value %14 
>+PASS XMLHttpRequest with value %15 
>+PASS fetch() with value %15 
>+PASS XMLHttpRequest with value %16 
>+PASS fetch() with value %16 
>+PASS XMLHttpRequest with value %17 
>+PASS fetch() with value %17 
>+PASS XMLHttpRequest with value %18 
>+PASS fetch() with value %18 
>+PASS XMLHttpRequest with value %19 
>+PASS fetch() with value %19 
>+PASS XMLHttpRequest with value %1A 
>+PASS fetch() with value %1A 
>+PASS XMLHttpRequest with value %1B 
>+PASS fetch() with value %1B 
>+PASS XMLHttpRequest with value %1C 
>+PASS fetch() with value %1C 
>+PASS XMLHttpRequest with value %1D 
>+PASS fetch() with value %1D 
>+PASS XMLHttpRequest with value %1E 
>+PASS fetch() with value %1E 
>+PASS XMLHttpRequest with value %1F 
>+PASS fetch() with value %1F 
> PASS XMLHttpRequest with value %20 
> PASS fetch() with value %20 
>

Flags:

ews-watchlist: commit-queue-

Actions: View | Formatted Diff | Diff

Attachments on bug 167112: 299159 | 299160 | 299165 | 299166 | 299167 | 299168 | 299246 | 340629 | 340639 | 340643 | 342114