diff options
| author | Raghavendra Bhat <raghavendrabhat@gluster.com> | 2012-04-16 14:50:16 +0530 |
|---|---|---|
| committer | Vijay Bellur <vijay@gluster.com> | 2012-04-18 23:53:51 -0700 |
| commit | f8bbf1cef9da21ea55649f5a05a88479853cd8eb (patch) | |
| tree | 1cb0965e2e91ad63eeef3e2f71da654cb201949c /cli/src/cli-cmd-peer.c | |
| parent | 678efee04855f88919904d98176b0d7c44e543d6 (diff) | |
cli: do not put the frame->local to mempool since its a calloced structure
In cli, frame->local is not allocated from the internal mempool, rather its
calloced. So while destroying the stack, we have to set frame->local to NULL.
Otherwise if frame->local is not NULL, STACK_DESTROY will try to put it back
to the mempool and thus segfault.
Change-Id: I5bd17d099e16fb8e73f3250cc022008e26e28f50
BUG: 812735
Signed-off-by: Raghavendra Bhat <raghavendrabhat@gluster.com>
Reviewed-on: http://review.gluster.com/3167
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Amar Tumballi <amarts@redhat.com>
Diffstat (limited to 'cli/src/cli-cmd-peer.c')
| -rw-r--r-- | cli/src/cli-cmd-peer.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/cli/src/cli-cmd-peer.c b/cli/src/cli-cmd-peer.c index 60756f31085..4ac1630e557 100644 --- a/cli/src/cli-cmd-peer.c +++ b/cli/src/cli-cmd-peer.c @@ -96,8 +96,8 @@ out: if ((sent == 0) && (parse_error == 0)) cli_out ("Peer probe failed"); } - if (frame) - STACK_DESTROY (frame->root); + + CLI_STACK_DESTROY (frame); return ret; } @@ -164,8 +164,7 @@ out: cli_out ("Peer detach failed"); } - if (frame) - STACK_DESTROY (frame->root); + CLI_STACK_DESTROY (frame); return ret; } @@ -203,8 +202,7 @@ out: cli_out ("Peer status failed"); } - if (frame) - STACK_DESTROY (frame->root); + CLI_STACK_DESTROY (frame); return ret; } |