NFSD: fix problems with cleanup on errors in nfsd4_copy
authorDai Ngo <dai.ngo@oracle.com>
Tue, 31 Jan 2023 19:12:29 +0000 (11:12 -0800)
committerChuck Lever <chuck.lever@oracle.com>
Mon, 20 Feb 2023 14:20:57 +0000 (09:20 -0500)
commit81e722978ad21072470b73d8f6a50ad62c7d5b7d
treeaf1ab500dab0f4b9d25b6571277649a1d1b7f6d2
parentfb610c4dbc996415d57d7090957ecddd4fd64fb6
NFSD: fix problems with cleanup on errors in nfsd4_copy

When nfsd4_copy fails to allocate memory for async_copy->cp_src, or
nfs4_init_copy_state fails, it calls cleanup_async_copy to do the
cleanup for the async_copy which causes page fault since async_copy
is not yet initialized.

This patche rearranges the order of initializing the fields in
async_copy and adds checks in cleanup_async_copy to skip un-initialized
fields.

Fixes: ce0887ac96d3 ("NFSD add nfs4 inter ssc to nfsd4_copy")
Fixes: 87689df69491 ("NFSD: Shrink size of struct nfsd4_copy")
Signed-off-by: Dai Ngo <dai.ngo@oracle.com>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
fs/nfsd/nfs4proc.c
fs/nfsd/nfs4state.c