From 47573b0015943efd0564a6db6fd17fd8a7e76b39 Mon Sep 17 00:00:00 2001 From: NeilBrown Date: Tue, 15 Feb 2011 10:45:01 +1100 Subject: [PATCH] Fix regression with removing 'failed' and 'detached' devices. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit If a request to remove all 'failed' or 'detached' devices chooses to remove the first device, it will not actually try the removal and will skip any following devices. This fixes it. Reported-by: Rémi Rérolle Tested-by: Rémi Rérolle Signed-off-by: NeilBrown --- Manage.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Manage.c b/Manage.c index 6e9d4a03..1b474798 100644 --- a/Manage.c +++ b/Manage.c @@ -417,7 +417,7 @@ int Manage_subdevs(char *devname, int fd, dnprintable = dvname; break; } - if (jnext == 0) + if (next != dv) continue; } else if (strcmp(dv->devname, "detached") == 0) { if (dv->disposition != 'r' && dv->disposition != 'f') { @@ -457,7 +457,7 @@ int Manage_subdevs(char *devname, int fd, dnprintable = dvname; break; } - if (jnext == 0) + if (next != dv) continue; } else if (strcmp(dv->devname, "missing") == 0) { if (dv->disposition != 'a' || dv->re_add == 0) { -- 2.39.5