tequilaOS/platform_system_vold
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge "Shell no longer globs, so glob in pushBackContents"

Browse source
This commit is contained in:
Paul Crowley 2019-03-19 19:13:56 +00:00 committed by Gerrit Code Review
commit a41b7849d0
1 changed files with 16 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
MoveStorage.cpp
View file

@ -56,27 +56,27 @@ static void notifyProgress(int progress,
} }
} }
static status_t pushBackContents(const std::string& path, std::vector<std::string>& cmd, static bool pushBackContents(const std::string& path, std::vector<std::string>& cmd,
bool addWildcard) { int searchLevels) {
DIR* dir = opendir(path.c_str()); if (searchLevels == 0) {
if (dir == NULL) { cmd.emplace_back(path);
return -1; return true;
}
auto dirp = std::unique_ptr<DIR, int (*)(DIR*)>(opendir(path.c_str()), closedir);
if (!dirp) {
PLOG(ERROR) << "Unable to open directory: " << path;
return false;
} }
bool found = false; bool found = false;
struct dirent* ent; struct dirent* ent;
while ((ent = readdir(dir)) != NULL) { while ((ent = readdir(dirp.get())) != NULL) {
if ((!strcmp(ent->d_name, ".")) || (!strcmp(ent->d_name, ".."))) { if ((!strcmp(ent->d_name, ".")) || (!strcmp(ent->d_name, ".."))) {
continue; continue;
} }
if (addWildcard) { auto subdir = path + "/" + ent->d_name;
cmd.push_back(StringPrintf("%s/%s/*", path.c_str(), ent->d_name)); found |= pushBackContents(subdir, cmd, searchLevels - 1);
} else {
cmd.push_back(StringPrintf("%s/%s", path.c_str(), ent->d_name));
}
found = true;
} }
closedir(dir); return found;
return found ? OK : -1;
} }
static status_t execRm(const std::string& path, int startProgress, int stepProgress, static status_t execRm(const std::string& path, int startProgress, int stepProgress,
@ -90,7 +90,7 @@ static status_t execRm(const std::string& path, int startProgress, int stepProgr
cmd.push_back(kRmPath); cmd.push_back(kRmPath);
cmd.push_back("-f"); /* force: remove without confirmation, no error if it doesn't exist */ cmd.push_back("-f"); /* force: remove without confirmation, no error if it doesn't exist */
cmd.push_back("-R"); /* recursive: remove directory contents */ cmd.push_back("-R"); /* recursive: remove directory contents */
if (pushBackContents(path, cmd, true) != OK) { if (!pushBackContents(path, cmd, 2)) {
LOG(WARNING) << "No contents in " << path; LOG(WARNING) << "No contents in " << path;
return OK; return OK;
} }
@ -143,7 +143,7 @@ static status_t execCp(const std::string& fromPath, const std::string& toPath, i
cmd.push_back("-R"); /* recurse into subdirectories (DEST must be a directory) */ cmd.push_back("-R"); /* recurse into subdirectories (DEST must be a directory) */
cmd.push_back("-P"); /* Do not follow symlinks [default] */ cmd.push_back("-P"); /* Do not follow symlinks [default] */
cmd.push_back("-d"); /* don't dereference symlinks */ cmd.push_back("-d"); /* don't dereference symlinks */
if (pushBackContents(fromPath, cmd, false) != OK) { if (!pushBackContents(fromPath, cmd, 1)) {
LOG(WARNING) << "No contents in " << fromPath; LOG(WARNING) << "No contents in " << fromPath;
return OK; return OK;
} }