bybrooklyn/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-05-05 23:05:25 -04:00
Code Issues Packages Projects Releases Wiki Activity

do_loopback(): use __free(path_put) to deal with old_path

Browse Source 
preparations for making unlock_mount() a __cleanup();
can't have path_put() inside mount_lock scope.

Reviewed-by: Christian Brauner <brauner@kernel.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
This commit is contained in:
Al Viro
2025-08-21 22:28:51 -04:00
parent 11941610b0
commit 76dfde13d6
1 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
fs/namespace.c
View File

@@ -3014,7 +3014,7 @@ static struct mount *__do_loopback(struct path *old_path, int recurse)
static int do_loopback(struct path *path, const char *old_name,
int recurse)
{
struct path old_path;
struct path old_path __free(path_put) = {};
struct mount *mnt = NULL, *parent;
struct pinned_mountpoint mp = {};
int err;
@@ -3024,13 +3024,12 @@ static int do_loopback(struct path *path, const char *old_name,
if (err)
return err;
err = -EINVAL;
if (mnt_ns_loop(old_path.dentry))
goto out;
return -EINVAL;
err = lock_mount(path, &mp);
if (err)
goto out;
return err;
parent = real_mount(path->mnt);
if (!check_mnt(parent))
@@ -3050,8 +3049,6 @@ static int do_loopback(struct path *path, const char *old_name,
}
out2:
unlock_mount(&mp);
out:
path_put(&old_path);
return err;
}