Merge lp:~ted/ubuntu-app-launch/stop-proxy-unref into lp:ubuntu-app-launch/13.10

Proposed by Ted Gould
Status: Merged
Approved by: Ted Gould
Approved revision: 77
Merged at revision: 80
Proposed branch: lp:~ted/ubuntu-app-launch/stop-proxy-unref
Merge into: lp:ubuntu-app-launch/13.10
Diff against target: 11 lines (+1/-0)
1 file modified
libupstart-app-launch/upstart-app-launch.c (+1/-0)
To merge this branch: bzr merge lp:~ted/ubuntu-app-launch/stop-proxy-unref
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Steve Langasek Approve
Indicator Applet Developers Pending
Review via email: mp+190479@code.launchpad.net

Commit message

Unref the proxy after stopping an application

Description of the change

#ubuntu-unity where bugs go to die!

To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) wrote :

This looks correct to me.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'libupstart-app-launch/upstart-app-launch.c'
2--- libupstart-app-launch/upstart-app-launch.c 2013-09-26 21:48:27 +0000
3+++ libupstart-app-launch/upstart-app-launch.c 2013-10-10 20:31:02 +0000
4@@ -208,6 +208,7 @@
5 g_free(appiddash);
6
7 g_array_free(apps, TRUE);
8+ nih_unref(proxy, NULL);
9
10 return found;
11 }

Subscribers

People subscribed via source and target branches