start-stop-daemon не отправляет SIGTERM

У меня есть простой демон, который сводится к

#include <unistd.h>
#include <signal.h>
#include <sys/stat.h>

#include <fstream>
#include <iostream>

#include <boost/thread.hpp>

bool running = true;
std::ofstream log_output;

void close_log_output()
{
    log_output.close();
}

void signal_handler(int)
{
    running = false;
}

int detach_service()
{
    if(pid_t pid = fork()) exit(pid < 0);
    umask(0);
    close(STDIN_FILENO);
    close(STDOUT_FILENO);
    close(STDERR_FILENO);
    chdir("/");
    log_output.open("/var/log/mydaemon.log");
    std::cout.rdbuf(log_output.rdbuf());
    std::cerr.rdbuf(log_output.rdbuf());
    std::clog.rdbuf(log_output.rdbuf());
    atexit(&close_log_output);
    signal(SIGTERM, &signal_handler);
    return (setsid() < 0);
}

int main(int argc, char **argv)
{
    if(int err = detach_service()) return err;
    while(running) boost::this_thread::sleep(boost::posix_time::milliseconds(30));
    std::cout << "terminated\n";
    return 0;
}

и скрипт init.d, который представляет собой исходный скелет с отрегулированными $NAME, $DAEMON и $DAEMON_ARGS. Если я уничтожу процесс через его PID, он получит SIGTERM и завершится должным образом, однако, если я попытаюсь остановить службу (которая по умолчанию настроена на отправку TERM первой), команда зависнет, и процесс будет уничтожен, даже не получив SIGTERM.

Что мне нужно сделать/изменить, чтобы процесс завершился правильно (либо на стороне оболочки, либо на стороне С++)

Модифицированный скелет (только NAME и DAEMON_ARGS изменены по сравнению с исходным скелетом Ubuntu init.d):

#! /bin/sh
### BEGIN INIT INFO
# Provides:          skeleton
# Required-Start:    $remote_fs $syslog
# Required-Stop:     $remote_fs $syslog
# Default-Start:     2 3 4 5
# Default-Stop:      0 1 6
# Short-Description: Example initscript
# Description:       This file should be used to construct scripts to be
#                    placed in /etc/init.d.
### END INIT INFO

# Author: Foo Bar <[email protected]>
#
# Please remove the "Author" lines above and replace them
# with your own name if you copy and modify this script.

# Do NOT "set -e"

# PATH should only include /usr/* if it runs after the mountnfs.sh script
PATH=/sbin:/usr/sbin:/bin:/usr/bin
DESC="Description of the service"
NAME=mydaemon
DAEMON=/usr/sbin/$NAME
DAEMON_ARGS=""
PIDFILE=/var/run/$NAME.pid
SCRIPTNAME=/etc/init.d/$NAME

# Exit if the package is not installed
[ -x "$DAEMON" ] || exit 0

# Read configuration variable file if it is present
[ -r /etc/default/$NAME ] && . /etc/default/$NAME

# Load the VERBOSE setting and other rcS variables
. /lib/init/vars.sh

# Define LSB log_* functions.
# Depend on lsb-base (>= 3.2-14) to ensure that this file is present
# and status_of_proc is working.
. /lib/lsb/init-functions

#
# Function that starts the daemon/service
#
do_start()
{
        # Return
        #   0 if daemon has been started
        #   1 if daemon was already running
        #   2 if daemon could not be started
        start-stop-daemon --start --quiet --pidfile $PIDFILE --exec $DAEMON --test > /dev/null \
                || return 1
        start-stop-daemon --start --quiet --pidfile $PIDFILE --exec $DAEMON -- \
                $DAEMON_ARGS \
                || return 2
        # Add code here, if necessary, that waits for the process to be ready
        # to handle requests from services started subsequently which depend
        # on this one.  As a last resort, sleep for some time.
}

#
# Function that stops the daemon/service
#
do_stop()
{
        # Return
        #   0 if daemon has been stopped
        #   1 if daemon was already stopped
        #   2 if daemon could not be stopped
        #   other if a failure occurred
        start-stop-daemon --stop --quiet --retry=TERM/30/KILL/5 --pidfile $PIDFILE --name $NAME
        RETVAL="$?"
        [ "$RETVAL" = 2 ] && return 2
        # Wait for children to finish too if this is a daemon that forks
        # and if the daemon is only ever run from this initscript.
        # If the above conditions are not satisfied then add some other code
        # that waits for the process to drop all resources that could be
        # needed by services started subsequently.  A last resort is to
        # sleep for some time.
        start-stop-daemon --stop --quiet --oknodo --retry=0/30/KILL/5 --exec $DAEMON
        [ "$?" = 2 ] && return 2
        # Many daemons don't delete their pidfiles when they exit.
        rm -f $PIDFILE
        return "$RETVAL"
}

#
# Function that sends a SIGHUP to the daemon/service
#
do_reload() {
        #
        # If the daemon can reload its configuration without
        # restarting (for example, when it is sent a SIGHUP),
        # then implement that here.
        #
        start-stop-daemon --stop --signal 1 --quiet --pidfile $PIDFILE --name $NAME
        return 0
}

case "$1" in
  start)
        [ "$VERBOSE" != no ] && log_daemon_msg "Starting $DESC" "$NAME"
        do_start
        case "$?" in
                0|1) [ "$VERBOSE" != no ] && log_end_msg 0 ;;
                2) [ "$VERBOSE" != no ] && log_end_msg 1 ;;
        esac
        ;;
  stop)
        [ "$VERBOSE" != no ] && log_daemon_msg "Stopping $DESC" "$NAME"
        do_stop
        case "$?" in
                0|1) [ "$VERBOSE" != no ] && log_end_msg 0 ;;
                2) [ "$VERBOSE" != no ] && log_end_msg 1 ;;
        esac
        ;;
  status)
        status_of_proc "$DAEMON" "$NAME" && exit 0 || exit $?
        ;;
  #reload|force-reload)
        #
        # If do_reload() is not implemented then leave this commented out
        # and leave 'force-reload' as an alias for 'restart'.
        #
        #log_daemon_msg "Reloading $DESC" "$NAME"
        #do_reload
        #log_end_msg $?
        #;;
  restart|force-reload)
        #
        # If the "reload" option is implemented then remove the
        # 'force-reload' alias
        #
        log_daemon_msg "Restarting $DESC" "$NAME"
        do_stop
        case "$?" in
          0|1)
                do_start
                case "$?" in
                        0) log_end_msg 0 ;;
                        1) log_end_msg 1 ;; # Old process is still running
                        *) log_end_msg 1 ;; # Failed to start
                esac
                ;;
          *)
                # Failed to stop
                log_end_msg 1
                ;;
        esac
        ;;
  *)
        #echo "Usage: $SCRIPTNAME {start|stop|restart|reload|force-reload}" >&2
        echo "Usage: $SCRIPTNAME {start|stop|status|restart|force-reload}" >&2
        exit 3
        ;;
esac

:

person Joe    schedule 06.03.2014    source источник
comment
Пожалуйста, предоставьте соответствующие части вашего сценария инициализации скелета.   -  person user2719058    schedule 06.03.2014
comment
Готово, хотя я уже сказал, что было изменено в исходном вопросе.   -  person Joe    schedule 06.03.2014
comment
Однако не все скелеты одинаковы. Спасибо за предоставление вашего.   -  person user2719058    schedule 06.03.2014


Ответы (1)


Ваш демон не создает pidfile, как и start-stop-daemon. Я думаю, первая строка

start-stop-daemon --stop --quiet --retry=TERM/30/KILL/5 --pidfile $PIDFILE --name $NAME

ничего не делает, потому что критерий выбора процесса --pidfile никогда не выполняется, но вторая строка

start-stop-daemon --stop --quiet --oknodo --retry=0/30/KILL/5 --exec $DAEMON

убивает демона с помощью SIGKILL, потому что здесь опция --pidfile не включена.

Вы должны исправить свой демон, чтобы создать pidfile.

person user2719058    schedule 06.03.2014
comment
Хорошо, но как мне правильно создать pidfile? добавление --make-pidfile к параметрам start-stop-daemon не работает - person Joe; 06.03.2014
comment
Параметр --make-pidfile для start-stop-daemon — это всего лишь обходной путь для сломанных демонов, и, как говорится на странице руководства, он не помогает при разветвлении демонов. Просто позвольте демону самому создать его, это правильный способ сделать это. - person user2719058; 06.03.2014
comment
Очевидно. Я скорее имел в виду, как это правильно сделать в демоне? Просто fopen, fprintf и fclose? - person Joe; 06.03.2014
comment
Должно быть достаточно хорошо. :) Только обязательно сделайте это после форка. Если вы добры, вы также можете отключить его при выходе. :) - person user2719058; 06.03.2014