diff options
author | Nigel Babu <nigelb@redhat.com> | 2017-03-01 07:16:19 +0530 |
---|---|---|
committer | Jeff Darcy <jdarcy@redhat.com> | 2017-03-01 12:30:17 -0500 |
commit | 9babd965ba885ca8bea4c58e708a3b9f0d6f076c (patch) | |
tree | 7bd528bf1637947c9f32986759b8612fc0b38183 /rpc/rpc-transport/rdma/src/rdma.c | |
parent | c78d07fb8efdd5f63284332473852619d5da4244 (diff) |
Return ret as soon as `this` is freed
Omitting this return causes a confusing error message in the next block,
which is also a case of use after free. This bug was found by Coverity
scan.
BUG: 789278
Change-Id: Ifd4932de437e8ae875ff191033ea43cff81b701d
Signed-off-by: Nigel Babu <nigelb@redhat.com>
Reviewed-on: https://review.gluster.org/16790
Smoke: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Jeff Darcy <jdarcy@redhat.com>
Diffstat (limited to 'rpc/rpc-transport/rdma/src/rdma.c')
-rw-r--r-- | rpc/rpc-transport/rdma/src/rdma.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/rpc/rpc-transport/rdma/src/rdma.c b/rpc/rpc-transport/rdma/src/rdma.c index d2f04bd6d0c..14b068c05cb 100644 --- a/rpc/rpc-transport/rdma/src/rdma.c +++ b/rpc/rpc-transport/rdma/src/rdma.c @@ -1130,6 +1130,7 @@ gf_rdma_cm_handle_addr_resolved (struct rdma_cm_event *event) "failed (me:%s peer:%s)", this->myinfo.identifier, this->peerinfo.identifier); gf_rdma_cm_handle_disconnect (this); + return ret; } gf_msg_trace (this->name, 0, "Address resolved (me:%s peer:%s)", |