[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v9 1/7] qemu-char: Add MSG_CMSG_CLOEXEC flag to recv
From: |
Corey Bryant |
Subject: |
[Qemu-devel] [PATCH v9 1/7] qemu-char: Add MSG_CMSG_CLOEXEC flag to recvmsg |
Date: |
Sat, 11 Aug 2012 09:14:32 -0400 |
Set the close-on-exec flag for the file descriptor received
via SCM_RIGHTS.
Signed-off-by: Corey Bryant <address@hidden>
---
v4
-This patch is new in v4 (address@hidden)
v5
-Fallback to FD_CLOEXEC if MSG_CMSG_CLOEXEC is not available
(address@hidden, address@hidden)
v6
-Set cloexec on correct fd (address@hidden)
v7-v9
-No changes
qemu-char.c | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/qemu-char.c b/qemu-char.c
index c2aaaee..ab4a928 100644
--- a/qemu-char.c
+++ b/qemu-char.c
@@ -2238,6 +2238,9 @@ static void unix_process_msgfd(CharDriverState *chr,
struct msghdr *msg)
if (fd < 0)
continue;
+#ifndef MSG_CMSG_CLOEXEC
+ qemu_set_cloexec(fd);
+#endif
if (s->msgfd != -1)
close(s->msgfd);
s->msgfd = fd;
@@ -2253,6 +2256,7 @@ static ssize_t tcp_chr_recv(CharDriverState *chr, char
*buf, size_t len)
struct cmsghdr cmsg;
char control[CMSG_SPACE(sizeof(int))];
} msg_control;
+ int flags = 0;
ssize_t ret;
iov[0].iov_base = buf;
@@ -2263,9 +2267,13 @@ static ssize_t tcp_chr_recv(CharDriverState *chr, char
*buf, size_t len)
msg.msg_control = &msg_control;
msg.msg_controllen = sizeof(msg_control);
- ret = recvmsg(s->fd, &msg, 0);
- if (ret > 0 && s->is_unix)
+#ifdef MSG_CMSG_CLOEXEC
+ flags |= MSG_CMSG_CLOEXEC;
+#endif
+ ret = recvmsg(s->fd, &msg, flags);
+ if (ret > 0 && s->is_unix) {
unix_process_msgfd(chr, &msg);
+ }
return ret;
}
--
1.7.10.4
- Re: [Qemu-devel] [PATCH v9 6/7] block: Enable qemu_open/close to work with fd sets, (continued)
- Re: [Qemu-devel] [PATCH v9 6/7] block: Enable qemu_open/close to work with fd sets, Eric Blake, 2012/08/11
- Re: [Qemu-devel] [PATCH v9 6/7] block: Enable qemu_open/close to work with fd sets, Corey Bryant, 2012/08/13
- Re: [Qemu-devel] [PATCH v9 6/7] block: Enable qemu_open/close to work with fd sets, Eric Blake, 2012/08/13
- Re: [Qemu-devel] [PATCH v9 6/7] block: Enable qemu_open/close to work with fd sets, Corey Bryant, 2012/08/13
- Re: [Qemu-devel] [PATCH v9 6/7] block: Enable qemu_open/close to work with fd sets, Eric Blake, 2012/08/13
- Re: [Qemu-devel] [PATCH v9 6/7] block: Enable qemu_open/close to work with fd sets, Corey Bryant, 2012/08/13
[Qemu-devel] [PATCH v9 5/7] block: Convert close calls to qemu_close, Corey Bryant, 2012/08/11
[Qemu-devel] [PATCH v9 7/7] monitor: Clean up fd sets on monitor disconnect, Corey Bryant, 2012/08/11
[Qemu-devel] [PATCH v9 1/7] qemu-char: Add MSG_CMSG_CLOEXEC flag to recvmsg,
Corey Bryant <=