[lxc-devel] [lxc/master] tree-wide: s/steal_{fd, ptr}/move_{fd, ptr}/g

brauner on Github lxc-bot at linuxcontainers.org
Mon Feb 18 21:22:56 UTC 2019


A non-text attachment was scrubbed...
Name: not available
Type: text/x-mailbox
Size: 422 bytes
Desc: not available
URL: <http://lists.linuxcontainers.org/pipermail/lxc-devel/attachments/20190218/fa33fa92/attachment.bin>
-------------- next part --------------
From e4edf5d7bcee31ee159d6943c741d7ef06814b66 Mon Sep 17 00:00:00 2001
From: Christian Brauner <christian.brauner at ubuntu.com>
Date: Mon, 18 Feb 2019 22:12:29 +0100
Subject: [PATCH 1/2] tree-wide: s/steal_ptr/move_ptr/g

Suggested-by: Wolfgang Bumiller <w.bumiller at proxmox.com>
Signed-off-by: Christian Brauner <christian.brauner at ubuntu.com>
---
 src/lxc/cgroups/cgfsng.c |  2 +-
 src/lxc/macro.h          | 10 +++++-----
 src/lxc/rexec.c          |  2 +-
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/lxc/cgroups/cgfsng.c b/src/lxc/cgroups/cgfsng.c
index 971c6a523..76d4dfe6d 100644
--- a/src/lxc/cgroups/cgfsng.c
+++ b/src/lxc/cgroups/cgfsng.c
@@ -1451,7 +1451,7 @@ __cgfsng_ops static inline bool cgfsng_payload_create(struct cgroup_ops *ops,
 		return false;
 
 	INFO("The container process uses \"%s\" as cgroup", container_cgroup);
-	ops->container_cgroup = steal_ptr(container_cgroup);
+	ops->container_cgroup = move_ptr(container_cgroup);
 	return true;
 }
 
diff --git a/src/lxc/macro.h b/src/lxc/macro.h
index 61e626f2e..3817ab526 100644
--- a/src/lxc/macro.h
+++ b/src/lxc/macro.h
@@ -393,11 +393,11 @@ enum {
 /* Maximum number of bytes sendfile() is able to send in one go. */
 #define LXC_SENDFILE_MAX 0x7ffff000
 
-#define steal_ptr(ptr)                     \
-	({                                 \
-		typeof(ptr) _ptr_ = (ptr); \
-		(ptr) = NULL;              \
-		_ptr_;                     \
+#define move_ptr(ptr)                                 \
+	({                                            \
+		typeof(ptr) __internal_ptr__ = (ptr); \
+		(ptr) = NULL;                         \
+		__internal_ptr__;                     \
 	})
 
 #define steal_fd(fd)             \
diff --git a/src/lxc/rexec.c b/src/lxc/rexec.c
index 2258baae0..58f3ff486 100644
--- a/src/lxc/rexec.c
+++ b/src/lxc/rexec.c
@@ -77,7 +77,7 @@ static int parse_argv(char ***argv)
 	if (ret <= 0)
 		return -1;
 
-	steal_ptr(cmdline);
+	move_ptr(cmdline);
 	return 0;
 }
 

From 240fecd0e11feb3b72a2693c13917eb6c0248265 Mon Sep 17 00:00:00 2001
From: Christian Brauner <christian.brauner at ubuntu.com>
Date: Mon, 18 Feb 2019 22:13:42 +0100
Subject: [PATCH 2/2] tree-wide: s/steal_fd/move_fd/g

Suggested-by: Wolfgang Bumiller <w.bumiller at proxmox.com>
Signed-off-by: Christian Brauner <christian.brauner at ubuntu.com>
---
 src/lxc/commands.c   | 14 +++++++-------
 src/lxc/file_utils.c |  4 ++--
 src/lxc/macro.h      | 10 +++++-----
 3 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/src/lxc/commands.c b/src/lxc/commands.c
index f4304e640..ad2c578e1 100644
--- a/src/lxc/commands.c
+++ b/src/lxc/commands.c
@@ -245,7 +245,7 @@ static int lxc_cmd_send(const char *name, struct lxc_cmd_rr *cmd,
 		return -1;
 
 	if (cmd->req.datalen <= 0)
-		return steal_fd(client_fd);
+		return move_fd(client_fd);
 
 	errno = EMSGSIZE;
 	ret = lxc_send_nointr(client_fd, (void *)cmd->req.data,
@@ -253,7 +253,7 @@ static int lxc_cmd_send(const char *name, struct lxc_cmd_rr *cmd,
 	if (ret < 0 || ret != (ssize_t)cmd->req.datalen)
 		return -1;
 
-	return steal_fd(client_fd);
+	return move_fd(client_fd);
 }
 
 /*
@@ -304,7 +304,7 @@ static int lxc_cmd(const char *name, struct lxc_cmd_rr *cmd, int *stopped,
 		*stopped = 1;
 
 	if (stay_connected && ret > 0)
-		cmd->rsp.ret = steal_fd(client_fd);
+		cmd->rsp.ret = move_fd(client_fd);
 
 	return ret;
 }
@@ -866,7 +866,7 @@ int lxc_cmd_add_state_client(const char *name, const char *lxcpath,
 		return state;
 	}
 
-	*state_client_fd = steal_fd(clientfd);
+	*state_client_fd = move_fd(clientfd);
 	TRACE("Added state client %d to state client list", *state_client_fd);
 	return MAX_STATE;
 }
@@ -1184,7 +1184,7 @@ static int lxc_cmd_accept(int fd, uint32_t events, void *data,
 		return ret;
 	}
 
-	steal_fd(connection);
+	move_fd(connection);
 	return ret;
 }
 
@@ -1214,7 +1214,7 @@ int lxc_cmd_init(const char *name, const char *lxcpath, const char *suffix)
 	}
 
 	TRACE("Created abstract unix socket \"%s\"", &path[1]);
-	return steal_fd(fd);
+	return move_fd(fd);
 }
 
 int lxc_cmd_mainloop_add(const char *name, struct lxc_epoll_descr *descr,
@@ -1229,6 +1229,6 @@ int lxc_cmd_mainloop_add(const char *name, struct lxc_epoll_descr *descr,
 		return ret;
 	}
 
-	steal_fd(fd);
+	move_fd(fd);
 	return ret;
 }
diff --git a/src/lxc/file_utils.c b/src/lxc/file_utils.c
index f7581eeeb..f80aa6421 100644
--- a/src/lxc/file_utils.c
+++ b/src/lxc/file_utils.c
@@ -234,13 +234,13 @@ int lxc_make_tmpfile(char *template, bool rm)
 		return -1;
 
 	if (!rm)
-		return steal_fd(fd);
+		return move_fd(fd);
 
 	ret = unlink(template);
 	if (ret < 0)
 		return -1;
 
-	return steal_fd(fd);
+	return move_fd(fd);
 }
 
 bool is_fs_type(const struct statfs *fs, fs_type_magic magic_val)
diff --git a/src/lxc/macro.h b/src/lxc/macro.h
index 3817ab526..6e07516f2 100644
--- a/src/lxc/macro.h
+++ b/src/lxc/macro.h
@@ -400,11 +400,11 @@ enum {
 		__internal_ptr__;                     \
 	})
 
-#define steal_fd(fd)             \
-	({                       \
-		int _fd_ = (fd); \
-		(fd) = -EBADF;   \
-		_fd_;            \
+#define move_fd(fd)                         \
+	({                                  \
+		int __internal_fd__ = (fd); \
+		(fd) = -EBADF;              \
+		__internal_fd__;            \
 	})
 
 #endif /* __LXC_MACRO_H */


More information about the lxc-devel mailing list