commit a3e96631a4ecee8c78e42f34700b97548546dc6c
author David Riley <davidriley@chromium.org> Fri Dec 01 16:35:29 2017 -0800
committer chrome-bot <chrome-bot@chromium.org> Tue Dec 12 11:09:10 2017 -0800
tree 47d0763b45863e2f13e510beb75ccf810379c59a
parent 363c395de791b41381c5390814c59d3c0500f34a

quick-provision: Remove tmpdir if postinst fails

BUG=chromium:788473
TEST=repeat 20 autotest/files/contrib/loadtest.py $DS config.json --simultaneous 100 --total 2000
Change-Id: I3968dc3dfcb2faba7739cdc6362e203bb8b6fc0c
Reviewed-on: https://chromium-review.googlesource.com/804963
Commit-Ready: David Riley <davidriley@chromium.org>
Tested-by: David Riley <davidriley@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

quick-provision/quick-provision

diff --git a/quick-provision/quick-provision b/quick-provision/quick-provision
index e384ed9..c763830 100644
--- a/quick-provision/quick-provision
+++ b/quick-provision/quick-provision
@@ -198,9 +198,13 @@
   info "Update next kernel to try (via postinst)"
   local tmpmnt="$(mktemp -d)"
   mount -o ro "${NEXT_ROOT}" "${tmpmnt}" || die "Unable to mount ${NEXT_ROOT}."
-  "${tmpmnt}/postinst" "${NEXT_ROOT}" || die "postinst failed."
+  "${tmpmnt}/postinst" "${NEXT_ROOT}"
+  local retval="$?"
   umount "${tmpmnt}"
   rmdir "${tmpmnt}"
+  if [[ "${retval}" -ne "0" ]]; then
+    die "postinst failed."
+  fi
 }
 
 main() {