From ef169f5029bd31601d691cbee5ef5c0117bfcfef Mon Sep 17 00:00:00 2001
From: N Balachandran <nbalacha@redhat.com>
Date: Mon, 24 Jul 2017 17:48:47 +0530
Subject: [PATCH 572/576] cluster/dht: Correct iterator for decommissioned
 bricks

Corrected the iterator for looping over the list of
decommissioned bricks while checking if the new target
determined because of min-free-disk values has been
decommissioned.

> BUG: 1474318
> Signed-off-by: N Balachandran <nbalacha@redhat.com>
> Reviewed-on: https://review.gluster.org/17861
> Reviewed-by: Susant Palai <spalai@redhat.com>

Change-Id: Iee778547eb7370a8069e954b5d629fcedf54e59b
BUG: 1474284
Signed-off-by: N Balachandran <nbalacha@redhat.com>
Reviewed-on: https://code.engineering.redhat.com/gerrit/113511
Reviewed-by: Susant Palai <spalai@redhat.com>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
---
 xlators/cluster/dht/src/dht-rebalance.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xlators/cluster/dht/src/dht-rebalance.c b/xlators/cluster/dht/src/dht-rebalance.c
index c60ce27..9ae4062 100644
--- a/xlators/cluster/dht/src/dht-rebalance.c
+++ b/xlators/cluster/dht/src/dht-rebalance.c
@@ -1023,7 +1023,7 @@ find_new_subvol:
                  * removed-brick, do not mark the error as failure */
                 if (conf->decommission_subvols_cnt) {
                         *ignore_failure = _gf_true;
-                        for (i = 0; i < conf->decommission_subvols_cnt; i++) {
+                        for (i = 0; i < conf->subvolume_cnt; i++) {
                                 if (conf->decommissioned_bricks[i] == from) {
                                         *ignore_failure = _gf_false;
                                          break;
-- 
1.8.3.1