diff options
author | NeilBrown <neilb@suse.de> | 2015-07-02 08:16:59 +1000 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2015-07-02 08:16:59 +1000 |
commit | 52b6ccad34b1a6c974198f4b90b7ca15089b546d (patch) | |
tree | d6cd47f76c2d5709bac0c7a36f6bd1909e070eb0 | |
parent | 9581efb1ae9348ef33d10d781cedde375c0d62c4 (diff) | |
download | mdadm-52b6ccad34b1a6c974198f4b90b7ca15089b546d.tar.gz |
Manage: fix no-op test in Manage_stop.
A 'devnm' never starts with '/', so this test is pointless.
The code should use the passed-in devname unless it is clearly
not usable. So fix it to do that.
Signed-off-by: NeilBrown <neilb@suse.de>
-rw-r--r-- | Manage.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -235,7 +235,7 @@ int Manage_stop(char *devname, int fd, int verbose, int will_retry) container[0] = 0; close(fd); count = 5; - while (((fd = ((devnm[0] == '/') + while (((fd = ((devname[0] == '/') ?open(devname, O_RDONLY|O_EXCL) :open_dev_flags(devnm, O_RDONLY|O_EXCL))) < 0 || strcmp(fd2devnm(fd), devnm) != 0) |