/mandos/release

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/release

« back to all changes in this revision

Viewing changes to init.d-mandos

  • Committer: teddy at recompile
  • Date: 2020-04-05 21:30:59 UTC
  • mto: This revision was merged to the branch mainline in revision 398.
  • Revision ID: teddy@recompile.se-20200405213059-fb2a61ckqynrmatk
Fix file descriptor leak in mandos-client

When the local network has Mandos servers announcing themselves using
real, globally reachable, IPv6 addresses (i.e. not link-local
addresses), but there is no router on the local network providing IPv6
RA (Router Advertisement) packets, the client cannot reach the server
by normal means, since the client only has a link-local IPv6 address,
and has no usable route to reach the server's global IPv6 address.
(This is not a common situation, and usually only happens when the
router itself reboots and runs a Mandos client, since it cannot then
give RA packets to itself.)  The client code has a solution for
this, which consists of adding a temporary local route to reach the
address of the server during communication, and removing this
temporary route afterwards.

This solution with a temporary route works, but has a file descriptor
leak; it leaks one file descriptor for each addition and for each
removal of a route.  If one server requiring an added route is present
on the network, but no servers gives a password, making the client
retry after the default ten seconds, and we furthermore assume a
default 1024 open files limit, the client runs out of file descriptors
after about 90 minutes, after which time the client process will be
useless and fail to retrieve any passwords, necessitating manual
password entry via the keyboard.

Fix this by eliminating the file descriptor leak in the client.

* plugins.d/mandos-client.c (add_delete_local_route): Do
  close(devnull) also in parent process, also if fork() fails, and on
  any failure in child process.

Show diffs side-by-side

added added

removed removed

Lines of Context:
init.d-mandos
 
1
#! /bin/sh
 
2
### BEGIN INIT INFO
 
3
# Provides:          mandos
 
4
# Required-Start:    $remote_fs $syslog avahi-daemon
 
5
# Required-Stop:     $remote_fs $syslog avahi-daemon
 
6
# Default-Start:     2 3 4 5
 
7
# Default-Stop:      0 1 6
 
8
# Short-Description: Mandos server
 
9
# Description:       Server of encrypted passwords to Mandos clients
 
10
### END INIT INFO
 
11
 
 
12
# Author: Teddy Hogeborn <teddy@recompile.se>
 
13
# Author: Björn Påhlsson <belorn@recompile.se>
 
14
 
 
15
# Do NOT "set -e"
 
16
 
 
17
# PATH should only include /usr/* if it runs after the mountnfs.sh script
 
18
PATH=/sbin:/usr/sbin:/bin:/usr/bin
 
19
DESC="Mandos root file system password server"
 
20
NAME=mandos
 
21
DAEMON=/usr/sbin/$NAME
 
22
DAEMON_ARGS=""
 
23
if [ -d /run/. ]; then
 
24
    PIDFILE=/run/$NAME.pid
 
25
else
 
26
    PIDFILE=/var/run/$NAME.pid
 
27
fi
 
28
SCRIPTNAME=/etc/init.d/$NAME
 
29
 
 
30
# Exit if the package is not installed
 
31
[ -x "$DAEMON" ] || exit 0
 
32
 
 
33
# Read configuration variable file if it is present
 
34
[ -r /etc/default/$NAME ] && . /etc/default/$NAME
 
35
 
 
36
if [ -n "$CONFIGDIR" ]; then
 
37
   DAEMON_ARGS="$DAEMON_ARGS --configdir $CONFIGDIR"
 
38
fi
 
39
 
 
40
# Load the VERBOSE setting and other rcS variables
 
41
. /lib/init/vars.sh
 
42
 
 
43
# Define LSB log_* functions.
 
44
# Depend on lsb-base (>= 3.2-14) to ensure that this file is present
 
45
# and status_of_proc is working.
 
46
. /lib/lsb/init-functions
 
47
 
 
48
#
 
49
# Function that starts the daemon/service
 
50
#
 
51
do_start()
 
52
{
 
53
        # Return
 
54
        #   0 if daemon has been started
 
55
        #   1 if daemon was already running
 
56
        #   2 if daemon could not be started
 
57
        start-stop-daemon --start --quiet --pidfile $PIDFILE --exec $DAEMON --test > /dev/null \
 
58
                || return 1
 
59
        start-stop-daemon --start --quiet --pidfile $PIDFILE --exec $DAEMON -- \
 
60
                $DAEMON_ARGS \
 
61
                || return 2
 
62
        # Add code here, if necessary, that waits for the process to be ready
 
63
        # to handle requests from services started subsequently which depend
 
64
        # on this one.  As a last resort, sleep for some time.
 
65
}
 
66
 
 
67
#
 
68
# Function that stops the daemon/service
 
69
#
 
70
do_stop()
 
71
{
 
72
        # Return
 
73
        #   0 if daemon has been stopped
 
74
        #   1 if daemon was already stopped
 
75
        #   2 if daemon could not be stopped
 
76
        #   other if a failure occurred
 
77
        start-stop-daemon --stop --quiet --retry=TERM/30/KILL/5 --pidfile $PIDFILE --name $NAME
 
78
        RETVAL="$?"
 
79
        [ "$RETVAL" = 2 ] && return 2
 
80
        # Wait for children to finish too if this is a daemon that forks
 
81
        # and if the daemon is only ever run from this initscript.
 
82
        # If the above conditions are not satisfied then add some other code
 
83
        # that waits for the process to drop all resources that could be
 
84
        # needed by services started subsequently.  A last resort is to
 
85
        # sleep for some time.
 
86
        start-stop-daemon --stop --quiet --oknodo --retry=0/30/KILL/5 --exec $DAEMON
 
87
        [ "$?" = 2 ] && return 2
 
88
        # Many daemons don't delete their pidfiles when they exit.
 
89
        rm -f $PIDFILE
 
90
        return "$RETVAL"
 
91
}
 
92
 
 
93
#
 
94
# Function that sends a SIGHUP to the daemon/service
 
95
#
 
96
do_reload() {
 
97
        #
 
98
        # If the daemon can reload its configuration without
 
99
        # restarting (for example, when it is sent a SIGHUP),
 
100
        # then implement that here.
 
101
        #
 
102
        start-stop-daemon --stop --signal 1 --quiet --pidfile $PIDFILE --name $NAME
 
103
        return 0
 
104
}
 
105
 
 
106
case "$1" in
 
107
  start)
 
108
        [ "$VERBOSE" != no ] && log_daemon_msg "Starting $DESC" "$NAME"
 
109
        do_start
 
110
        case "$?" in
 
111
                0|1) [ "$VERBOSE" != no ] && log_end_msg 0 ;;
 
112
                2) [ "$VERBOSE" != no ] && log_end_msg 1 ;;
 
113
        esac
 
114
        ;;
 
115
  stop)
 
116
        [ "$VERBOSE" != no ] && log_daemon_msg "Stopping $DESC" "$NAME"
 
117
        do_stop
 
118
        case "$?" in
 
119
                0|1) [ "$VERBOSE" != no ] && log_end_msg 0 ;;
 
120
                2) [ "$VERBOSE" != no ] && log_end_msg 1 ;;
 
121
        esac
 
122
        ;;
 
123
  status)
 
124
        status_of_proc "$DAEMON" "$NAME" -p "$PIDFILE" && exit 0 || exit $?
 
125
        ;;
 
126
  #reload|force-reload)
 
127
        #
 
128
        # If do_reload() is not implemented then leave this commented out
 
129
        # and leave 'force-reload' as an alias for 'restart'.
 
130
        #
 
131
        #log_daemon_msg "Reloading $DESC" "$NAME"
 
132
        #do_reload
 
133
        #log_end_msg $?
 
134
        #;;
 
135
  restart|force-reload)
 
136
        #
 
137
        # If the "reload" option is implemented then remove the
 
138
        # 'force-reload' alias
 
139
        #
 
140
        log_daemon_msg "Restarting $DESC" "$NAME"
 
141
        do_stop
 
142
        case "$?" in
 
143
          0|1)
 
144
                do_start
 
145
                case "$?" in
 
146
                        0) log_end_msg 0 ;;
 
147
                        1) log_end_msg 1 ;; # Old process is still running
 
148
                        *) log_end_msg 1 ;; # Failed to start
 
149
                esac
 
150
                ;;
 
151
          *)
 
152
                # Failed to stop
 
153
                log_end_msg 1
 
154
                ;;
 
155
        esac
 
156
        ;;
 
157
  *)
 
158
        #echo "Usage: $SCRIPTNAME {start|stop|restart|reload|force-reload}" >&2
 
159
        echo "Usage: $SCRIPTNAME {start|stop|status|restart|force-reload}" >&2
 
160
        exit 3
 
161
        ;;
 
162
esac
 
163
 
 
164
: