jabra_vold: Daemonize properly
jabra_vold currently performs the daemonization 'manually' and misses
at least one step. It doesn't close the parents file descriptors, in
consequence the daemon process isn't fully detached from its parent.
As a result the udev handler that starts the daemon and is supposed to
be short-lived runs until the daemon exits. This hasn't caused evident
problems with udev-208, however it prevents other udev handlers from
running with udev-225. One of the blocked handlers is in charge of
changing the ownership and permissions of /dev/uinput, which is needed
by jabra_vold to be able open the device node.
Replace the manual daemonization with a call to daemon().
BUG=chromium:700006
TEST=build for panther
boot with Jabra speaker connected
grep jabra_vold /var/log/messages | tail -n 1
=> <ts> NOTICE jabra_vold[xxx]: ALSA dev hw:X (Jabra SPEAK 410 USB)
boot w/o Jabra speaker connected
connect speaker
wait 2s
grep jabra_vold /var/log/messages | tail -n 1
=> <ts> NOTICE jabra_vold[xxx]: ALSA dev hw:X (Jabra SPEAK 410 USB)
Change-Id: I0079a97d6abe520ed9bbaf88387efa2e65bb3b27
Reviewed-on: https://chromium-review.googlesource.com/455327
Commit-Ready: Matthias Kaehlcke <mka@chromium.org>
Tested-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Matthias Kaehlcke <mka@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
1 file changed
