powerd

Merge lp:~ricmm/powerd/clear-power-dbus-dead into lp:powerd

  1. clear-power-dbus-dead
  2. Merge into trunk
Proposed by Ricardo Mendoza
Status: Merged
Approved by: Ricardo Salveti
Approved revision: 161
Merged at revision: 161
Proposed branch: lp:~ricmm/powerd/clear-power-dbus-dead
Merge into: lp:powerd
Diff against target: 11 lines (+1/-0)
1 file modified
src/powerd-object.c (+1/-0)
To merge this branch: bzr merge lp:~ricmm/powerd/clear-power-dbus-dead
Reviewer Review Type Date Requested Status
Ricardo Salveti (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+257940@code.launchpad.net

Commit message

Clear sys power requests for clients that have dropped off the bus.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:161
http://jenkins.qa.ubuntu.com/job/powerd-ci/176/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/powerd-vivid-armhf-ci/4
        deb: http://jenkins.qa.ubuntu.com/job/powerd-vivid-armhf-ci/4/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/powerd-ci/176/rebuild

review: Approve (continuous-integration)
Revision history for this message
Ricardo Salveti (rsalveti) wrote :

Looks good, works as expected.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/powerd-object.c'
2--- src/powerd-object.c 2014-09-30 20:52:33 +0000
3+++ src/powerd-object.c 2015-04-30 17:28:37 +0000
4@@ -412,6 +412,7 @@
5 gpointer user_data)
6 {
7 powerd_warn("%s vanished from dbus, clearing associated requests", name);
8+ clear_sys_state_by_owner(name);
9 powerd_client_unregister(name);
10 // This can return false if the hash entry was removed because the
11 // last request was dropped, in other words, this call will fail

Subscribers

People subscribed via source and target branches