powerd

powerd can run multiple instances at once causing confusion

Bug #1195800 reported by Matt Fischer
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
powerd
Fix Released
Medium
Matt Fischer
powerd (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

powerd can run multiple instances at once. This can cause problems in the system as we're not sure which will get the dbus messages (probably the first one) and who will be talking to the sensor (probably both). To solve this we should fail to start powerd if it is already running.

Related branches

lp:~mfisch/powerd/lp1195800
PS Jenkins bot: Approve (continuous-integration)
Seth Forshee (community): Approve
Diff: 126 lines (+17/-35)
4 files modified
debian/upstart (+6/-25)
src/powerd-internal.h (+1/-1)
src/powerd-object.c (+4/-3)
src/powerd.cpp (+6/-6)
lp:ubuntu/saucy-proposed/powerd
Matt Fischer (mfisch)
Changed in powerd:
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Matthew Fischer (mfisch)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:powerd at revision 63, scheduled for release in powerd, milestone ubuntu-13.05

Changed in powerd:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package powerd - 0.13+13.10.20130711-0ubuntu1

---------------
powerd (0.13+13.10.20130711-0ubuntu1) saucy; urgency=low

  [ Seth Forshee ]
  * Enable -Wall for all files and fix resulting warnings.
  * Type check powerd_log() arguments against the format string.

  [ Matthew Fischer ]
  * Cleanup upstart job by removing unneeded environment variables and
    have the job respawn if powerd dies unexpectedly. Simplify the
    sigterm handler, which also removes the duplicate calls to unown
    name which were causing warnings. Finally, fix LP: #1195800 by
    simply exiting if we lose or never get our dbus name. . (LP:
    #1195800)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 65
 -- Ubuntu daily release <email address hidden> Thu, 11 Jul 2013 02:01:51 +0000

Changed in powerd (Ubuntu):
status: New → Fix Released
Matt Fischer (mfisch)
Changed in powerd:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.