diff options
| author | Tom Gundersen <teg@jklm.no> | 2014-06-02 21:50:50 +0200 |
|---|---|---|
| committer | Tom Gundersen <teg@jklm.no> | 2014-06-03 00:40:23 +0200 |
| commit | bddfc8afd329ac68a23f66a3512d4e249af25191 (patch) | |
| tree | ebd7a09270848e57e4664cd664e17e81e8cf9135 /src/network/networkd.c | |
| parent | a613382bbf4357ce13f17c988713b80172e091fb (diff) | |
networkd: drop CAP_SYS_MODULE
Rely on modules being built-in or autoloaded on-demand.
As networkd is a network facing service, we want to limits its capabilities,
as much as possible. Also, we may not have CAP_SYS_MODULE in a container,
and we want networkd to work the same there.
Module autoloading does not always work, but should be fixed by the kernel
patch f98f89a0104454f35a: 'net: tunnels - enable module autoloading', which
is currently in net-next and which people may consider backporting if they
want tunneling support without compiling in the modules.
Early adopters may also use a module-load.d snippet and order
systemd-modules-load.service before networkd to force the module
loading of tunneling modules.
This sholud fix the various build issues people have reported.
Diffstat (limited to 'src/network/networkd.c')
| -rw-r--r-- | src/network/networkd.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/network/networkd.c b/src/network/networkd.c index cd7dd3ca0..d8f31a490 100644 --- a/src/network/networkd.c +++ b/src/network/networkd.c @@ -71,8 +71,7 @@ int main(int argc, char *argv[]) { (1ULL << CAP_NET_ADMIN) | (1ULL << CAP_NET_BIND_SERVICE) | (1ULL << CAP_NET_BROADCAST) | - (1ULL << CAP_NET_RAW) | - (1ULL << CAP_SYS_MODULE)); + (1ULL << CAP_NET_RAW)); if (r < 0) goto out; |