Title: [280686] trunk/Source/WebCore
Revision
280686
Author
[email protected]
Date
2021-08-05 08:02:38 -0700 (Thu, 05 Aug 2021)

Log Message

Fix warning in HTTPParsers.parseStructuredFieldValue
https://bugs.webkit.org/show_bug.cgi?id=228815

Patch by Rob Buis <[email protected]> on 2021-08-05
Reviewed by Sam Weinig.

UChar is unsigned, so it is not needed to check that it is
zewro or greater.

* platform/network/HTTPParsers.cpp:
(WebCore::parseStructuredFieldValue):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (280685 => 280686)


--- trunk/Source/WebCore/ChangeLog	2021-08-05 14:34:41 UTC (rev 280685)
+++ trunk/Source/WebCore/ChangeLog	2021-08-05 15:02:38 UTC (rev 280686)
@@ -1,3 +1,16 @@
+2021-08-05  Rob Buis  <[email protected]>
+
+        Fix warning in HTTPParsers.parseStructuredFieldValue
+        https://bugs.webkit.org/show_bug.cgi?id=228815
+
+        Reviewed by Sam Weinig.
+
+        UChar is unsigned, so it is not needed to check that it is
+        zewro or greater.
+
+        * platform/network/HTTPParsers.cpp:
+        (WebCore::parseStructuredFieldValue):
+
 2021-08-05  Imanol Fernandez  <[email protected]>
 
         Fix XR related clang warnings in WPE

Modified: trunk/Source/WebCore/platform/network/HTTPParsers.cpp (280685 => 280686)


--- trunk/Source/WebCore/platform/network/HTTPParsers.cpp	2021-08-05 14:34:41 UTC (rev 280685)
+++ trunk/Source/WebCore/platform/network/HTTPParsers.cpp	2021-08-05 15:02:38 UTC (rev 280686)
@@ -648,7 +648,7 @@
                     } else if (header[index] == '\"') {
                         value = valueBuilder.toString();
                         break;
-                    } else if ((header[index] >= 0x00 && header[index] <= 0x1F) || (header[index] >= 0x7F && header[index] <= 0xFF)) // Not in VCHAR or SP.
+                    } else if (header[index] <= 0x1F || (header[index] >= 0x7F && header[index] <= 0xFF)) // Not in VCHAR or SP.
                         return std::nullopt;
                     else
                         valueBuilder.append(header[index]);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to