diff options
author | Treehugger Robot <treehugger-gerrit@google.com> | 2021-02-22 19:10:37 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2021-02-22 19:10:37 +0000 |
commit | 41ec4141a542e847226bb848518293d9658cb997 (patch) | |
tree | b3d7d5886ca1e332e85438e54febb7e9c02aa72c | |
parent | c4d64008ac77f150e7d5191a6da1f59699f0f7dc (diff) | |
parent | e187074136c895d0b0649c66ace7f54e0936301e (diff) | |
download | gsid-41ec4141a542e847226bb848518293d9658cb997.tar.gz |
Merge "Move external storage mkdir to ValidateInstallParams"
-rw-r--r-- | gsi_service.cpp | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/gsi_service.cpp b/gsi_service.cpp index e4052b2..002c51f 100644 --- a/gsi_service.cpp +++ b/gsi_service.cpp @@ -130,15 +130,6 @@ binder::Status GsiService::openInstall(const std::string& install_dir, int* _aid *_aidl_return = status; return binder::Status::ok(); } - if (access(install_dir_.c_str(), F_OK) != 0 && errno == ENOENT) { - if (IsExternalStoragePath(install_dir_)) { - if (mkdir(install_dir_.c_str(), 0755) != 0) { - PLOG(ERROR) << "Failed to create " << install_dir_; - *_aidl_return = IGsiService::INSTALL_ERROR_GENERIC; - return binder::Status::ok(); - } - } - } std::string message; auto dsu_slot = GetDsuSlot(install_dir_); if (!RemoveFileIfExists(GetCompleteIndication(dsu_slot), &message)) { @@ -845,6 +836,14 @@ int GsiService::ValidateInstallParams(std::string& install_dir) { install_dir = kDefaultDsuImageFolder; } + if (access(install_dir.c_str(), F_OK) != 0 && (errno == ENOENT)) { + if (android::base::StartsWith(install_dir, kDsuSDPrefix)) { + if (mkdir(install_dir.c_str(), 0755) != 0) { + PLOG(ERROR) << "Failed to create " << install_dir; + return INSTALL_ERROR_GENERIC; + } + } + } // Normalize the path and add a trailing slash. std::string origInstallDir = install_dir; if (!android::base::Realpath(origInstallDir, &install_dir)) { |