summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2014-08-21 17:35:19 +0200
committerLennart Poettering <lennart@poettering.net>2014-08-21 17:35:19 +0200
commitf461c8073dee9cd10bfae5ae3586e785ec8a5d07 (patch)
treee3b9ba72c39139cfa9e68fc404d185928da20cb5
parent430e21c2f7e77d600257ead56419f511e48e854a (diff)
execute: explain in a comment, why close_all_fds() is invoked the second time differently
-rw-r--r--src/core/execute.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/core/execute.c b/src/core/execute.c
index 2544a2470c..b5b22472d5 100644
--- a/src/core/execute.c
+++ b/src/core/execute.c
@@ -1635,7 +1635,9 @@ int exec_spawn(ExecCommand *command,
}
/* We repeat the fd closing here, to make sure that
- * nothing is leaked from the PAM modules */
+ * nothing is leaked from the PAM modules. Note that
+ * we are more aggressive this time since socket_fd
+ * and the netns fds we don#t need anymore. */
err = close_all_fds(fds, n_fds);
if (err >= 0)
err = shift_fds(fds, n_fds);