diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2012-08-12 16:17:59 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-09-26 21:08:53 -0400 |
commit | f33ff9927f42045116d738ee47ff7bc59f739bd7 (patch) | |
tree | 917e0d7c10392a884a0aeda8733bdd2ef8e2c105 /fs/fcntl.c | |
parent | 352e3b249284235e00745f3e71fc348b913e5deb (diff) |
take rlimit check to callers of expand_files()
... except for one in android, where the check is different
and already done in caller. No need to recalculate rlimit
many times in alloc_fd() either.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/fcntl.c')
-rw-r--r-- | fs/fcntl.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/fs/fcntl.c b/fs/fcntl.c index 887b5ba8c9b5..08e6af5c1b1f 100644 --- a/fs/fcntl.c +++ b/fs/fcntl.c | |||
@@ -64,6 +64,9 @@ SYSCALL_DEFINE3(dup3, unsigned int, oldfd, unsigned int, newfd, int, flags) | |||
64 | if (unlikely(oldfd == newfd)) | 64 | if (unlikely(oldfd == newfd)) |
65 | return -EINVAL; | 65 | return -EINVAL; |
66 | 66 | ||
67 | if (newfd >= rlimit(RLIMIT_NOFILE)) | ||
68 | return -EMFILE; | ||
69 | |||
67 | spin_lock(&files->file_lock); | 70 | spin_lock(&files->file_lock); |
68 | err = expand_files(files, newfd); | 71 | err = expand_files(files, newfd); |
69 | file = fcheck(oldfd); | 72 | file = fcheck(oldfd); |