From 0e16b52aecb9efb40df3384931b2f3afe0d44466 Mon Sep 17 00:00:00 2001 From: dfr Date: Wed, 16 Apr 2008 09:09:50 +0000 Subject: Tighten up the error-handling in nlm_get_rpc. While I'm here, fix a couple of spelling mistakes in comments. --- sys/nlm/nlm_prot_impl.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) (limited to 'sys/nlm') diff --git a/sys/nlm/nlm_prot_impl.c b/sys/nlm/nlm_prot_impl.c index 626fa02..d48b8a5 100644 --- a/sys/nlm/nlm_prot_impl.c +++ b/sys/nlm/nlm_prot_impl.c @@ -144,7 +144,7 @@ static CLIENT *nlm_lockd; */ /* - * A pending asynchronous lock request, stored on the nc_pending list + * A pending asynchronous lock request, stored on the nh_pending list * of the NLM host. */ struct nlm_async_lock { @@ -323,6 +323,10 @@ again: */ struct netbuf *a; a = __rpc_uaddr2taddr_af(ss.ss_family, uaddr); + if (!a) { + CLNT_DESTROY(rpcb); + return (NULL); + } memcpy(&ss, a->buf, a->len); free(a->buf, M_RPC); free(a, M_RPC); @@ -337,7 +341,6 @@ again: rpcvers = PMAPVERS; CLNT_CONTROL(rpcb, CLSET_VERS, &rpcvers); - mapping.pm_prog = parms.r_prog; mapping.pm_vers = parms.r_vers; mapping.pm_prot = IPPROTO_UDP; @@ -366,6 +369,7 @@ again: if (stat != RPC_SUCCESS) { printf("NLM: failed to contact remote rpcbind, stat = %d\n", (int) stat); + CLNT_DESTROY(rpcb); return (NULL); } @@ -399,7 +403,7 @@ nlm_lock_callback(void *arg, int pending) * Send the results back to the host. * * Note: there is a possible race here with nlm_host_notify - * destroying teh RPC client. To avoid problems, the first + * destroying the RPC client. To avoid problems, the first * thing nlm_host_notify does is to cancel pending async lock * requests. */ -- cgit v1.1