Summary: | [Cairo] Conditional jump or move depends on uninitialised value(s) | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||
Component: | Platform | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bugs-noreply, commit-queue, mcatanzaro, mmaxfield | ||||
Priority: | P2 | Keywords: | Gtk | ||||
Version: | WebKit Local Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Carlos Garcia Campos
2016-12-20 06:20:45 PST
Created attachment 297518 [details]
Patch
It reminds me of https://bugs.freedesktop.org/show_bug.cgi?id=91271, which is very annoying when valgrinding as it happens every time when loading any page. But clearly this is a separate issue. Comment on attachment 297518 [details] Patch Clearing flags on attachment: 297518 Committed r210056: <http://trac.webkit.org/changeset/210056> All reviewed patches have been landed. Closing bug. Comment on attachment 297518 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=297518&action=review > Source/WebCore/platform/graphics/cairo/FontCairo.cpp:246 > cairo_glyph_t cairoGlyph; > cairoGlyph.index = m_glyphBuffer.glyphAt(m_index); > + cairoGlyph.x = cairoGlyph.y = 0; These could have been written like this: cairo_glyph_t cairoGlyph = { m_glyphBuffer.glyphAt(m_index), 0, 0 }; (In reply to comment #5) > Comment on attachment 297518 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=297518&action=review > > > Source/WebCore/platform/graphics/cairo/FontCairo.cpp:246 > > cairo_glyph_t cairoGlyph; > > cairoGlyph.index = m_glyphBuffer.glyphAt(m_index); > > + cairoGlyph.x = cairoGlyph.y = 0; > > These could have been written like this: > > cairo_glyph_t cairoGlyph = { m_glyphBuffer.glyphAt(m_index), 0, 0 }; Indeed! Done in r210321, thanks. |