Compiz Core

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
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Compiz Core
Fix Released
Medium
Daniel van Vugt
Compiz Core 0.9.7.0
compiz (Ubuntu)
Fix Released
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

lp:~vanvugt/compiz-core/fix-921451
Alan Griffiths: Approve
Diff: 249 lines (+30/-22)
4 files modified
plugins/place/src/place.cpp (+3/-1)
plugins/resize/src/resize.cpp (+4/-2)
plugins/switcher/src/switcher.cpp (+3/-1)
src/window.cpp (+20/-18)
lp:~compiz/compiz/ubuntu
lp:ubuntu/precise/compiz
Daniel van Vugt (vanvugt)
Changed in compiz-core:
status: New → In Progress
assignee: nobody → Daniel van Vugt (vanvugt)
Revision history for this message
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
Daniel van Vugt (vanvugt)
Changed in compiz-core:
status: In Progress → Fix Committed
milestone: none → 0.9.7.0
Revision history for this message
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
Daniel van Vugt (vanvugt)
Changed in compiz-core:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.