]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
hurd: Don't pass FD_CLOEXEC in CMSG_DATA
authorSamuel Thibault <samuel.thibault@ens-lyon.org>
Mon, 24 Apr 2023 21:05:15 +0000 (23:05 +0200)
committerSamuel Thibault <samuel.thibault@ens-lyon.org>
Mon, 24 Apr 2023 21:05:15 +0000 (23:05 +0200)
The flags are used by _hurd_intern_fd, which takes O_* flags, not FD_*.

Also, it is of no concern to the receiving process whether or not
the sender process wants to close its copy of sent file descriptor
upon exec, and it should not influence whether or not the received
file descriptor gets the FD_CLOEXEC flag set in the receiving process.

The latter should in fact be dependent on the MSG_CMSG_CLOEXEC flag
being passed to the recvmsg () call, which is going to be implemented
in the following commit.

Fixes 344e755248ce02c0f8d095d11cc49e340703d926
"hurd: Support sending file descriptors over Unix sockets"

Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
Signed-off-by: Sergey Bugaev <bugaevc@gmail.com>
sysdeps/mach/hurd/sendmsg.c

index 7106faf0dedf1f200471a43eb9ac8667b54ec242..3c9cdc4e66a796ae5f1136375368fd1b1966edf6 100644 (file)
@@ -138,8 +138,8 @@ __libc_sendmsg (int fd, const struct msghdr *message, int flags)
                                             0, 0, 0, 0);
                   if (! err)
                     nports++;
-                  /* We pass the flags in the control data.  */
-                  fds[i] = descriptor->flags;
+                  /* We do not currently have flags to pass.  */
+                  fds[i] = 0;
                   err;
                 }));