diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2012-08-27 14:48:26 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-09-26 21:10:25 -0400 |
commit | cb0942b81249798e15c3f04eee2946ef543e8115 (patch) | |
tree | 7d494c580a847342577661782c5173f76178aa81 /security | |
parent | cecb46f194460d23cacf3b13593f9f5a4f7a0fed (diff) |
make get_file() return its argument
simplifies a bunch of callers...
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'security')
-rw-r--r-- | security/selinux/hooks.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c index 4dfbcea10eb7..651d8456611a 100644 --- a/security/selinux/hooks.c +++ b/security/selinux/hooks.c | |||
@@ -2135,8 +2135,7 @@ static inline void flush_unauthorized_files(const struct cred *cred, | |||
2135 | if (!IS_ERR(devnull)) { | 2135 | if (!IS_ERR(devnull)) { |
2136 | /* replace all the matching ones with this */ | 2136 | /* replace all the matching ones with this */ |
2137 | do { | 2137 | do { |
2138 | get_file(devnull); | 2138 | replace_fd(n - 1, get_file(devnull), 0); |
2139 | replace_fd(n - 1, devnull, 0); | ||
2140 | } while ((n = iterate_fd(files, n, match_file, cred)) != 0); | 2139 | } while ((n = iterate_fd(files, n, match_file, cred)) != 0); |
2141 | fput(devnull); | 2140 | fput(devnull); |
2142 | } else { | 2141 | } else { |