Compiz

resize leaks memory in multiple locations calling resize::CompScreenImpl::findWindow() -> operator new()

Bug #1097126 reported by Daniel van Vugt
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Compiz
Fix Released
High
Daniel van Vugt
Compiz 0.9.9.0
0.9.8
Won't Fix
High
Unassigned
compiz (Ubuntu)
Fix Released
High
Daniel van Vugt
Nominated for Quantal by Daniel van Vugt

Bug Description

==17459== 416 (160 direct, 256 indirect) bytes in 4 blocks are definitely lost in loss record 1,180 of 1,323
==17459== at 0x4C2AF8E: operator new(unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==17459== by 0x11DCEC2C: resize::CompScreenImpl::findWindow(unsigned long) (window-impl.h:71)
==17459== by 0x11DD9C2D: ResizeLogic::handleEvent(_XEvent*) (resize-logic.cpp:141)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459== by 0x11FFF9DE: RotateScreen::handleEvent(_XEvent*) (rotate.cpp:910)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459== by 0x12D3AE1A: PlaceScreen::handleEvent(_XEvent*) (place.cpp:209)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459== by 0x1488700C: PrivateScaleScreen::handleEvent(_XEvent*) (scale.cpp:1700)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459== by 0x152F0C8B: MoveScreen::handleEvent(_XEvent*) (move.cpp:634)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459==
==17459== 416 (160 direct, 256 indirect) bytes in 4 blocks are definitely lost in loss record 1,181 of 1,323
==17459== at 0x4C2AF8E: operator new(unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==17459== by 0x11DCEC2C: resize::CompScreenImpl::findWindow(unsigned long) (window-impl.h:71)
==17459== by 0x11DD6DD4: ResizeLogic::initiateResizeDefaultMode(CompAction*, unsigned int, std::vector<CompOption, std::allocator<CompOption> >&) (resize-logic.cpp:1538)
==17459== by 0x11DDA13C: ResizeLogic::handleEvent(_XEvent*) (resize-logic.cpp:199)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459== by 0x11FFF9DE: RotateScreen::handleEvent(_XEvent*) (rotate.cpp:910)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459== by 0x12D3AE1A: PlaceScreen::handleEvent(_XEvent*) (place.cpp:209)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459== by 0x1488700C: PrivateScaleScreen::handleEvent(_XEvent*) (scale.cpp:1700)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459== by 0x152F0C8B: MoveScreen::handleEvent(_XEvent*) (move.cpp:634)
==17459==
==17459== 416 (160 direct, 256 indirect) bytes in 4 blocks are definitely lost in loss record 1,182 of 1,323
==17459== at 0x4C2AF8E: operator new(unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==17459== by 0x11DCEC2C: resize::CompScreenImpl::findWindow(unsigned long) (window-impl.h:71)
==17459== by 0x11DD62B5: ResizeLogic::initiateResize(CompAction*, unsigned int, std::vector<CompOption, std::allocator<CompOption> >&, unsigned int) (resize-logic.cpp:1194)
==17459== by 0x11DD6F11: ResizeLogic::initiateResizeDefaultMode(CompAction*, unsigned int, std::vector<CompOption, std::allocator<CompOption> >&) (resize-logic.cpp:1553)
==17459== by 0x11DDA13C: ResizeLogic::handleEvent(_XEvent*) (resize-logic.cpp:199)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459== by 0x11FFF9DE: RotateScreen::handleEvent(_XEvent*) (rotate.cpp:910)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459== by 0x12D3AE1A: PlaceScreen::handleEvent(_XEvent*) (place.cpp:209)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459== by 0x1488700C: PrivateScaleScreen::handleEvent(_XEvent*) (scale.cpp:1700)
==17459== by 0x4EA933C: CompScreen::handleEvent(_XEvent*) (event.cpp:1155)
==17459==

Related branches

lp:~vanvugt/compiz/fix-1097126
Daniel d'Andrada (community): Approve
PS Jenkins bot (community): Approve (continuous-integration)
Compiz Maintainers: Pending requested
Diff: 121 lines (+34/-15)
4 files modified
plugins/resize/src/logic/include/screen-interface.h (+1/-0)
plugins/resize/src/logic/src/resize-logic.cpp (+23/-15)
plugins/resize/src/screen-impl.h (+5/-0)
plugins/resize/src/window-impl.h (+5/-0)
Daniel van Vugt (vanvugt)
Changed in compiz (Ubuntu):
status: New → Triaged
importance: Undecided → High
Daniel van Vugt (vanvugt)
Changed in compiz:
assignee: nobody → Daniel van Vugt (vanvugt)
status: Triaged → In Progress
Changed in compiz (Ubuntu):
assignee: nobody → Daniel van Vugt (vanvugt)
status: Triaged → In Progress
PS Jenkins bot (ps-jenkins)
Changed in compiz:
status: In Progress → Fix Committed
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Fix committed into lp:compiz at revision 3549, scheduled for release in Compiz 0.9.9.0

Changed in compiz (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package compiz - 1:0.9.9~daily13.01.14-0ubuntu1

---------------
compiz (1:0.9.9~daily13.01.14-0ubuntu1) raring; urgency=low

  [ sampo555 ]
  * compiz crashed with SIGSEGV in DodgeAnim::applyDodgeTransform() (LP:
    #1048840)
  * compiz crashing if window un-/minimize animation is "Random" (LP:
    #1098185)

  [ Daniel van Vugt ]
  * Several leaks in new GLProgram from compileProgram() from
    GLScreen::getProgram() from GLWindowAutoProgram::getProgram() (LP:
    #1097644)

  [ Sam Spilsbury ]
  * Several leaks in ccsIntegratedSettingListAppend() ... from
    ccsGNOMEIntegrationBackendGetIntegratedSetting() from readSetting
    (gsettings.c:375) (LP: #1097661)

  [ MC Return ]
  * Thumbnail Window Previews: Flickering of background/glow and window
    title text (LP: #1098758)

  [ Automatic PS uploader ]
  * Automatic snapshot from revision 3561
 -- Automatic PS uploader <email address hidden> Mon, 14 Jan 2013 04:03:09 +0000

Changed in compiz (Ubuntu):
status: Fix Committed → Fix Released
Daniel van Vugt (vanvugt)
Changed in compiz:
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.