tree 5014633607a77f9f9954207189b7e80f1a845154
parent 9e21dc27a8cdc1409251fcd5e90e23f313100b59
author Matthias Kaehlcke <mka@chromium.org> 1489606428 -0700
committer chrome-bot <chrome-bot@chromium.org> 1489779653 -0700

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>