about summary refs log tree commit diff
path: root/posix/spawn_faction_adddup2.c
diff options
context:
space:
mode:
authorFlorian Weimer <fweimer@redhat.com>2016-02-19 14:49:38 +0100
committerFlorian Weimer <fweimer@redhat.com>2016-02-19 14:49:38 +0100
commit6793dd51d8e7d4f3408c183c6f7dc86acd7f1330 (patch)
tree7d8f6b314549b47a510f3cfb387743f8fd4e7001 /posix/spawn_faction_adddup2.c
parentff889b196575c2fbf6aa7130abb1ec862714ea4e (diff)
downloadglibc-6793dd51d8e7d4f3408c183c6f7dc86acd7f1330.tar.gz
glibc-6793dd51d8e7d4f3408c183c6f7dc86acd7f1330.tar.xz
glibc-6793dd51d8e7d4f3408c183c6f7dc86acd7f1330.zip
Improve file descriptor checks for posix_spawn actions [BZ #19505]
Diffstat (limited to 'posix/spawn_faction_adddup2.c')
-rw-r--r--posix/spawn_faction_adddup2.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/posix/spawn_faction_adddup2.c b/posix/spawn_faction_adddup2.c
index 8beee1099b..a04fc521c3 100644
--- a/posix/spawn_faction_adddup2.c
+++ b/posix/spawn_faction_adddup2.c
@@ -27,11 +27,9 @@ int
 posix_spawn_file_actions_adddup2 (posix_spawn_file_actions_t *file_actions,
 				  int fd, int newfd)
 {
-  int maxfd = __sysconf (_SC_OPEN_MAX);
   struct __spawn_action *rec;
 
-  /* Test for the validity of the file descriptor.  */
-  if (fd < 0 || newfd < 0 || fd >= maxfd || newfd >= maxfd)
+  if (!__spawn_valid_fd (fd) || !__spawn_valid_fd (newfd))
     return EBADF;
 
   /* Allocate more memory if needed.  */