[lxc-devel] [lxc/master] attach: silence stdio permission adjust warnings
brauner on Github
lxc-bot at linuxcontainers.org
Thu Nov 5 19:26:28 UTC 2020
A non-text attachment was scrubbed...
Name: not available
Type: text/x-mailbox
Size: 380 bytes
Desc: not available
URL: <http://lists.linuxcontainers.org/pipermail/lxc-devel/attachments/20201105/cf69ee24/attachment.bin>
-------------- next part --------------
From a2c26befc91ab2a93b97b6135f24c14734a3cc08 Mon Sep 17 00:00:00 2001
From: Christian Brauner <christian.brauner at ubuntu.com>
Date: Thu, 5 Nov 2020 20:25:29 +0100
Subject: [PATCH] attach: silence stdio permission adjust warnings
Closes: #3576.
Signed-off-by: Christian Brauner <christian.brauner at ubuntu.com>
---
src/lxc/attach.c | 2 +-
src/lxc/utils.c | 11 ++++++-----
2 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/src/lxc/attach.c b/src/lxc/attach.c
index acbffa238d..cfddf9b36a 100644
--- a/src/lxc/attach.c
+++ b/src/lxc/attach.c
@@ -883,7 +883,7 @@ static int attach_child_main(struct attach_clone_payload *payload)
/* Make sure that the processes STDIO is correctly owned by the user that we are switching to */
ret = fix_stdio_permissions(new_uid);
if (ret)
- WARN("Failed to adjust stdio permissions");
+ INFO("Failed to adjust stdio permissions");
if (!lxc_switch_uid_gid(new_uid, new_gid))
goto on_error;
diff --git a/src/lxc/utils.c b/src/lxc/utils.c
index baf80b7f5c..fdf437ef35 100644
--- a/src/lxc/utils.c
+++ b/src/lxc/utils.c
@@ -1885,11 +1885,11 @@ int fix_stdio_permissions(uid_t uid)
devnull_fd = open_devnull();
if (devnull_fd < 0)
- return log_warn_errno(-1, errno, "Failed to open \"/dev/null\"");
+ return log_trace_errno(-1, errno, "Failed to open \"/dev/null\"");
ret = fstat(devnull_fd, &st_null);
if (ret)
- return log_warn_errno(-errno, errno, "Failed to stat \"/dev/null\"");
+ return log_trace_errno(-errno, errno, "Failed to stat \"/dev/null\"");
for (int i = 0; i < ARRAY_SIZE(std_fds); i++) {
ret = fstat(std_fds[i], &st);
@@ -1904,14 +1904,15 @@ int fix_stdio_permissions(uid_t uid)
ret = fchown(std_fds[i], uid, st.st_gid);
if (ret) {
- SYSWARN("Failed to chown standard I/O file descriptor %d to uid %d and gid %d",
- std_fds[i], uid, st.st_gid);
+ TRACE("Failed to chown standard I/O file descriptor %d to uid %d and gid %d",
+ std_fds[i], uid, st.st_gid);
fret = -1;
+ continue;
}
ret = fchmod(std_fds[i], 0700);
if (ret) {
- SYSWARN("Failed to chmod standard I/O file descriptor %d", std_fds[i]);
+ TRACE("Failed to chmod standard I/O file descriptor %d", std_fds[i]);
fret = -1;
}
}
More information about the lxc-devel
mailing list