Merge "Minor clean up to install.cpp." am: 08146728c1 am: 668e84233d
am: 6650c0426f
Change-Id: Iacabb5816f811b519bde548c620929dc7b1841dd
This commit is contained in:
+6
-11
@@ -48,7 +48,6 @@
|
|||||||
|
|
||||||
#include "common.h"
|
#include "common.h"
|
||||||
#include "error_code.h"
|
#include "error_code.h"
|
||||||
#include "minui/minui.h"
|
|
||||||
#include "otautil/SysUtil.h"
|
#include "otautil/SysUtil.h"
|
||||||
#include "otautil/ThermalUtil.h"
|
#include "otautil/ThermalUtil.h"
|
||||||
#include "roots.h"
|
#include "roots.h"
|
||||||
@@ -57,15 +56,11 @@
|
|||||||
|
|
||||||
using namespace std::chrono_literals;
|
using namespace std::chrono_literals;
|
||||||
|
|
||||||
#define PUBLIC_KEYS_FILE "/res/keys"
|
|
||||||
static constexpr const char* METADATA_PATH = "META-INF/com/android/metadata";
|
static constexpr const char* METADATA_PATH = "META-INF/com/android/metadata";
|
||||||
static constexpr const char* UNCRYPT_STATUS = "/cache/recovery/uncrypt_status";
|
|
||||||
|
|
||||||
// Default allocation of progress bar segments to operations
|
// Default allocation of progress bar segments to operations
|
||||||
static constexpr int VERIFICATION_PROGRESS_TIME = 60;
|
static constexpr int VERIFICATION_PROGRESS_TIME = 60;
|
||||||
static constexpr float VERIFICATION_PROGRESS_FRACTION = 0.25;
|
static constexpr float VERIFICATION_PROGRESS_FRACTION = 0.25;
|
||||||
static constexpr float DEFAULT_FILES_PROGRESS_FRACTION = 0.4;
|
|
||||||
static constexpr float DEFAULT_IMAGE_PROGRESS_FRACTION = 0.1;
|
|
||||||
|
|
||||||
static std::condition_variable finish_log_temperature;
|
static std::condition_variable finish_log_temperature;
|
||||||
|
|
||||||
@@ -621,10 +616,8 @@ really_install_package(const char *path, bool* wipe_cache, bool needs_mount,
|
|||||||
return result;
|
return result;
|
||||||
}
|
}
|
||||||
|
|
||||||
int
|
int install_package(const char* path, bool* wipe_cache, const char* install_file, bool needs_mount,
|
||||||
install_package(const char* path, bool* wipe_cache, const char* install_file,
|
int retry_count) {
|
||||||
bool needs_mount, int retry_count)
|
|
||||||
{
|
|
||||||
modified_flash = true;
|
modified_flash = true;
|
||||||
auto start = std::chrono::system_clock::now();
|
auto start = std::chrono::system_clock::now();
|
||||||
|
|
||||||
@@ -648,6 +641,7 @@ install_package(const char* path, bool* wipe_cache, const char* install_file,
|
|||||||
bool has_cache = volume_for_path("/cache") != nullptr;
|
bool has_cache = volume_for_path("/cache") != nullptr;
|
||||||
// Skip logging the uncrypt_status on devices without /cache.
|
// Skip logging the uncrypt_status on devices without /cache.
|
||||||
if (has_cache) {
|
if (has_cache) {
|
||||||
|
static constexpr const char* UNCRYPT_STATUS = "/cache/recovery/uncrypt_status";
|
||||||
if (ensure_path_mounted(UNCRYPT_STATUS) != 0) {
|
if (ensure_path_mounted(UNCRYPT_STATUS) != 0) {
|
||||||
LOG(WARNING) << "Can't mount " << UNCRYPT_STATUS;
|
LOG(WARNING) << "Can't mount " << UNCRYPT_STATUS;
|
||||||
} else {
|
} else {
|
||||||
@@ -682,8 +676,8 @@ install_package(const char* path, bool* wipe_cache, const char* install_file,
|
|||||||
log_buffer.push_back("temperature_max: " + std::to_string(max_temperature));
|
log_buffer.push_back("temperature_max: " + std::to_string(max_temperature));
|
||||||
}
|
}
|
||||||
|
|
||||||
std::string log_content = android::base::Join(log_header, "\n") + "\n" +
|
std::string log_content =
|
||||||
android::base::Join(log_buffer, "\n") + "\n";
|
android::base::Join(log_header, "\n") + "\n" + android::base::Join(log_buffer, "\n") + "\n";
|
||||||
if (!android::base::WriteStringToFile(log_content, install_file)) {
|
if (!android::base::WriteStringToFile(log_content, install_file)) {
|
||||||
PLOG(ERROR) << "failed to write " << install_file;
|
PLOG(ERROR) << "failed to write " << install_file;
|
||||||
}
|
}
|
||||||
@@ -695,6 +689,7 @@ install_package(const char* path, bool* wipe_cache, const char* install_file,
|
|||||||
}
|
}
|
||||||
|
|
||||||
bool verify_package(const unsigned char* package_data, size_t package_size) {
|
bool verify_package(const unsigned char* package_data, size_t package_size) {
|
||||||
|
static constexpr const char* PUBLIC_KEYS_FILE = "/res/keys";
|
||||||
std::vector<Certificate> loadedKeys;
|
std::vector<Certificate> loadedKeys;
|
||||||
if (!load_keys(PUBLIC_KEYS_FILE, loadedKeys)) {
|
if (!load_keys(PUBLIC_KEYS_FILE, loadedKeys)) {
|
||||||
LOG(ERROR) << "Failed to load keys";
|
LOG(ERROR) << "Failed to load keys";
|
||||||
|
|||||||
Reference in New Issue
Block a user