summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorM S Vishwanath Bhat <vishwanath@gluster.com>2011-10-14 17:29:41 +0530
committerVijay Bellur <vijay@gluster.com>2011-10-16 23:24:46 -0700
commit90d691d049420f712f5bd1898c75cc66edd1b315 (patch)
tree898f9b622b235ba89333d96697fce838f1e30aa8
parentf9e58c75f6bffabc29872f20688ac3a30a6a2bee (diff)
makes 'volume top' exit with exit status '0' upon success
volume top exit status was not zero even when command was successful Change-Id: I12761cb6f00ad5c851bfee7cf0aaf0b92dfb94f4 BUG: 2882 testcase: run any valid 'volume top' command and check for exit status Reviewed-on: http://review.gluster.com/587 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
-rw-r--r--cli/src/cli-rpc-ops.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c
index 761c67cbbb5..29589591ea6 100644
--- a/cli/src/cli-rpc-ops.c
+++ b/cli/src/cli-rpc-ops.c
@@ -3238,7 +3238,7 @@ gf_cli3_1_top_volume_cbk (struct rpc_req *req, struct iovec *iov,
ret = dict_get_int32 (dict, key, (int32_t*)&top_op);
if (ret)
goto out;
- while (i <= brick_count) {
+ while (i < brick_count) {
i++;
snprintf (brick, sizeof (brick), "%d-brick", i);
ret = dict_get_str (dict, brick, &bricks);