Title: [210056] trunk/Source/WebCore
- Revision
- 210056
- Author
- commit-qu...@webkit.org
- Date
- 2016-12-20 23:48:37 -0800 (Tue, 20 Dec 2016)
Log Message
[Cairo] Conditional jump or move depends on uninitialised value(s)
https://bugs.webkit.org/show_bug.cgi?id=166058
Patch by Carlos Garcia Campos <cgar...@igalia.com> 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: trunk/Source/WebCore/ChangeLog (210055 => 210056)
--- trunk/Source/WebCore/ChangeLog 2016-12-21 07:07:13 UTC (rev 210055)
+++ trunk/Source/WebCore/ChangeLog 2016-12-21 07:48:37 UTC (rev 210056)
@@ -1,3 +1,15 @@
+2016-12-20 Carlos Garcia Campos <cgar...@igalia.com>
+
+ [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-12-20 Myles C. Maxfield <mmaxfi...@apple.com>
Skia lighter wght variation looks bolder than regular
Modified: trunk/Source/WebCore/platform/graphics/cairo/FontCairo.cpp (210055 => 210056)
--- trunk/Source/WebCore/platform/graphics/cairo/FontCairo.cpp 2016-12-21 07:07:13 UTC (rev 210055)
+++ trunk/Source/WebCore/platform/graphics/cairo/FontCairo.cpp 2016-12-21 07:48:37 UTC (rev 210056)
@@ -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
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes