summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeff Layton <jlayton@redhat.com>2012-10-10 15:25:20 -0400
committerAl Viro <viro@zeniv.linux.org.uk>2012-10-12 00:31:59 -0400
commit1c2e51e8c162417d2831007ec256ede06c3a0201 (patch)
treec21a641405f7de220f16be4d2e74fec387ec92a9
parentf78570dd6ad9563fffd24cc5e1808e1a1242f16e (diff)
downloadlinux-3.10-1c2e51e8c162417d2831007ec256ede06c3a0201.tar.gz
linux-3.10-1c2e51e8c162417d2831007ec256ede06c3a0201.tar.bz2
linux-3.10-1c2e51e8c162417d2831007ec256ede06c3a0201.zip
audit: pass in dentry to audit_copy_inode wherever possible
In some cases, we were passing in NULL even when we have a dentry. Reported-by: Eric Paris <eparis@redhat.com> Signed-off-by: Jeff Layton <jlayton@redhat.com> Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r--kernel/auditsc.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/auditsc.c b/kernel/auditsc.c
index f4a7756f999..4d1bd62b090 100644
--- a/kernel/auditsc.c
+++ b/kernel/auditsc.c
@@ -2212,7 +2212,7 @@ void __audit_inode_child(const struct dentry *dentry,
if (!strcmp(dname, n->name) ||
!audit_compare_dname_path(dname, n->name, &dirlen)) {
if (inode)
- audit_copy_inode(n, NULL, inode);
+ audit_copy_inode(n, dentry, inode);
else
n->ino = (unsigned long)-1;
found_child = n->name;
@@ -2244,7 +2244,7 @@ add_names:
}
if (inode)
- audit_copy_inode(n, NULL, inode);
+ audit_copy_inode(n, dentry, inode);
}
}
EXPORT_SYMBOL_GPL(__audit_inode_child);