Merge lp:~ken-vandine/powerd/lp1186706 into lp:powerd

Proposed by Ken VanDine
Status: Merged
Approved by: Ricardo Mendoza
Approved revision: 30
Merged at revision: 30
Proposed branch: lp:~ken-vandine/powerd/lp1186706
Merge into: lp:powerd
Diff against target: 16 lines (+5/-3)
1 file modified
src/powerd-sensors.cpp (+5/-3)
To merge this branch: bzr merge lp:~ken-vandine/powerd/lp1186706
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Ubuntu Phablet Team Pending
Review via email: mp+167116@code.launchpad.net

Commit message

Don't listen for proximity events if prox_sensor is NULL (LP: #1186706)

Description of the change

Don't listen for proximity events if prox_sensor is NULL (LP: #1186706)

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

PASSED: Continuous integration, rev:30
http://jenkins.qa.ubuntu.com/job/powerd-ci/46/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/powerd-raring-armhf-ci/46

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

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/powerd-sensors.cpp'
2--- src/powerd-sensors.cpp 2013-05-29 21:06:00 +0000
3+++ src/powerd-sensors.cpp 2013-06-03 18:27:24 +0000
4@@ -57,7 +57,9 @@
5 {
6 prox_sensor = ua_sensors_proximity_new();
7
8- ua_sensors_proximity_set_reading_cb(prox_sensor,
9- on_new_proximity_event,
10- NULL);
11+ if (prox_sensor != NULL) {
12+ ua_sensors_proximity_set_reading_cb(prox_sensor,
13+ on_new_proximity_event,
14+ NULL);
15+ }
16 }

Subscribers

People subscribed via source and target branches