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

Bug #1097126 reported by Daniel van Vugt on 2013-01-08
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Compiz
High
Daniel van Vugt
0.9.8
High
Unassigned
compiz (Ubuntu)
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

Changed in compiz (Ubuntu):
status: New → Triaged
importance: Undecided → High
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
Changed in compiz:
status: In Progress → Fix Committed
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
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
Changed in compiz:
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