summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTreehugger Robot <treehugger-gerrit@google.com>2019-01-24 06:35:51 +0100
committerGerrit Code Review <noreply-gerritcodereview@google.com>2019-01-24 06:35:51 +0100
commit05d48050c1944d84542b88610a33548d384546f7 (patch)
tree1bfcc933fd0638982b5071793f038caab9982b32
parentMerge "Add libprocessgroup dependency" (diff)
parentCheck that install file exists before unlink (diff)
downloadandroid_bootable_recovery-05d48050c1944d84542b88610a33548d384546f7.tar
android_bootable_recovery-05d48050c1944d84542b88610a33548d384546f7.tar.gz
android_bootable_recovery-05d48050c1944d84542b88610a33548d384546f7.tar.bz2
android_bootable_recovery-05d48050c1944d84542b88610a33548d384546f7.tar.lz
android_bootable_recovery-05d48050c1944d84542b88610a33548d384546f7.tar.xz
android_bootable_recovery-05d48050c1944d84542b88610a33548d384546f7.tar.zst
android_bootable_recovery-05d48050c1944d84542b88610a33548d384546f7.zip
-rw-r--r--recovery-persist.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/recovery-persist.cpp b/recovery-persist.cpp
index ebb42d223..e2a6699f6 100644
--- a/recovery-persist.cpp
+++ b/recovery-persist.cpp
@@ -158,7 +158,7 @@ int main(int argc, char **argv) {
// Collects and reports the non-a/b update metrics from last_install; and removes the file
// to avoid duplicate report.
report_metrics_from_last_install(LAST_INSTALL_FILE_IN_CACHE);
- if (unlink(LAST_INSTALL_FILE_IN_CACHE) == -1) {
+ if (access(LAST_INSTALL_FILE_IN_CACHE, F_OK) && unlink(LAST_INSTALL_FILE_IN_CACHE) == -1) {
PLOG(ERROR) << "Failed to unlink " << LAST_INSTALL_FILE_IN_CACHE;
}
@@ -182,7 +182,7 @@ int main(int argc, char **argv) {
// /data/misc/recovery from pmsg. Looks for the sideload history only.
if (!has_cache) {
report_metrics_from_last_install(LAST_INSTALL_FILE);
- if (unlink(LAST_INSTALL_FILE) == -1) {
+ if (access(LAST_INSTALL_FILE, F_OK) && unlink(LAST_INSTALL_FILE) == -1) {
PLOG(ERROR) << "Failed to unlink " << LAST_INSTALL_FILE;
}
}