Commit 47b1e343 authored by Tyler Hicks's avatar Tyler Hicks Committed by Dominique Martinet
Browse files

9p: Remove unnecessary variable for old fids while walking from d_parent 

Remove the ofid variable that's local to the conditional block in favor
of the old_fid variable that's local to the entire function.

Link: https://lkml.kernel.org/r/20220527000003.355812-5-tyhicks@linux.microsoft.com


Signed-off-by: default avatarTyler Hicks <tyhicks@linux.microsoft.com>
Signed-off-by: default avatarDominique Martinet <asmadeus@codewreck.org>
parent c58c72d3

fs/9p/fid.c

+3 −3
Original line number Diff line number Diff line
@@ -169,10 +169,10 @@ static struct p9_fid *v9fs_fid_lookup_with_uid(struct dentry *dentry, 
	fid = v9fs_fid_find(ds, uid, any);

	if (fid) {

		/* Found the parent fid do a lookup with that */

		struct p9_fid *ofid = fid;

		old_fid = fid;



		fid = p9_client_walk(ofid, 1, &dentry->d_name.name, 1);

		p9_client_clunk(ofid);

		fid = p9_client_walk(old_fid, 1, &dentry->d_name.name, 1);

		p9_client_clunk(old_fid);

		goto fid_out;

	}

	up_read(&v9ses->rename_sem);