This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
2008-12-17 Thomas Schwinge <tschwinge@gnu.org> * sysdeps/mach/hurd/dup3.c: New file, copy from dup2.c. Evolve it to implement dup3 and do some further code clean-ups. * sysdeps/mach/hurd/dup2.c (__dup2): Reimplement using dup3. Recipe: Copy sysdeps/mach/hurd/dup2.c to sysdeps/mach/hurd/dup3.c. Evolve the latter one to implement dup3: --- sysdeps/mach/hurd/dup2.c.O 2008-12-17 00:55:30.000000000 +0100 +++ sysdeps/mach/hurd/dup3.c 2008-12-17 15:27:40.000000000 +0100 @@ -23,21 +23,28 @@ #include <hurd/fd.h> /* Duplicate FD to FD2, closing the old FD2 and making FD2 be - open on the same file as FD is. Return FD2 or -1. */ + open the same file as FD is which setting flags according to + FLAGS. Return FD2 or -1. */ int -__dup2 (fd, fd2) +dup3 (fd, fd2, flags) int fd; int fd2; + int flags; { struct hurd_fd *d; + /* Both passing flags different from O_CLOEXEC and FD2 being the same as FD + are invalid. */ + if ((flags & ~O_CLOEXEC || fd2 == fd) && + /* ... with the exception in case that dup2 behavior is requested: if FD + is valid and FD2 is already the same then just return it. */ + ! (flags == -1 && fd2 == fd)) + return __hurd_fail (EINVAL); + /* Extract the ports and flags from FD. */ d = _hurd_fd_get (fd); if (d == NULL) - { - errno = EBADF; - return -1; - } + return __hurd_fail (EBADF); HURD_CRITICAL_BEGIN; @@ -45,24 +52,19 @@ __dup2 (fd, fd2) if (d->port.port == MACH_PORT_NULL) { __spin_unlock (&d->port.lock); - errno = EBADF; - fd2 = -1; + fd2 = __hurd_fail (EBADF); } else if (fd2 == fd) - /* FD is valid and FD2 is already the same; just return it. */ __spin_unlock (&d->port.lock); else { struct hurd_userlink ulink, ctty_ulink; - int flags = d->flags; + int d_flags = d->flags; io_t ctty = _hurd_port_get (&d->ctty, &ctty_ulink); io_t port = _hurd_port_locked_get (&d->port, &ulink); /* Unlocks D. */ if (fd2 < 0) - { - errno = EBADF; - fd2 = -1; - } + fd2 = __hurd_fail (EBADF); else { /* Get a hold of the destination descriptor. */ @@ -114,7 +116,11 @@ __dup2 (fd, fd2) /* Install the ports and flags in the new descriptor slot. */ __spin_lock (&d2->port.lock); - d2->flags = flags & ~FD_CLOEXEC; /* Dup clears FD_CLOEXEC. */ + if (flags & O_CLOEXEC) + d2->flags = d_flags | FD_CLOEXEC; + else + /* dup clears FD_CLOEXEC. */ + d2->flags = d_flags & ~FD_CLOEXEC; _hurd_port_set (&d2->ctty, ctty); _hurd_port_locked_set (&d2->port, port); /* Unlocks D2. */ } @@ -130,5 +136,3 @@ __dup2 (fd, fd2) return fd2; } -libc_hidden_def (__dup2) -weak_alias (__dup2, dup2) Reimplement __dup2 using dup3: diff --git a/sysdeps/mach/hurd/dup2.c b/sysdeps/mach/hurd/dup2.c index 3abd30c..1ec8cb6 100644 --- a/sysdeps/mach/hurd/dup2.c +++ b/sysdeps/mach/hurd/dup2.c @@ -16,11 +16,7 @@ Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA. */ -#include <errno.h> -#include <fcntl.h> #include <unistd.h> -#include <hurd.h> -#include <hurd/fd.h> /* Duplicate FD to FD2, closing the old FD2 and making FD2 be open on the same file as FD is. Return FD2 or -1. */ @@ -29,106 +25,13 @@ __dup2 (fd, fd2) int fd; int fd2; { - struct hurd_fd *d; + int flags = 0; - /* Extract the ports and flags from FD. */ - d = _hurd_fd_get (fd); - if (d == NULL) - { - errno = EBADF; - return -1; - } + if (fd2 == fd) + /* See the comment in dup3. */ + flags = -1; - HURD_CRITICAL_BEGIN; - - __spin_lock (&d->port.lock); - if (d->port.port == MACH_PORT_NULL) - { - __spin_unlock (&d->port.lock); - errno = EBADF; - fd2 = -1; - } - else if (fd2 == fd) - /* FD is valid and FD2 is already the same; just return it. */ - __spin_unlock (&d->port.lock); - else - { - struct hurd_userlink ulink, ctty_ulink; - int flags = d->flags; - io_t ctty = _hurd_port_get (&d->ctty, &ctty_ulink); - io_t port = _hurd_port_locked_get (&d->port, &ulink); /* Unlocks D. */ - - if (fd2 < 0) - { - errno = EBADF; - fd2 = -1; - } - else - { - /* Get a hold of the destination descriptor. */ - struct hurd_fd *d2; - - if (fd2 >= _hurd_dtablesize) - { - /* The table is not large enough to hold the destination - descriptor. Enlarge it as necessary to allocate this - descriptor. */ - __mutex_unlock (&_hurd_dtable_lock); - /* We still hold FD1's lock, but this is safe because - _hurd_alloc_fd will only examine the cells starting - at FD2. */ - d2 = _hurd_alloc_fd (NULL, fd2); - if (d2) - __spin_unlock (&d2->port.lock); - __mutex_lock (&_hurd_dtable_lock); - } - else - { - d2 = _hurd_dtable[fd2]; - if (d2 == NULL) - { - /* Must allocate a new one. We don't initialize the port - cells with this call so that if it fails (out of - memory), we will not have already added user - references for the ports, which we would then have to - deallocate. */ - d2 = _hurd_dtable[fd2] = _hurd_new_fd (MACH_PORT_NULL, - MACH_PORT_NULL); - } - } - - if (d2 == NULL) - { - fd2 = -1; - if (errno == EINVAL) - errno = EBADF; /* POSIX.1-1990 6.2.1.2 ll 54-55. */ - } - else - { - /* Give the ports each a user ref for the new descriptor. */ - __mach_port_mod_refs (__mach_task_self (), port, - MACH_PORT_RIGHT_SEND, 1); - if (ctty != MACH_PORT_NULL) - __mach_port_mod_refs (__mach_task_self (), ctty, - MACH_PORT_RIGHT_SEND, 1); - - /* Install the ports and flags in the new descriptor slot. */ - __spin_lock (&d2->port.lock); - d2->flags = flags & ~FD_CLOEXEC; /* Dup clears FD_CLOEXEC. */ - _hurd_port_set (&d2->ctty, ctty); - _hurd_port_locked_set (&d2->port, port); /* Unlocks D2. */ - } - } - __mutex_unlock (&_hurd_dtable_lock); - - _hurd_port_free (&d->port, &ulink, port); - if (ctty != MACH_PORT_NULL) - _hurd_port_free (&d->ctty, &ctty_ulink, port); - } - - HURD_CRITICAL_END; - - return fd2; + return dup3 (fd, fd2, flags); } libc_hidden_def (__dup2) weak_alias (__dup2, dup2)
Attachment:
signature.asc
Description: Digital signature
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |