attach: don't shutdown ipc socket in child

shutdown() affects sockets even across forked processes. The attached child process doesn't have any interest in using the IPC socket, so just close it in the child process and let the intermediate process handle shutting it down. This fixes a bug seen with lxc exec in crbug.com/884244 Signed-off-by: Stephen Barber <smbarber@chromium.org>

attach: don't shutdown ipc socket in child
4523e218 · Stephen Barber · Christian Brauner · 0c0f0059 · 4523e218
Unverified Commit 4523e218 authored Sep 17, 2018 by Stephen Barber Committed by Christian Brauner Dec 10, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 0 additions and 2 deletions

attach.c src/lxc/attach.c +0 -2

No files found.
--- a/src/lxc/attach.c
+++ b/src/lxc/attach.c
@@ -706,7 +706,6 @@ struct attach_clone_payload {
 static void lxc_put_attach_clone_payload(struct attach_clone_payload *p)
 {
 	if (p->ipc_socket >= 0) {
-		shutdown(p->ipc_socket, SHUT_RDWR);
 		close(p->ipc_socket);
 		p->ipc_socket = -EBADF;
 	}
@@ -848,7 +847,6 @@ static int attach_child_main(struct attach_clone_payload *payload)
 			goto on_error;
 		TRACE("Loaded seccomp profile");
 	}
-	shutdown(payload->ipc_socket, SHUT_RDWR);
 	close(payload->ipc_socket);
 	payload->ipc_socket = -EBADF;
 	lxc_proc_put_context_info(init_ctx);