diff options
author | Nandaja Varma <nandaja.varma@gmail.com> | 2015-02-06 11:44:22 +0530 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2015-04-10 11:29:42 +0000 |
commit | 26cbd3bdf5dad190559afbdf0ac125262c4e90a6 (patch) | |
tree | 9d8fed20e8d46f33e9dddd321ca014917d473c06 /rpc/rpc-lib/src/rpc-clnt-ping.c | |
parent | 320455bddfbadcfbe1b173b0de54b3b25bf228f7 (diff) |
rpc-lib: Fixing the coverity issues
Coverity CIDs:
1210973
1124887
1124888
1124682
1124849
1124503
Change-Id: I012f6cf9d14753f572ab94aae6d442d1ef8df79a
BUG: 789278
Signed-off-by: Nandaja Varma <nandaja.varma@gmail.com>
Reviewed-on: http://review.gluster.org/9600
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'rpc/rpc-lib/src/rpc-clnt-ping.c')
-rw-r--r-- | rpc/rpc-lib/src/rpc-clnt-ping.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/rpc/rpc-lib/src/rpc-clnt-ping.c b/rpc/rpc-lib/src/rpc-clnt-ping.c index 0b32990e31e..17f8874a6a4 100644 --- a/rpc/rpc-lib/src/rpc-clnt-ping.c +++ b/rpc/rpc-lib/src/rpc-clnt-ping.c @@ -237,7 +237,7 @@ rpc_clnt_ping (struct rpc_clnt *rpc) conn = &rpc->conn; frame = create_frame (THIS, THIS->ctx->pool); if (!frame) - goto fail; + return ret; frame->local = rpc; @@ -258,13 +258,6 @@ rpc_clnt_ping (struct rpc_clnt *rpc) return ret; -fail: - if (frame) { - STACK_DESTROY (frame->root); - } - - return ret; - } static void |