uninitialized variable valgrind warning in gfxFontGroup::ForEachFontInternal

RESOLVED FIXED in mozilla1.9alpha1

Status

()

Core
Graphics
P2
normal
RESOLVED FIXED
11 years ago
10 years ago

People

(Reporter: dbaron, Assigned: dbaron)

Tracking

({valgrind})

Trunk
mozilla1.9alpha1
x86
Linux
valgrind
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [patch])

Attachments

(1 attachment)

Starting up, I'm seeing:

==25135== Conditional jump or move depends on uninitialised value(s)
==25135==    at 0x40788DA: gfxFontGroup::ForEachFontInternal(nsAString_internal const&, nsACString_internal const&, int, int (*)(nsAString_internal const&, nsACString_internal const&, nsACString_internal const&, void*), void*) (/builds/trunk/mozilla/gfx/thebes/src/gfxFont.cpp:174)
==25135==    by 0x40791EC: gfxFontGroup::ForEachFont(int (*)(nsAString_internal const&, nsACString_internal const&, nsACString_internal const&, void*), void*) (/builds/trunk/mozilla/gfx/thebes/src/gfxFont.cpp:67)
==25135==    by 0x4082634: gfxPangoFontGroup::gfxPangoFontGroup(nsAString_internal const&, gfxFontStyle const*) (/builds/trunk/mozilla/gfx/thebes/src/gfxPangoFonts.cpp:142)
...

Patch coming.
Created attachment 242921 [details] [diff] [review]
patch
Attachment #242921 - Flags: review?(vladimir)
Summary: valgrind warning in gfxFontGroup::ForEachFontInternal → uninitialized variable valgrind warning in gfxFontGroup::ForEachFontInternal
Status: NEW → ASSIGNED
Priority: -- → P2
Whiteboard: [patch]
Target Milestone: --- → mozilla1.9alpha

Updated

11 years ago
Attachment #242921 - Flags: review?(vladimir) → review+
Checked in to trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Wow, thank you for the fix, that is my mistake...
Keywords: valgrind
You need to log in before you can comment on or make changes to this bug.