compiz::X11::PendingConfigureEvent::dump(): Conditional jump or move depends on uninitialised value(s) / Use of uninitialised value of size 8

Bug #921451 reported by Daniel van Vugt on 2012-01-25
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Compiz Core
Medium
Daniel van Vugt
compiz (Ubuntu)
Undecided
Unassigned

Bug Description

Valgrind reports lots of these errors:

==32649== Conditional jump or move depends on uninitialised value(s)
==32649== at 0x5FC207C: vfprintf (vfprintf.c:1619)
==32649== by 0x606DD8F: __vsnprintf_chk (vsnprintf_chk.c:65)
==32649== by 0x40EF07: compLogMessage(char const*, CompLogLevel, char const*, ...) (stdio2.h:79)
==32649== by 0x4E75127: compiz::X11::PendingConfigureEvent::dump() (window.cpp:2270)
==32649== by 0x4E76C97: compiz::X11::PendingEventQueue::add(boost::shared_ptr<compiz::X11::PendingEvent>) (window.cpp:2238)
==32649== by 0x4E7F561: PrivateWindow::reconfigureXWindow(unsigned int, XWindowChanges*) (window.cpp:3496)
==32649== by 0x4E7FDBF: CompWindow::configureXWindow(unsigned int, XWindowChanges*) (window.cpp:3745)
==32649== by 0x4E80C01: PrivateWindow::updateSize() (window.cpp:4267)
==32649== by 0x4E8398A: CompWindow::map() (window.cpp:1624)
==32649== by 0x4E83EB5: CompWindow::CompWindow(unsigned long, XWindowAttributes&, PrivateWindow*) (window.cpp:6410)
==32649== by 0x4E841CF: CoreWindow::manage(unsigned long, XWindowAttributes&) (window.cpp:6262)
==32649== by 0x4E6DF1A: CompScreen::init(char const*) (screen.cpp:4742)
==32649==
==32649== Use of uninitialised value of size 8
==32649== at 0x5FBE31B: _itoa_word (_itoa.c:195)
==32649== by 0x5FC07D6: vfprintf (vfprintf.c:1619)
==32649== by 0x606DD8F: __vsnprintf_chk (vsnprintf_chk.c:65)
==32649== by 0x40EF07: compLogMessage(char const*, CompLogLevel, char const*, ...) (stdio2.h:79)
==32649== by 0x4E75127: compiz::X11::PendingConfigureEvent::dump() (window.cpp:2270)
==32649== by 0x4E76C97: compiz::X11::PendingEventQueue::add(boost::shared_ptr<compiz::X11::PendingEvent>) (window.cpp:2238)
==32649== by 0x4E7F561: PrivateWindow::reconfigureXWindow(unsigned int, XWindowChanges*) (window.cpp:3496)
==32649== by 0x4E7FDBF: CompWindow::configureXWindow(unsigned int, XWindowChanges*) (window.cpp:3745)
==32649== by 0x4E80C01: PrivateWindow::updateSize() (window.cpp:4267)
==32649== by 0x4E8398A: CompWindow::map() (window.cpp:1624)
==32649== by 0x4E83EB5: CompWindow::CompWindow(unsigned long, XWindowAttributes&, PrivateWindow*) (window.cpp:6410)
==32649== by 0x4E841CF: CoreWindow::manage(unsigned long, XWindowAttributes&) (window.cpp:6262)
==32649==

Related branches

Changed in compiz-core:
status: New → In Progress
assignee: nobody → Daniel van Vugt (vanvugt)
Daniel van Vugt (vanvugt) wrote :

It's also a bit scary that configure events are causing compiz to call vfprintf so frequently. But that's a different issue.

summary: - compLogMessage: Conditional jump or move depends on uninitialised
- value(s) / Use of uninitialised value of size 8
+ compiz::X11::PendingConfigureEvent::dump(): Conditional jump or move
+ depends on uninitialised value(s) / Use of uninitialised value of size 8
Changed in compiz-core:
importance: Undecided → Medium
Changed in compiz-core:
status: In Progress → Fix Committed
milestone: none → 0.9.7.0
Daniel van Vugt (vanvugt) wrote :

Fix released in:
compiz (1:0.9.7.0~bzr2995-0ubuntu1) precise; urgency=low

Changed in compiz (Ubuntu):
status: New → Fix Released
Changed in compiz-core:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers