From cc6d7b12a0714b8a3b9e52b1877a713af569a6c5 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Thu, 13 Sep 2012 00:00:04 -0400 Subject: [PATCH] umount wrapper: fix inverted logic The current logic exits when the output *is* "device is busy" when it should be exiting when the output *isn't*. BUG=chromium-os:23443 TEST=`./mod_image_for_recovery.sh` works again Change-Id: I78eef746611de1044da664f32f8ec5e3b22b21f7 Reviewed-on: https://gerrit.chromium.org/gerrit/33170 Reviewed-by: Brian Harring Commit-Ready: Mike Frysinger Tested-by: Mike Frysinger --- path-overrides/umount | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/path-overrides/umount b/path-overrides/umount index bcb17bbd11..c855bf7d8f 100755 --- a/path-overrides/umount +++ b/path-overrides/umount @@ -21,7 +21,7 @@ for x in {1..10}; do # the output. output=$(LC_ALL=C "${binary}" "$@" 2>&1) ret=$? - if [ ${ret} -eq 0 ] || [[ "${output}" == *"device is busy"* ]]; then + if [ ${ret} -eq 0 ] || [[ "${output}" != *"device is busy"* ]]; then # Nothing to do in these scenarios; either ran fine, or it failed in a non # busy fashion. break