From fb125e0ced340f0f0126cdcd3a230188007ef3f4 Mon Sep 17 00:00:00 2001 From: Ulrich Drepper Date: Wed, 12 Apr 2000 06:08:31 +0000 Subject: [PATCH] Update. * sysdeps/unix/sysv/linux/shm_open.c (shm_open): Set FD_CLOEXEC for descriptor. --- ChangeLog | 3 +++ sysdeps/unix/sysv/linux/shm_open.c | 25 ++++++++++++++++++++++++- 2 files changed, 27 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index fcecf36..61e7fd5 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,8 @@ 2000-04-11 Ulrich Drepper + * sysdeps/unix/sysv/linux/shm_open.c (shm_open): Set FD_CLOEXEC + for descriptor. + * misc/sys/mman.h: Add prototypes for shm_open and shm_unlink. * rt/Makefile (librt-routines): Add shm_open and shm_unlink. * rt/Versions [librt] (GLIBC_2.2): Add shm_open and shm_unlink. diff --git a/sysdeps/unix/sysv/linux/shm_open.c b/sysdeps/unix/sysv/linux/shm_open.c index 08bdaea..ca24424 100644 --- a/sysdeps/unix/sysv/linux/shm_open.c +++ b/sysdeps/unix/sysv/linux/shm_open.c @@ -121,6 +121,7 @@ shm_open (const char *name, int oflag, mode_t mode) { size_t namelen; char *fname; + int fd; /* Determine where the shmfs is mounted. */ __libc_once (once, where_is_shmfs); @@ -153,7 +154,29 @@ shm_open (const char *name, int oflag, mode_t mode) file on the shmfs. If this is what should be done the whole function should be revamped since we can determine whether shmfs is available while trying to open the file, all in one turn. */ - return open (fname, oflag, mode); + fd = open (fname, oflag, mode); + if (fd != -1) + { + /* We got a descriptor. Now set the FD_CLOEXEC bit. */ + int flags = fcntl (fd, F_GETFD, 0); + + if (__builtin_expect (flags, 0) >= 0) + { + flags |= FD_CLOEXEC; + flags = fcntl (fd, F_SETFD, flags); + } + + if (flags == -1) + { + /* Something went wrong. We cannot return the descriptor. */ + int save_errno = errno; + close (fd); + fd = -1; + __set_errno (save_errno); + } + } + + return fd; } -- 2.7.4