/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 clients.conf

  • 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:
4
4
 
5
5
# How long until a client is disabled and not be allowed to get the
6
6
# data this server holds.
7
 
;timeout = 5m
 
7
;timeout = PT5M
8
8
 
9
9
# How often to run the checker to confirm that a client is still up.
10
10
# Note: a new checker will not be started if an old one is still
11
11
# running.  The server will wait for a checker to complete until the
12
12
# above "timeout" occurs, at which time the client will be disabled,
13
13
# and any running checker killed.
14
 
;interval = 2m
 
14
;interval = PT2M
15
15
 
16
16
# Extended timeout is an added timeout that is given once after a
17
17
# password has been sent sucessfully to a client.  This allows for
18
18
# additional delays caused by file system checks and quota checks.
19
 
;extended_timeout = 15m
 
19
;extended_timeout = PT15M
20
20
 
21
21
# What command to run as "the checker".
22
22
;checker = fping -q -- %%(host)s
25
25
;approved_by_default = True
26
26
 
27
27
# How long to wait for approval.
28
 
;approval_delay = 0s
 
28
;approval_delay = PT0S
29
29
 
30
30
# How long one approval will last.
31
 
;approval_duration = 1s
 
31
;approval_duration = PT1S
32
32
 
33
33
# Whether this client is enabled by default
34
34
;enabled = True
38
38
;# Example client
39
39
;[foo]
40
40
;
 
41
;# TLS public key ID
 
42
;key_id = f33fcbed11ed5e03073f6a55b86ffe92af0e24c045fb6e3b40547b3dc0c030ed
 
43
;
41
44
;# OpenPGP key fingerprint
42
45
;fingerprint =  7788 2722 5BA7 DE53 9C5A  7CFA 59CF F7CD BD9A 5920
43
46
;
68
71
;####
69
72
;# Another example client, named "bar".
70
73
;[bar]
 
74
;# The key ID is not space or case sensitive
 
75
;key_id = F33FCBED11ED5E03073F6A55B86FFE92 AF0E24C045FB6E3B40547B3DC0C030ED
 
76
;
71
77
;# The fingerprint is not space or case sensitive
72
78
;fingerprint = 3e393aeaefb84c7e89e2f547b3a107558fca3a27
73
79
;
74
80
;# If "secret" is not specified, a file can be read for the data.
75
 
;secfile = /etc/mandos/bar-secret.bin
 
81
;secfile = /etc/keys/mandos/bar-secret.bin
76
82
;
77
83
;# An IP address for host is also fine, if the checker accepts it.
78
84
;host = 192.0.2.3
79
85
;
80
86
;# Parameters from the [DEFAULT] section can be overridden per client.
81
 
;interval = 1m
 
87
;interval = PT1M
82
88
;
83
89
;# This client requires manual approval before it receives its secret.
84
90
;approved_by_default = False
85
91
;# Require approval within 30 seconds.
86
 
;approval_delay = 30s
 
92
;approval_delay = PT30S
87
93
;####