Fix the exit status of umount -a, a casualty of the TRUE/FALSE normalization.

Thanks to Francois-R Boyer <boyerf@IRO.UMontreal.CA> for the report.

2 files changed, 8 insertions, 8 deletions

diff --git a/umount.c b/umount.c
index 0867118c0..40d25f90a 100644
--- a/umount.c
+++ b/umount.c
@@ -212,18 +212,18 @@ static int umount_all(int useMtab)
 		/* Never umount /proc on a umount -a */
 		if (strstr(mountpt, "proc")!= NULL)
 			continue;
-		status = do_umount(mountpt, useMtab);
-		if (status != 0) {
+		if (!do_umount(mountpt, useMtab)) {
 			/* Don't bother retrying the umount on busy devices */
 			if (errno == EBUSY) {
 				perror_msg("%s", mountpt);
+				status = FALSE;
 				continue;
 			}
-			status = do_umount(mountpt, useMtab);
-			if (status != 0) {
+			if (!do_umount(mountpt, useMtab)) {
 				printf("Couldn't umount %s on %s: %s\n",
 					   mountpt, mtab_getinfo(mountpt, MTAB_GETDEVICE),
 					   strerror(errno));
+				status = FALSE;
 			}
 		}
 	}
diff --git a/util-linux/umount.c b/util-linux/umount.c
index 0867118c0..40d25f90a 100644
--- a/util-linux/umount.c
+++ b/util-linux/umount.c
@@ -212,18 +212,18 @@ static int umount_all(int useMtab)
 		/* Never umount /proc on a umount -a */
 		if (strstr(mountpt, "proc")!= NULL)
 			continue;
-		status = do_umount(mountpt, useMtab);
-		if (status != 0) {
+		if (!do_umount(mountpt, useMtab)) {
 			/* Don't bother retrying the umount on busy devices */
 			if (errno == EBUSY) {
 				perror_msg("%s", mountpt);
+				status = FALSE;
 				continue;
 			}
-			status = do_umount(mountpt, useMtab);
-			if (status != 0) {
+			if (!do_umount(mountpt, useMtab)) {
 				printf("Couldn't umount %s on %s: %s\n",
 					   mountpt, mtab_getinfo(mountpt, MTAB_GETDEVICE),
 					   strerror(errno));
+				status = FALSE;
 			}
 		}
 	}