commit 1790fe66e74f05f8d349b141cb0b9bef4248f43e
author Jeff Sharkey <jsharkey@google.com> Mon Oct 12 18:38:56 2015 +0000
committer Android Git Automerger <android-git-automerger@android.com> Mon Oct 12 18:38:56 2015 +0000
tree 5ac22c4b4fb62fc4c68ebd3481006a6876bdc145
parent aeca741ce6c16653a7880055c0c48a8706f5ff0f
parent f68525931e1b9ab6938d87bb95a2fa7966f046c4

am f6852593: am cb42a40a: Merge "Clean up any/all stale partition tables." into mnc-dr-dev

* commit 'f68525931e1b9ab6938d87bb95a2fa7966f046c4':
  Clean up any/all stale partition tables.

Disk.cpp

diff --git a/Disk.cpp b/Disk.cpp
index 1e76bee..d5f3e5d 100644
--- a/Disk.cpp
+++ b/Disk.cpp
@@ -340,10 +340,24 @@
 }
 
 status_t Disk::partitionPublic() {
+    int res;
+
     // TODO: improve this code
     destroyAllVolumes();
     mJustPartitioned = true;
 
+    // First nuke any existing partition table
+    std::vector<std::string> cmd;
+    cmd.push_back(kSgdiskPath);
+    cmd.push_back("--zap-all");
+    cmd.push_back(mDevPath);
+
+    // Zap sometimes returns an error when it actually succeeded, so
+    // just log as warning and keep rolling forward.
+    if ((res = ForkExecvp(cmd)) != 0) {
+        LOG(WARNING) << "Failed to zap; status " << res;
+    }
+
     struct disk_info dinfo;
     memset(&dinfo, 0, sizeof(dinfo));