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

Proposed by Daniel van Vugt
Status: Merged
Approved by: Alan Griffiths
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
Sam Spilsbury Approve
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.
Revision history for this message
Sam Spilsbury (smspillaz) :
review: Approve
Revision history for this message
Alan Griffiths (alan-griffiths) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/decor/src/decor.cpp'
--- plugins/decor/src/decor.cpp 2012-02-09 07:48:57 +0000
+++ plugins/decor/src/decor.cpp 2012-02-24 08:26:17 +0000
@@ -2257,6 +2257,9 @@
2257 false, XA_WINDOW, &actualType, &actualFmt,2257 false, XA_WINDOW, &actualType, &actualFmt,
2258 &nitems, &nleft, (unsigned char **) &data) == Success)2258 &nitems, &nleft, (unsigned char **) &data) == Success)
2259 {2259 {
2260 if (data)
2261 XFree (data);
2262
2260 if (nitems == 1)2263 if (nitems == 1)
2261 {2264 {
2262 isSwitcher = true;2265 isSwitcher = true;

Subscribers

People subscribed via source and target branches