diff options
author | Dan Carpenter <error27@gmail.com> | 2010-06-01 09:14:04 +0200 |
---|---|---|
committer | James Morris <jmorris@namei.org> | 2010-08-02 15:33:39 +1000 |
commit | 3e62cbb8436f6c0cb799c8b7f106de7f662a7b8d (patch) | |
tree | d36565a4a6f7e0372a1fd9a8750b005635c9c335 /fs/open.c | |
parent | c3ef1500ec833890275172c7d063333404b64d60 (diff) | |
download | kernel-common-3e62cbb8436f6c0cb799c8b7f106de7f662a7b8d.tar.gz kernel-common-3e62cbb8436f6c0cb799c8b7f106de7f662a7b8d.tar.bz2 kernel-common-3e62cbb8436f6c0cb799c8b7f106de7f662a7b8d.zip |
smack: opt_dentry is never null in in smack_d_instantiate()
This patch removes some unneeded code for if opt_dentry is null because
that can never happen.
The function dereferences "opt_dentry" earlier when it checks
"if (opt_dentry->d_parent == opt_dentry) {". That code was added in
2008.
This function called from security_d_instantiate(). I checked all the
places which call security_d_instantiate() and dentry is always non-null.
I also checked the selinux version of this hook and there is a comment
which says that dentry should be non-null if called from
d_instantiate().
Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Casey Schaufler <casey@schaufler-ca.com>
Signed-off-by: James Morris <jmorris@namei.org>
Diffstat (limited to 'fs/open.c')
0 files changed, 0 insertions, 0 deletions