On Mon, Jun 23, 2025 at 08:22:28PM +0530, Venkat Rao Bagalkote wrote:
>
> On 23/06/25 7:26 pm, Al Viro wrote:
> > On Mon, Jun 23, 2025 at 07:20:03PM +0530, Venkat Rao Bagalkote wrote:
> >
> > [NULL pointer dereference somewhere in collect_paths()]
> >
> > Could you put objdump -d of the function in question somewhere?
> > Or just fs/namespace.o from your build...
> >
>
> Attached is the namespace.o file.
Huh...
That looks like NULL first argument (path), which blows up on
struct mount *root = real_mount(path->mnt);
just prior to grabbing namespace_sem...
*blinks*
<obscenities>
Could you check if the delta below fixes it?
diff --git a/kernel/audit_tree.c b/kernel/audit_tree.c
index 68e042ae93c7..b0eae2a3c895 100644
--- a/kernel/audit_tree.c
+++ b/kernel/audit_tree.c
@@ -832,7 +832,7 @@ int audit_add_tree_rule(struct audit_krule *rule)
err = kern_path(tree->pathname, 0, &path);
if (err)
goto Err;
- paths = collect_paths(paths, array, 16);
+ paths = collect_paths(&path, array, 16);
path_put(&path);
if (IS_ERR(paths)) {
err = PTR_ERR(paths);
