commit 4a634e3d8c0c1425736978d044e3c69c7429c75d
author Treehugger Robot <treehugger-gerrit@google.com> Fri Jun 29 16:47:11 2018 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Fri Jun 29 16:47:11 2018 +0000
tree 80858f79b87b25b655ac01a039bc6ab2f60c4515
parent 17a93d50018956573bb2fa3b5934679bea075e3f
parent e98adee732f773c281aeb1acef8a992a67d22571

Merge "Exclude /mnt/vendor from vold's ownership."

VolumeManager.cpp

diff --git a/VolumeManager.cpp b/VolumeManager.cpp
index c3c95f6..98f12e8 100644
--- a/VolumeManager.cpp
+++ b/VolumeManager.cpp
@@ -39,6 +39,7 @@
 #include <android-base/logging.h>
 #include <android-base/parseint.h>
 #include <android-base/stringprintf.h>
+#include <android-base/strings.h>
 #include <cutils/fs.h>
 #include <cutils/log.h>
 
@@ -751,9 +752,11 @@
     std::list<std::string> toUnmount;
     mntent* mentry;
     while ((mentry = getmntent(fp)) != NULL) {
-        if (strncmp(mentry->mnt_dir, "/mnt/", 5) == 0
-                || strncmp(mentry->mnt_dir, "/storage/", 9) == 0) {
-            toUnmount.push_front(std::string(mentry->mnt_dir));
+        auto test = std::string(mentry->mnt_dir);
+        if ((android::base::StartsWith(test, "/mnt/") &&
+             !android::base::StartsWith(test, "/mnt/vendor")) ||
+            android::base::StartsWith(test, "/storage/")) {
+            toUnmount.push_front(test);
         }
     }
     endmntent(fp);