From e5cc7d469f09faa2f6666d61b7c719136e959d09 Mon Sep 17 00:00:00 2001 From: NeilBrown Date: Tue, 15 Mar 2011 14:54:46 +1100 Subject: [PATCH] ddf: use correct loop variable in activate_spare Using 'i' when you mean 'j' just shows how silly it is to use variables named 'i' and 'j'. Signed-off-by: NeilBrown --- super-ddf.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/super-ddf.c b/super-ddf.c index 96d75ea..9013319 100644 --- a/super-ddf.c +++ b/super-ddf.c @@ -3619,13 +3619,14 @@ static struct mdinfo *ddf_activate_spare(struct active_array *a, esize = ex[j].start - pos; if (esize >= a->info.component_size) break; - pos = ex[i].start + ex[i].size; - i++; - } while (ex[i-1].size); + pos = ex[j].start + ex[j].size; + j++; + } while (ex[j-1].size); free(ex); if (esize < a->info.component_size) { - dprintf("%x:%x has no room: %llu %llu\n", dl->major, dl->minor, + dprintf("%x:%x has no room: %llu %llu\n", + dl->major, dl->minor, esize, a->info.component_size); /* No room */ continue;