diff options
author | Jeff Layton <jlayton@redhat.com> | 2012-10-10 15:25:20 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-10-12 00:31:59 -0400 |
commit | 1c2e51e8c162417d2831007ec256ede06c3a0201 (patch) | |
tree | c21a641405f7de220f16be4d2e74fec387ec92a9 /kernel/auditsc.c | |
parent | f78570dd6ad9563fffd24cc5e1808e1a1242f16e (diff) |
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>
Diffstat (limited to 'kernel/auditsc.c')
-rw-r--r-- | kernel/auditsc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/auditsc.c b/kernel/auditsc.c index f4a7756f999c..4d1bd62b090b 100644 --- a/kernel/auditsc.c +++ b/kernel/auditsc.c | |||
@@ -2212,7 +2212,7 @@ void __audit_inode_child(const struct dentry *dentry, | |||
2212 | if (!strcmp(dname, n->name) || | 2212 | if (!strcmp(dname, n->name) || |
2213 | !audit_compare_dname_path(dname, n->name, &dirlen)) { | 2213 | !audit_compare_dname_path(dname, n->name, &dirlen)) { |
2214 | if (inode) | 2214 | if (inode) |
2215 | audit_copy_inode(n, NULL, inode); | 2215 | audit_copy_inode(n, dentry, inode); |
2216 | else | 2216 | else |
2217 | n->ino = (unsigned long)-1; | 2217 | n->ino = (unsigned long)-1; |
2218 | found_child = n->name; | 2218 | found_child = n->name; |
@@ -2244,7 +2244,7 @@ add_names: | |||
2244 | } | 2244 | } |
2245 | 2245 | ||
2246 | if (inode) | 2246 | if (inode) |
2247 | audit_copy_inode(n, NULL, inode); | 2247 | audit_copy_inode(n, dentry, inode); |
2248 | } | 2248 | } |
2249 | } | 2249 | } |
2250 | EXPORT_SYMBOL_GPL(__audit_inode_child); | 2250 | EXPORT_SYMBOL_GPL(__audit_inode_child); |