diff options
| author | Carlos Santos <casantos@datacom.com.br> | 2018-12-20 20:18:38 -0200 |
|---|---|---|
| committer | Peter Korsgaard <peter@korsgaard.com> | 2018-12-21 14:55:17 +0100 |
| commit | 6568b939293e46718434e284bded04b746fbd80f (patch) | |
| tree | 8085df4c58c19b24f4c02cbdce56809fc11e7a97 | |
| parent | 416d46d72ba2f80398942b5cd29e412f28c43ee7 (diff) | |
| download | buildroot-6568b939293e46718434e284bded04b746fbd80f.tar.gz buildroot-6568b939293e46718434e284bded04b746fbd80f.zip | |
package/shairport-sync: fix init script (stop)
shairport-sync creates its pidfile at /var/run/shairport-sync/, so pass
that path to start-stop-daemon in the stop operation.
Also pass the executable path, allowing start-stop-daemon to check if
the PID matches the shairport-sync process, preventing killing some
other inocent daemon.
Fixes:
https://bugs.busybox.net/show_bug.cgi?id=11566
Reported-by: Bin Zhang <yangtze31@gmail.com>
Signed-off-by: Carlos Santos <casantos@datacom.com.br>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
| -rw-r--r-- | package/shairport-sync/S99shairport-sync | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/package/shairport-sync/S99shairport-sync b/package/shairport-sync/S99shairport-sync index 5443ed138c..80b2fddbc6 100644 --- a/package/shairport-sync/S99shairport-sync +++ b/package/shairport-sync/S99shairport-sync @@ -6,16 +6,18 @@ OPTIONS="-d" case "$1" in start) printf "Starting shairport-sync: " - start-stop-daemon -S -q -p /var/run/shairport-sync.pid --exec /usr/bin/shairport-sync -- $OPTIONS + start-stop-daemon -S -q --exec /usr/bin/shairport-sync -- $OPTIONS [ $? = 0 ] && echo "OK" || echo "FAIL" ;; stop) printf "Stopping shairport-sync: " - start-stop-daemon -K -q -p /var/run/shairport-sync.pid + start-stop-daemon -K -q --exec /usr/bin/shairport-sync \ + -p /var/run/shairport-sync/shairport-sync.pid [ $? = 0 ] && echo "OK" || echo "FAIL" ;; restart) $0 stop + sleep 1 $0 start ;; *) |
