Title: [210172] releases/WebKitGTK/webkit-2.14/Source/WebCore
Revision
210172
Author
[email protected]
Date
2016-12-27 08:41:46 -0800 (Tue, 27 Dec 2016)

Log Message

Merge r210056 - [Cairo] Conditional jump or move depends on uninitialised value(s)
https://bugs.webkit.org/show_bug.cgi?id=166058

Patch by Carlos Garcia Campos <[email protected]> on 2016-12-20
Reviewed by Michael Catanzaro.

Initialize x and y members of cairo_glyph_t struct before passing it to cairo_glyph_path.

* platform/graphics/cairo/FontCairo.cpp:
(WebCore::CairoGlyphToPathTranslator::path):

Modified Paths

Diff

Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/ChangeLog (210171 => 210172)


--- releases/WebKitGTK/webkit-2.14/Source/WebCore/ChangeLog	2016-12-27 16:39:27 UTC (rev 210171)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/ChangeLog	2016-12-27 16:41:46 UTC (rev 210172)
@@ -1,3 +1,15 @@
+2016-12-20  Carlos Garcia Campos  <[email protected]>
+
+        [Cairo] Conditional jump or move depends on uninitialised value(s)
+        https://bugs.webkit.org/show_bug.cgi?id=166058
+
+        Reviewed by Michael Catanzaro.
+
+        Initialize x and y members of cairo_glyph_t struct before passing it to cairo_glyph_path.
+
+        * platform/graphics/cairo/FontCairo.cpp:
+        (WebCore::CairoGlyphToPathTranslator::path):
+
 2016-11-13  Fujii Hironori  <[email protected]>
 
         [HarfBuzz] HarfBuzzShaper should not assume numGlyphs is greater than 0

Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/platform/graphics/cairo/FontCairo.cpp (210171 => 210172)


--- releases/WebKitGTK/webkit-2.14/Source/WebCore/platform/graphics/cairo/FontCairo.cpp	2016-12-27 16:39:27 UTC (rev 210171)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/platform/graphics/cairo/FontCairo.cpp	2016-12-27 16:41:46 UTC (rev 210172)
@@ -243,6 +243,7 @@
 
     cairo_glyph_t cairoGlyph;
     cairoGlyph.index = m_glyphBuffer.glyphAt(m_index);
+    cairoGlyph.x = cairoGlyph.y = 0;
     cairo_set_scaled_font(path.platformPath()->context(), m_fontData->platformData().scaledFont());
     cairo_glyph_path(path.platformPath()->context(), &cairoGlyph, 1);
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to