From f406fa6394da8ba77ede090032510cab6c674a74 Mon Sep 17 00:00:00 2001 From: ericm Date: Wed, 8 Nov 2006 20:28:16 +0000 Subject: [PATCH] branch: b_new_cmd fix null pointer reference in CERROR. --- lustre/ptlrpc/gss/sec_gss.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/lustre/ptlrpc/gss/sec_gss.c b/lustre/ptlrpc/gss/sec_gss.c index a12e17b..5d4cd4b 100644 --- a/lustre/ptlrpc/gss/sec_gss.c +++ b/lustre/ptlrpc/gss/sec_gss.c @@ -2075,15 +2075,13 @@ int gss_svc_handle_data(struct ptlrpc_request *req, rc = -EINVAL; } - if (rc != 0) - goto error; - - RETURN(SECSVC_OK); + if (rc == 0) + RETURN(SECSVC_OK); -error: CERROR("svc %u failed: major 0x%08x: ctx %p(%u->%s)\n", gw->gw_svc, major, grctx->src_ctx, grctx->src_ctx->gsc_uid, libcfs_nid2str(req->rq_peer.nid)); +error: /* * we only notify client in case of NO_CONTEXT/BAD_SIG, which * might happen after server reboot, to allow recovery. -- 1.8.3.1