diff options
| author | Jeff Layton <jlayton@redhat.com> | 2012-10-10 21:25:20 +0200 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-10-12 06:31:59 +0200 |
| commit | 1c2e51e8c162417d2831007ec256ede06c3a0201 (patch) | |
| tree | c21a641405f7de220f16be4d2e74fec387ec92a9 /kernel | |
| parent | audit: remove unnecessary NULL ptr checks from do_path_lookup (diff) | |
| download | linux-1c2e51e8c162417d2831007ec256ede06c3a0201.tar.xz linux-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>
Diffstat (limited to 'kernel')
| -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, 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); |