[qml-phone-shell] ubuntu shape memory leak

Bug #1154817 reported by kevin gunn
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu UI Toolkit
Invalid
High
Loïc Molinari

Bug Description

1) ShapeNode (lp:ubuntu-ui-toolkit modules/Ubuntu/Components/plugin/shapeitem.cpp) does not seem to be deleted ever even when the ShapeItem (UbuntuShape) is destroyed.
line 285:
node = new ShapeNode(this);

but no deletion to be seen.

ShapeNode inherits from QObject so I suggest we set ShapeItem as the parent of the ShapeNode which will make the ShapeNode getting deleted when ShapeItem is.

2) In ShapeItem::setImage ShapeItem does not take ownership of image which means that image might not get deleted.

Tags: shape
kevin gunn (kgunn72)
Changed in manhattan:
importance: Undecided → High
assignee: nobody → Michał Sawicz (saviq)
milestone: none → update-2
Michał Sawicz (saviq)
description: updated
information type: Proprietary → Public
Changed in ubuntu-ui-toolkit:
importance: Undecided → High
assignee: nobody → Ubuntu SDK team (ubuntu-sdk-team)
information type: Public → Private
Revision history for this message
Florian Boucault (fboucault) wrote :

I think these findings (that I originally found :)) might not be relevant. Let Loic check that out.

Changed in ubuntu-ui-toolkit:
assignee: Ubuntu SDK team (ubuntu-sdk-team) → Loïc Molinari (loic.molinari)
Bill Filler (bfiller)
Changed in manhattan:
status: New → Triaged
information type: Private → Public
tags: added: shape
no longer affects: manhattan
Revision history for this message
Tim Peeters (tpeeters) wrote :

Is still valid?

Changed in ubuntu-ui-toolkit:
status: New → Incomplete
Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit:
status: Incomplete → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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