From 1909f2ca01efcd467164a2d7ed684cdc6e1437de Mon Sep 17 00:00:00 2001 From: Sunny Kumar Date: Fri, 10 Aug 2018 02:39:55 +0530 Subject: nfs-server-mount : fix coverity issues in mount3.c Fixes CID 1389033, 1388767, 1288782. Change-Id: I244f88b2ca8487f8926da45d886982558ad45c7a updates: bz#789278 Signed-off-by: Sunny Kumar --- xlators/nfs/server/src/mount3.c | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) (limited to 'xlators/nfs/server/src') diff --git a/xlators/nfs/server/src/mount3.c b/xlators/nfs/server/src/mount3.c index 472915fb067..ad52c6b7817 100644 --- a/xlators/nfs/server/src/mount3.c +++ b/xlators/nfs/server/src/mount3.c @@ -1757,11 +1757,11 @@ mnt3_check_client_net_udp (struct svc_req *req, char *volname, xlator_t *nfsx) * address family. Sigh. */ #endif - /* And let's make sure that it's actually an IPv4 address. */ - GF_ASSERT (sin->sin_family == AF_INET); - if (!sin) goto err; + /* And let's make sure that it's actually an IPv4 address. */ + + GF_ASSERT (sin->sin_family == AF_INET); (void) inet_ntop (AF_INET, &sin->sin_addr, ipaddr, INET_ADDRSTRLEN); @@ -1997,9 +1997,13 @@ _mnt3_authenticate_req (struct mount3_state *ms, rpcsvc_request_t *req, GF_VALIDATE_OR_GOTO (GF_MNT, req, out); peer_addr = _mnt3_get_peer_addr (req); + + if (!peer_addr) + goto free_and_out; + host_addr_ip = _mnt3_get_host_from_peer (peer_addr); - if (!host_addr_ip || !peer_addr) + if (!host_addr_ip) goto free_and_out; if (path) { @@ -2713,7 +2717,7 @@ mnt3_xlchildren_to_exports (rpcsvc_t *svc, struct mount3_state *ms) /* chain the groups together */ if (!elist->ex_groups) elist->ex_groups = group; - else + else if (!prev_group) prev_group->gr_next = group; prev_group = group; } -- cgit