Merge lp:~vanvugt/compiz-core/fix-940115 into lp:compiz-core

Proposed by Daniel van Vugt on 2012-02-24
Status: Merged
Approved by: Alan Griffiths on 2012-02-24
Approved revision: 3019
Merged at revision: 3020
Proposed branch: lp:~vanvugt/compiz-core/fix-940115
Merge into: lp:compiz-core
Diff against target: 13 lines (+3/-0)
1 file modified
plugins/decor/src/decor.cpp (+3/-0)
To merge this branch: bzr merge lp:~vanvugt/compiz-core/fix-940115
Reviewer Review Type Date Requested Status
Alan Griffiths Approve on 2012-02-24
Sam Spilsbury 2012-02-24 Approve on 2012-02-24
Review via email: mp+94504@code.launchpad.net

Description of the change

Fix memory leak (LP: #940115)

To post a comment you must log in.
Sam Spilsbury (smspillaz) :
review: Approve
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/decor/src/decor.cpp'
2--- plugins/decor/src/decor.cpp 2012-02-09 07:48:57 +0000
3+++ plugins/decor/src/decor.cpp 2012-02-24 08:26:17 +0000
4@@ -2257,6 +2257,9 @@
5 false, XA_WINDOW, &actualType, &actualFmt,
6 &nitems, &nleft, (unsigned char **) &data) == Success)
7 {
8+ if (data)
9+ XFree (data);
10+
11 if (nitems == 1)
12 {
13 isSwitcher = true;

Subscribers

People subscribed via source and target branches