[webkit-changes] [236109] releases/WebKitGTK/webkit-2.22

Tue, 18 Sep 2018 02:17:01 -0700

Title: [236109] releases/WebKitGTK/webkit-2.22
Revision
236109
Author
[email protected]
Date
2018-09-18 02:16:33 -0700 (Tue, 18 Sep 2018)

Log Message

Merge r235416 - Null pointer deref in WidthIterator
https://bugs.webkit.org/show_bug.cgi?id=188993

Reviewed by Brent Fulgham.

Source/WebCore:

Test: fast/text/rtl-justification.html

We simply need to guard glyphBuffer like we do in the rest of the function.

* platform/graphics/WidthIterator.cpp:
(WebCore::WidthIterator::advanceInternal):

LayoutTests:

* fast/text/rtl-justification-expected.html: Added.
* fast/text/rtl-justification.html: Added.

Modified Paths

Added Paths

Diff

Modified: releases/WebKitGTK/webkit-2.22/LayoutTests/ChangeLog (236108 => 236109)


--- releases/WebKitGTK/webkit-2.22/LayoutTests/ChangeLog	2018-09-18 09:16:25 UTC (rev 236108)
+++ releases/WebKitGTK/webkit-2.22/LayoutTests/ChangeLog	2018-09-18 09:16:33 UTC (rev 236109)
@@ -1,3 +1,13 @@
+2018-08-27  Myles C. Maxfield  <[email protected]>
+
+        Null pointer deref in WidthIterator
+        https://bugs.webkit.org/show_bug.cgi?id=188993
+
+        Reviewed by Brent Fulgham.
+
+        * fast/text/rtl-justification-expected.html: Added.
+        * fast/text/rtl-justification.html: Added.
+
 2018-08-27  Youenn Fablet  <[email protected]>
 
         Various IndexDB tests abandon documents

Added: releases/WebKitGTK/webkit-2.22/LayoutTests/fast/text/rtl-justification-expected.html (0 => 236109)


--- releases/WebKitGTK/webkit-2.22/LayoutTests/fast/text/rtl-justification-expected.html	                        (rev 0)
+++ releases/WebKitGTK/webkit-2.22/LayoutTests/fast/text/rtl-justification-expected.html	2018-09-18 09:16:33 UTC (rev 236109)
@@ -0,0 +1,42 @@
+<!DOCTYPE html>
+<html>
+<head>
+<meta charset="utf-8">
+</head>
+<body>
+This test passes if there is no crash.
+<div dir="rtl" style="width: 200px; text-align: justify;">
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+</div>
+</body>
+</html>

Added: releases/WebKitGTK/webkit-2.22/LayoutTests/fast/text/rtl-justification.html (0 => 236109)


--- releases/WebKitGTK/webkit-2.22/LayoutTests/fast/text/rtl-justification.html	                        (rev 0)
+++ releases/WebKitGTK/webkit-2.22/LayoutTests/fast/text/rtl-justification.html	2018-09-18 09:16:33 UTC (rev 236109)
@@ -0,0 +1,42 @@
+<!DOCTYPE html>
+<html>
+<head>
+<meta charset="utf-8">
+</head>
+<body>
+This test passes if there is no crash.
+<div dir="rtl" style="width: 200px; text-align: justify;">
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+תמיד להגיב על
+</div>
+</body>
+</html>

Modified: releases/WebKitGTK/webkit-2.22/Source/WebCore/ChangeLog (236108 => 236109)


--- releases/WebKitGTK/webkit-2.22/Source/WebCore/ChangeLog	2018-09-18 09:16:25 UTC (rev 236108)
+++ releases/WebKitGTK/webkit-2.22/Source/WebCore/ChangeLog	2018-09-18 09:16:33 UTC (rev 236109)
@@ -1,3 +1,17 @@
+2018-08-27  Myles C. Maxfield  <[email protected]>
+
+        Null pointer deref in WidthIterator
+        https://bugs.webkit.org/show_bug.cgi?id=188993
+
+        Reviewed by Brent Fulgham.
+
+        Test: fast/text/rtl-justification.html
+
+        We simply need to guard glyphBuffer like we do in the rest of the function.
+
+        * platform/graphics/WidthIterator.cpp:
+        (WebCore::WidthIterator::advanceInternal):
+
 2018-08-27  Youenn Fablet  <[email protected]>
 
         Various IndexDB tests abandon documents

Modified: releases/WebKitGTK/webkit-2.22/Source/WebCore/platform/graphics/WidthIterator.cpp (236108 => 236109)


--- releases/WebKitGTK/webkit-2.22/Source/WebCore/platform/graphics/WidthIterator.cpp	2018-09-18 09:16:25 UTC (rev 236108)
+++ releases/WebKitGTK/webkit-2.22/Source/WebCore/platform/graphics/WidthIterator.cpp	2018-09-18 09:16:33 UTC (rev 236109)
@@ -345,7 +345,7 @@
         previousCharacter = character;
     }
 
-    if (leftoverJustificationWidth) {
+    if (glyphBuffer && leftoverJustificationWidth) {
         if (m_forTextEmphasis)
             glyphBuffer->add(lastFontData->zeroWidthSpaceGlyph(), lastFontData, leftoverJustificationWidth, m_run.length() - 1);
         else

_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to