190645 - valgrind warnings in nsCSSRendering::FillPolygon

Status: VERIFIED DUPLICATE of bug 189723

(Core :: Layout, defect)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Valgrind spits out a lot of:

==16525== Conditional jump or move depends on uninitialised value(s)
==16525==    at 0x469487DC: nsCSSRendering::FillPolygon(nsIRenderingContext&,
nsPoint const*, int, nsRect*)
(/builds2/clean3/mozilla/layout/html/style/src/nsCSSRendering.cpp:244)

which can be fixed, I think, by checking that GetPenMode succeeded.

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: patch

untested

Comment 2

[Andrew Schultz]

identical patch in dupe "[valgrind] Use of uninitialized value in
nsCSSRendering::FillPolygon"

*** This bug has been marked as a duplicate of 189723 ***

Comment 3

[timeless]

vrfy dupe