From: Tom Rix <trix@redhat.com>
Date: Mon, 17 Jan 2022 17:50:10 +0000 (-0800)
Subject: NFS: simplify check for freeing cn_resp
X-Git-Tag: howlett/maple/20220722_2~866^2~90
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=98c27f276be85a73f0babc61c9f8128b7ef593c6;p=users%2Fjedix%2Flinux-maple.git

NFS: simplify check for freeing cn_resp

nfs42_files_from_same_server() is called to check if freeing
cn_resp is required, just do the free.

Signed-off-by: Tom Rix <trix@redhat.com>
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
---

diff --git a/fs/nfs/nfs4file.c b/fs/nfs/nfs4file.c
index e79ae4cbc395..ba117592a95b 100644
--- a/fs/nfs/nfs4file.c
+++ b/fs/nfs/nfs4file.c
@@ -180,8 +180,8 @@ retry:
 	ret = nfs42_proc_copy(file_in, pos_in, file_out, pos_out, count,
 				nss, cnrs, sync);
 out:
-	if (!nfs42_files_from_same_server(file_in, file_out))
-		kfree(cn_resp);
+	kfree(cn_resp);
+
 	if (ret == -EAGAIN)
 		goto retry;
 	return ret;