diff options
author | Akinobu Mita <akinobu.mita@gmail.com> | 2006-11-27 01:16:48 -0500 |
---|---|---|
committer | James Morris <jmorris@namei.org> | 2006-11-27 10:22:43 -0500 |
commit | fc5d81e69d15c65ca20d9e5b4e242690e3e9c27d (patch) | |
tree | 487e7c5e25fb91246712747cc9595f750cffa30b /security/selinux/hooks.c | |
parent | 2ea5814472c3c910aed5c5b60f1f3b1000e353f1 (diff) |
selinux: fix dentry_open() error check
The return value of dentry_open() shoud be checked by IS_ERR().
Cc: Stephen Smalley <sds@tycho.nsa.gov>
Cc: James Morris <jmorris@namei.org>
Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
Signed-off-by: James Morris <jmorris@namei.org>
Diffstat (limited to 'security/selinux/hooks.c')
-rw-r--r-- | security/selinux/hooks.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c index 8ab5679a37a3..28ee187ed224 100644 --- a/security/selinux/hooks.c +++ b/security/selinux/hooks.c | |||
@@ -1754,7 +1754,8 @@ static inline void flush_unauthorized_files(struct files_struct * files) | |||
1754 | get_file(devnull); | 1754 | get_file(devnull); |
1755 | } else { | 1755 | } else { |
1756 | devnull = dentry_open(dget(selinux_null), mntget(selinuxfs_mount), O_RDWR); | 1756 | devnull = dentry_open(dget(selinux_null), mntget(selinuxfs_mount), O_RDWR); |
1757 | if (!devnull) { | 1757 | if (IS_ERR(devnull)) { |
1758 | devnull = NULL; | ||
1758 | put_unused_fd(fd); | 1759 | put_unused_fd(fd); |
1759 | fput(file); | 1760 | fput(file); |
1760 | continue; | 1761 | continue; |