Merge lp://qastaging/~azzar1/unity/fix-1047760 into lp://qastaging/unity

Proposed by Andrea Azzarone
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: no longer in the source branch.
Merged at revision: 2840
Proposed branch: lp://qastaging/~azzar1/unity/fix-1047760
Merge into: lp://qastaging/unity
Diff against target: 66 lines (+16/-7)
2 files modified
launcher/LauncherController.cpp (+13/-6)
launcher/LauncherControllerPrivate.h (+3/-1)
To merge this branch: bzr merge lp://qastaging/~azzar1/unity/fix-1047760
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
PS Jenkins bot continuous-integration Pending
Review via email: mp+129390@code.qastaging.launchpad.net

Commit message

Unregister gdbus object in the dtor. Fix bug 1047760

Description of the change

== Problem ==
compiz crashed with SIGSEGV in unity::launcher::LauncherModel::begin() from unity::launcher::LauncherModel::GetSublist() from unity::launcher::Controller::Impl::OnLauncherAddRequestSpecial()

== Fix ==
unregister gdbus object

== Test ==
not applicable

To post a comment you must log in.
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Looks good, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.