Remove appDirRoot argument from setupAppDir.
This is no longer needed, because vold can deduce this itself now.
Bug: 146419093
Test: builds
Change-Id: Ib4f4a4109919af683722a63b305b343ef5fe972d
diff --git a/VolumeManager.h b/VolumeManager.h
index eb48736..5a2a481 100644
--- a/VolumeManager.h
+++ b/VolumeManager.h
@@ -133,20 +133,27 @@
/*
* Creates a directory 'path' for an application, automatically creating
- * directories along the given path if they don't exist yet. 'appDirRoot'
- * is the "root" directory for app-specific directories of this kind;
- * 'path' must always start with 'appDirRoot'.
+ * directories along the given path if they don't exist yet.
*
* Example:
* path = /storage/emulated/0/Android/data/com.foo/files/
- * appDirRoot = /storage/emulated/0/Android/data/
*
- * This function will set the UID of all app-specific directories below
- * 'appDirRoot' to the 'appUid' argument. In the given example, the UID
+ * This function will first match the first part of the path with the volume
+ * root of any known volumes; in this case, "/storage/emulated/0" matches
+ * with the volume root of the emulated volume for user 0.
+ *
+ * The subseqent part of the path must start with one of the well-known
+ * Android/ data directories, /Android/data, /Android/obb or
+ * /Android/media.
+ *
+ * The final part of the path is application specific. This function will
+ * create all directories, including the application-specific ones, and
+ * set the UID of all app-specific directories below the well-known data
+ * directories to the 'appUid' argument. In the given example, the UID
* of /storage/emulated/0/Android/data/com.foo and
* /storage/emulated/0/Android/data/com.foo/files would be set to 'appUid'.
*
- * The UID of the parent directories will be set according to the
+ * The UID/GID of the parent directories will be set according to the
* requirements of the underlying filesystem and are of no concern to the
* caller.
*
@@ -155,7 +162,7 @@
* and ignored, unless the path ends with "/". Also ensures that path
* belongs to a volume managed by vold.
*/
- int setupAppDir(const std::string& path, const std::string& appDirRoot, int32_t appUid);
+ int setupAppDir(const std::string& path, int32_t appUid);
int createObb(const std::string& path, const std::string& key, int32_t ownerGid,
std::string* outVolId);