Commit 47641248 authored by Chuck Lever's avatar Chuck Lever
Browse files

lockd: Use xdrgen XDR functions for the NLMv4 TEST_RES procedure 



Convert the TEST_RES procedure to use xdrgen functions
nlm4_svc_decode_nlm4_testres and nlm4_svc_encode_void.
TEST_RES is a callback procedure where the client sends
test lock results back to the server after an async TEST
request.

The pc_argzero field is set to zero because xdrgen decoders
reliably initialize all arguments, making the early
defensive memset unnecessary.

This change also corrects the pc_xdrressize field, which
previously contained a placeholder value.

Reviewed-by: default avatarJeff Layton <jlayton@kernel.org>
Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
parent 62721885

fs/lockd/svc4proc.c

+9 −9
Original line number Diff line number Diff line
@@ -1230,14 +1230,14 @@ static const struct svc_procedure nlm4svc_procedures[24] = { 
		.pc_xdrressize	= XDR_void,

		.pc_name	= "GRANTED_MSG",

	},

	[NLMPROC_TEST_RES] = {

	[NLMPROC4_TEST_RES] = {

		.pc_func	= nlm4svc_proc_null,

		.pc_decode = nlm4svc_decode_void,

		.pc_encode = nlm4svc_encode_void,

		.pc_argsize = sizeof(struct nlm_res),

		.pc_argzero = sizeof(struct nlm_res),

		.pc_ressize = sizeof(struct nlm_void),

		.pc_xdrressize = St,

		.pc_decode	= nlm4_svc_decode_nlm4_testres,

		.pc_encode	= nlm4_svc_encode_void,

		.pc_argsize	= sizeof(struct nlm4_testres),

		.pc_argzero	= 0,

		.pc_ressize	= 0,

		.pc_xdrressize	= XDR_void,

		.pc_name	= "TEST_RES",

	},

	[NLMPROC_LOCK_RES] = {