[PATCH 1/2] aufs: deduplicate vfsub_lookup_one_len[_unlocked]()

2021-10-06 Thread Mauricio Faria de Oliveira
Both are identical except for IMustLock() and lookup_one_len[_unlocked](). Deduplicate them w/ an internal function. No functional changes intended. Signed-off-by: Mauricio Faria de Oliveira --- fs/aufs/vfsub.c | 36 +--- 1 file changed, 17 insertions(+), 19

[PATCH 2/2] aufs: vfsub_lookup_one_len[_unlocked](): set struct path.mnt

2021-10-06 Thread Mauricio Faria de Oliveira
This adds a `struct vfsmount *mnt` parameters to vfsub_lookup_one_len[_unlocked]() so that it can pass a correct `struct path.mnt` value up to VFS in vfsub_getattr(). Many other places in aufs do set/pass `mnt` right, particularly seen w/ calls to vfsub_dentry_open(), including another two

[PATCH 0/2] aufs: fix old kernel oops in vfsub_lookup_one_len()

2021-10-06 Thread Mauricio Faria de Oliveira
Hi J. R. Okajima, Here's a fix for what might potentially be a 10 years old bug (it's at least 4 years, 10 months old per a public report [1].) The key change is very simple (set `path.mnt` in vfsub_lookup_one_len()) but its caller chain is large enough (reviewed/modified ~30 functions).

Re: [PATCH 0/2] aufs: fix old kernel oops in vfsub_lookup_one_len()

2021-10-06 Thread hooanon05g
Hi Mauricio, Mauricio Faria de Oliveira: > The key change is very simple (set `path.mnt` in vfsub_lookup_one_len()) > but its caller chain is large enough (reviewed/modified ~30 functions). > Fortunately most of them already had `path.mnt` set or easy to obtain. Thanx for the report. You made a