commit fed8ab1d7641a0c410bda6b2dd3fad64edd85f40
author Tom Cherry <tomcherry@google.com> Wed May 03 23:42:58 2017 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Wed May 03 23:43:00 2017 +0000
tree 5a1a3eb5063868e6550c97f55b63b78fcd2c3010
parent 399bd0866c23efa95b10af28b6301f34bb32c806
parent eb50114a83818554470396845e6532bc7dc778fb

Merge "BuildDataUserCePath always use dir instead of symbolic link"

Utils.cpp

diff --git a/Utils.cpp b/Utils.cpp
index cc30d5d..ecc56e4 100644
--- a/Utils.cpp
+++ b/Utils.cpp
@@ -608,15 +608,15 @@
 std::string BuildDataUserCePath(const char* volumeUuid, userid_t userId) {
     // TODO: unify with installd path generation logic
     std::string data(BuildDataPath(volumeUuid));
-    if (volumeUuid == nullptr) {
-        if (userId == 0) {
-            return StringPrintf("%s/data", data.c_str());
-        } else {
-            return StringPrintf("%s/user/%u", data.c_str(), userId);
+    if (volumeUuid == nullptr && userId == 0) {
+        std::string legacy = StringPrintf("%s/data", data.c_str());
+        struct stat sb;
+        if (lstat(legacy.c_str(), &sb) == 0 && S_ISDIR(sb.st_mode)) {
+            /* /data/data is dir, return /data/data for legacy system */
+            return legacy;
         }
-    } else {
-        return StringPrintf("%s/user/%u", data.c_str(), userId);
     }
+    return StringPrintf("%s/user/%u", data.c_str(), userId);
 }
 
 std::string BuildDataUserDePath(const char* volumeUuid, userid_t userId) {