From a41fe3a29372f8e6c4e7733bf85940a023811301 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Tue, 2 Aug 2011 04:49:37 +0200 Subject: [PATCH] nspawn: add new --no-net switch to turn off networking in the container --- Makefile.am | 3 ++- man/systemd-nspawn.xml | 11 +++++++++++ src/nspawn.c | 26 ++++++++++++++++++++------ 3 files changed, 33 insertions(+), 7 deletions(-) diff --git a/Makefile.am b/Makefile.am index 7971b6b77..c446527e1 100644 --- a/Makefile.am +++ b/Makefile.am @@ -1341,7 +1341,8 @@ systemd_cgls_LDADD = \ systemd_nspawn_SOURCES = \ src/nspawn.c \ - src/cgroup-util.c + src/cgroup-util.c \ + src/loopback-setup.c systemd_nspawn_CFLAGS = \ $(AM_CFLAGS) diff --git a/man/systemd-nspawn.xml b/man/systemd-nspawn.xml index 03c39fc3d..8b37519ad 100644 --- a/man/systemd-nspawn.xml +++ b/man/systemd-nspawn.xml @@ -123,6 +123,7 @@ + Prints a short help text and exits. @@ -152,6 +153,16 @@ + + + + Turn off networking in + the container. This makes all network + interfaces unavailable in the + container, with the exception of the + loopback device. + + diff --git a/src/nspawn.c b/src/nspawn.c index f4d63ea26..c97649d38 100644 --- a/src/nspawn.c +++ b/src/nspawn.c @@ -44,9 +44,11 @@ #include "cgroup-util.h" #include "sd-daemon.h" #include "strv.h" +#include "loopback-setup.h" static char *arg_directory = NULL; static char *arg_user = NULL; +static bool arg_no_net = false; static int help(void) { @@ -54,7 +56,8 @@ static int help(void) { "Spawn a minimal namespace container for debugging, testing and building.\n\n" " -h --help Show this help\n" " -D --directory=NAME Root directory for the container\n" - " -u --user=USER Run the command under specified user or uid\n", + " -u --user=USER Run the command under specified user or uid\n" + " --no-net Disable network in container\n", program_invocation_short_name); return 0; @@ -62,11 +65,16 @@ static int help(void) { static int parse_argv(int argc, char *argv[]) { + enum { + ARG_NO_NET = 0x100 + }; + static const struct option options[] = { - { "help", no_argument, NULL, 'h' }, - { "directory", required_argument, NULL, 'D' }, - { "user", optional_argument, NULL, 'u' }, - { NULL, 0, NULL, 0 } + { "help", no_argument, NULL, 'h' }, + { "directory", required_argument, NULL, 'D' }, + { "user", required_argument, NULL, 'u' }, + { "no-net", no_argument, NULL, ARG_NO_NET }, + { NULL, 0, NULL, 0 } }; int c; @@ -100,6 +108,10 @@ static int parse_argv(int argc, char *argv[]) { break; + case ARG_NO_NET: + arg_no_net = true; + break; + case '?': return -EINVAL; @@ -698,7 +710,7 @@ int main(int argc, char *argv[]) { sigset_add_many(&mask, SIGCHLD, SIGWINCH, SIGTERM, SIGINT, -1); assert_se(sigprocmask(SIG_BLOCK, &mask, NULL) == 0); - if ((pid = syscall(__NR_clone, SIGCHLD|CLONE_NEWIPC|CLONE_NEWNS|CLONE_NEWPID|CLONE_NEWUTS, NULL)) < 0) { + if ((pid = syscall(__NR_clone, SIGCHLD|CLONE_NEWIPC|CLONE_NEWNS|CLONE_NEWPID|CLONE_NEWUTS|(arg_no_net ? CLONE_NEWNET : 0), NULL)) < 0) { log_error("clone() failed: %m"); goto finish; } @@ -777,6 +789,8 @@ int main(int argc, char *argv[]) { umask(0022); + loopback_setup(); + if (drop_capabilities() < 0) goto child_fail; -- 2.30.2