diff options
| author | 2019-11-16 05:54:04 -0800 | |
|---|---|---|
| committer | 2019-11-16 05:54:04 -0800 | |
| commit | e9975827e8be35c0fcc3a200ef8b1d336d6d201e (patch) | |
| tree | 74b1a3605cce77d34a18dd88a6578a999351d2cf | |
| parent | e6d829d9688dabb230329d1c9b291906a745ad55 (diff) | |
| parent | 000b73f1c6e5bdd977b2416cb5cc35dfe704aa0d (diff) | |
Merge "[installd] Improve profile merging error logging" am: 781e508fcd am: a8fdc6c6ec
am: 000b73f1c6
Change-Id: I0c2cb331fdb0f70f593907f65a636b19c66d09d0
| -rw-r--r-- | cmds/installd/dexopt.cpp | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/cmds/installd/dexopt.cpp b/cmds/installd/dexopt.cpp index 768d900591..616c3b2687 100644 --- a/cmds/installd/dexopt.cpp +++ b/cmds/installd/dexopt.cpp @@ -708,12 +708,13 @@ static void open_profile_files(uid_t uid, const std::string& package_name, } } -static constexpr int PROFMAN_BIN_RETURN_CODE_COMPILE = 0; -static constexpr int PROFMAN_BIN_RETURN_CODE_SKIP_COMPILATION = 1; -static constexpr int PROFMAN_BIN_RETURN_CODE_BAD_PROFILES = 2; -static constexpr int PROFMAN_BIN_RETURN_CODE_ERROR_IO = 3; -static constexpr int PROFMAN_BIN_RETURN_CODE_ERROR_LOCKING = 4; -static constexpr int PROFMAN_BIN_RETURN_CODE_SUCCESS = 5; +static constexpr int PROFMAN_BIN_RETURN_CODE_SUCCESS = 0; +static constexpr int PROFMAN_BIN_RETURN_CODE_COMPILE = 1; +static constexpr int PROFMAN_BIN_RETURN_CODE_SKIP_COMPILATION = 2; +static constexpr int PROFMAN_BIN_RETURN_CODE_BAD_PROFILES = 3; +static constexpr int PROFMAN_BIN_RETURN_CODE_ERROR_IO = 4; +static constexpr int PROFMAN_BIN_RETURN_CODE_ERROR_LOCKING = 5; +static constexpr int PROFMAN_BIN_RETURN_CODE_ERROR_DIFFERENT_VERSIONS = 6; class RunProfman : public ExecVHelper { public: @@ -889,6 +890,11 @@ static bool analyze_profiles(uid_t uid, const std::string& package_name, should_clear_current_profiles = false; should_clear_reference_profile = false; break; + case PROFMAN_BIN_RETURN_CODE_ERROR_DIFFERENT_VERSIONS: + need_to_compile = false; + should_clear_current_profiles = true; + should_clear_reference_profile = true; + break; default: // Unknown return code or error. Unlink profiles. LOG(WARNING) << "Unexpected error code while processing profiles for location " |