diff options
| author | Josh Guilfoyle <Josh.Guilfoyle@T-Mobile.com> | 2010-12-22 19:26:58 -0800 |
|---|---|---|
| committer | Josh Guilfoyle <Josh.Guilfoyle@T-Mobile.com> | 2010-12-22 22:54:20 -0800 |
| commit | 3e2db8c6e979112da99fc75b384f8d143bf27205 (patch) | |
| tree | dda1cbda552378f4869e8d984f5f7232e8f76e70 /libs/utils/AssetManager.cpp | |
| parent | 8be0335c6f394e5860a4799645ee4240a0a411dc (diff) | |
| download | frameworks_base-3e2db8c6e979112da99fc75b384f8d143bf27205.zip frameworks_base-3e2db8c6e979112da99fc75b384f8d143bf27205.tar.gz frameworks_base-3e2db8c6e979112da99fc75b384f8d143bf27205.tar.bz2 | |
Create res-cache folders world-writable for now.
This represents a security vulnerability that permits tampering. Should
be closed by centralizing write access to this directory to a special
system service.
Diffstat (limited to 'libs/utils/AssetManager.cpp')
| -rw-r--r-- | libs/utils/AssetManager.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libs/utils/AssetManager.cpp b/libs/utils/AssetManager.cpp index 0d2c8e0..94f0fcd 100644 --- a/libs/utils/AssetManager.cpp +++ b/libs/utils/AssetManager.cpp @@ -752,9 +752,9 @@ void AssetManager::loadRedirectionMappings(ResTable* rt) const struct stat statbuf; String8 basePath(data); basePath.appendPath(kThemeResCacheDir); - createDirIfNecessary(basePath.string(), S_IRUSR|S_IWUSR|S_IRGRP|S_IROTH, &statbuf); + createDirIfNecessary(basePath.string(), 0777, &statbuf); basePath.appendPath(mThemePackageName); - createDirIfNecessary(basePath.string(), S_IRUSR|S_IWUSR|S_IRGRP|S_IROTH, &statbuf); + createDirIfNecessary(basePath.string(), 0777, &statbuf); String8 themeDirLockPath(basePath); themeDirLockPath.append(".lck"); |