[lxc-devel] [lxc/master] commands: simplify lxc_cmd_fd_cleanup()
brauner on Github
lxc-bot at linuxcontainers.org
Wed Mar 11 02:35:35 UTC 2020
A non-text attachment was scrubbed...
Name: not available
Type: text/x-mailbox
Size: 364 bytes
Desc: not available
URL: <http://lists.linuxcontainers.org/pipermail/lxc-devel/attachments/20200310/69e63806/attachment.bin>
-------------- next part --------------
From ab92468c62fe86061cf39aabf7e0ea8c7e983271 Mon Sep 17 00:00:00 2001
From: Christian Brauner <christian.brauner at ubuntu.com>
Date: Wed, 11 Mar 2020 03:33:16 +0100
Subject: [PATCH] commands: simplify lxc_cmd_fd_cleanup()
Signed-off-by: Christian Brauner <christian.brauner at ubuntu.com>
---
src/lxc/commands.c | 18 +++++++++++-------
1 file changed, 11 insertions(+), 7 deletions(-)
diff --git a/src/lxc/commands.c b/src/lxc/commands.c
index 247807c868..1feae25aab 100644
--- a/src/lxc/commands.c
+++ b/src/lxc/commands.c
@@ -1322,29 +1322,33 @@ static int lxc_cmd_process(int fd, struct lxc_cmd_req *req,
static void lxc_cmd_fd_cleanup(int fd, struct lxc_handler *handler,
struct lxc_epoll_descr *descr, const lxc_cmd_t cmd)
{
- struct lxc_list *cur, *next;
-
lxc_terminal_free(handler->conf, fd);
lxc_mainloop_del_handler(descr, fd);
if (cmd == LXC_CMD_ADD_STATE_CLIENT) {
+ struct lxc_list *cur, *next;
+
lxc_list_for_each_safe(cur, &handler->conf->state_clients, next) {
struct lxc_state_client *client = cur->elem;
if (client->clientfd != fd)
continue;
- /* kick client from list */
+ /*
+ * Only kick client from list so it can't be found
+ * anymore. The actual close happens, as for all other
+ * file descriptors, below.
+ */
lxc_list_del(cur);
- close(client->clientfd);
free(cur->elem);
free(cur);
+
/*
* No need to walk the whole list. If we found the state
* client fd there can't be a second one.
*/
- TRACE("Closed state client fd %d\n", fd);
- return;
+ TRACE("Found state client fd %d in state client list", fd);
+ break;
}
/*
@@ -1353,7 +1357,7 @@ static void lxc_cmd_fd_cleanup(int fd, struct lxc_handler *handler,
* was already reached by the time we were ready to add it. So
* fallthrough and clean it up.
*/
- TRACE("Closing state client fd %d not present in state client list\n", fd);
+ TRACE("Closing state client fd %d", fd);
}
close(fd);
More information about the lxc-devel
mailing list