diff options
| author | Sanju Rakonde <srakonde@redhat.com> | 2018-07-24 14:27:02 +0530 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2018-07-24 23:55:12 +0000 |
| commit | 8ad159b2a7e302320a24d1d4f5d0b90302e0f25b (patch) | |
| tree | 05d90618ff41ecd8d70f94473564ab38dfca9ea6 /xlators/mgmt/glusterd/src/glusterd-handshake.c | |
| parent | 59401e705b9432345f3b449b50cfa4095ce2169a (diff) | |
glusterd: Coverity issues with type FORWARD_NULL
This patch fixes coverity issues 102, 103, 112 and 119 from [1]
[1] https://download.gluster.org/pub/gluster/glusterfs/static-analysis/master/glusterfs-coverity/2018-07-23-5fa004f3/html/
Updates: bz#789278
Change-Id: I99762eb0bcbd974a5250434777db63520f2ce2e6
Signed-off-by: Sanju Rakonde <srakonde@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-handshake.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-handshake.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-handshake.c b/xlators/mgmt/glusterd/src/glusterd-handshake.c index 35b6bed409e..f2614f4db86 100644 --- a/xlators/mgmt/glusterd/src/glusterd-handshake.c +++ b/xlators/mgmt/glusterd/src/glusterd-handshake.c @@ -1828,9 +1828,10 @@ gd_validate_peer_op_version (xlator_t *this, glusterd_peerinfo_t *peerinfo, ret = 0; out: - gf_msg_debug ((this ? this->name : "glusterd") , 0, "Peer %s %s", - peerinfo->hostname, ((ret < 0) ? "rejected" : "accepted") - ); + if (peerinfo) + gf_msg_debug ((this ? this->name : "glusterd") , 0, "Peer %s %s", + peerinfo->hostname, + ((ret < 0) ? "rejected" : "accepted")); return ret; } |