]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
__inode_security_revalidate() never gets NULL opt_dentry
authorAl Viro <viro@zeniv.linux.org.uk>
Wed, 25 Apr 2018 01:31:02 +0000 (21:31 -0400)
committerAl Viro <viro@zeniv.linux.org.uk>
Mon, 14 May 2018 13:51:42 +0000 (09:51 -0400)
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
security/selinux/hooks.c

index 4cafe6a19167613cb64b29ac59c895e91285b390..b02315183b2f5063eac075b4c00645f407dc2a2d 100644 (file)
@@ -274,11 +274,10 @@ static int inode_doinit_with_dentry(struct inode *inode, struct dentry *opt_dent
  * Try reloading inode security labels that have been marked as invalid.  The
  * @may_sleep parameter indicates when sleeping and thus reloading labels is
  * allowed; when set to false, returns -ECHILD when the label is
- * invalid.  The @opt_dentry parameter should be set to a dentry of the inode;
- * when no dentry is available, set it to NULL instead.
+ * invalid.  The @dentry parameter should be set to a dentry of the inode.
  */
 static int __inode_security_revalidate(struct inode *inode,
-                                      struct dentry *opt_dentry,
+                                      struct dentry *dentry,
                                       bool may_sleep)
 {
        struct inode_security_struct *isec = inode->i_security;
@@ -295,7 +294,7 @@ static int __inode_security_revalidate(struct inode *inode,
                 * @opt_dentry is NULL and no dentry for this inode can be
                 * found; in that case, continue using the old label.
                 */
-               inode_doinit_with_dentry(inode, opt_dentry);
+               inode_doinit_with_dentry(inode, dentry);
        }
        return 0;
 }