array_try_spare(): open_dev() returns -1 on error, not zero

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: NeilBrown <neilb@suse.de>

diff --git a/Incremental.c b/Incremental.c
index c2c90519ce5b37cc538c9869c2eb5d7612ef00f8..d3724a48eba3babbbe653dbca51f805b1e227225 100644 (file)
--- a/Incremental.c
+++ b/Incremental.c
@@ -884,7 +884,7 @@ static int array_try_spare(char *devname, int *dfdp, struct dev_policy *pol,
                         * to obtain minimum spare size */
                        struct supertype *st3 = dup_super(st2);
                        int mdfd = open_dev(mp->devnum);
-                       if (!mdfd) {
+                       if (mdfd < 0) {
                                free(st3);
                                goto next;
                        }