KekenoBSD/src
3
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

namei: Preserve ABI root for absolute symlinks before fallback

Browse Source 
D40479 changed namei() so that an absolute symlink target
encountered during an ABI-root lookup restarts from the native root.
This helps the native fallback case, but it also makes successful
lookups inside an ABI root escape that root while following absolute
symlinks.

Only switch absolute symlink lookup to the native root after namei()
is already in the restarted/native fallback pass.
Do not mark the lookup as restarted merely because an absolute
symlink was encountered while still resolving inside the ABI root.

This preserves the intended native fallback behavior while keeping
absolute symlinks within a successfully resolved ABI-root path in
the ABI namespace.

Signed-off-by:	Ricardo Branco <rbranco@suse.de>
PR:		289739
Reviewed by:	kib
Fixes:		cea7c564c7 ("namei: Reset the lookup to ...")
Pull-Request:	https://github.com/freebsd/freebsd-src/pull/2166
This commit is contained in:
Ricardo Branco
2026-04-26 18:44:39 +02:00
committed by Pouria Mousavizadeh Tehrani
parent 19ff93c921
commit 459ac3044b
1 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
sys/kern/vfs_lookup.c
+9 -5
View File
@@ -778,13 +778,17 @@ namei(struct nameidata *ndp)
cnp->cn_nameptr = cnp->cn_pnbuf;
if (*(cnp->cn_nameptr) == '/') {
/*
* Reset the lookup to start from the real root without
* origin path name reloading.
* For ABI-root lookups, preserve the ABI root while
* following absolute symlinks during the first lookup.
*
* Only force the real root after the ABI lookup has
* already failed and namei() has restarted in the
* native namespace. Otherwise absolute symlinks inside
* /compat/linux, including the ELF interpreter symlink,
* incorrectly escape to the native root (PR 289739).
*/
if (__predict_false(ndp->ni_rootdir != pwd->pwd_rdir)) {
cnp->cn_flags |= ISRESTARTED;
if ((cnp->cn_flags & ISRESTARTED) != 0)
ndp->ni_rootdir = pwd->pwd_rdir;
}
vrele(dp);
error = namei_handle_root(ndp, &dp);
if (error != 0)