summaryrefslogtreecommitdiffstats
path: root/updater
diff options
context:
space:
mode:
authorTao Bao <tbao@google.com>2015-12-15 19:23:53 +0100
committerandroid-build-merger <android-build-merger@google.com>2015-12-15 19:23:53 +0100
commit125a6349b9b390bd531690099939aaaf8a7513a8 (patch)
treeebcae32374c84258018e995b10224c0f0eb6a78e /updater
parentMerge "update_verifier: Track the API change for isSlotBootable()." (diff)
parentMerge "updater: Use O_SYNC and fsync() for package_extract_file()." (diff)
downloadandroid_bootable_recovery-125a6349b9b390bd531690099939aaaf8a7513a8.tar
android_bootable_recovery-125a6349b9b390bd531690099939aaaf8a7513a8.tar.gz
android_bootable_recovery-125a6349b9b390bd531690099939aaaf8a7513a8.tar.bz2
android_bootable_recovery-125a6349b9b390bd531690099939aaaf8a7513a8.tar.lz
android_bootable_recovery-125a6349b9b390bd531690099939aaaf8a7513a8.tar.xz
android_bootable_recovery-125a6349b9b390bd531690099939aaaf8a7513a8.tar.zst
android_bootable_recovery-125a6349b9b390bd531690099939aaaf8a7513a8.zip
Diffstat (limited to 'updater')
-rw-r--r--updater/install.cpp19
1 files changed, 13 insertions, 6 deletions
diff --git a/updater/install.cpp b/updater/install.cpp
index e2b3db7ce..b09086964 100644
--- a/updater/install.cpp
+++ b/updater/install.cpp
@@ -555,14 +555,21 @@ Value* PackageExtractFileFn(const char* name, State* state,
}
{
- FILE* f = fopen(dest_path, "wb");
- if (f == NULL) {
- printf("%s: can't open %s for write: %s\n",
- name, dest_path, strerror(errno));
+ int fd = TEMP_FAILURE_RETRY(open(dest_path, O_WRONLY | O_CREAT | O_TRUNC | O_SYNC,
+ S_IRUSR | S_IWUSR));
+ if (fd == -1) {
+ printf("%s: can't open %s for write: %s\n", name, dest_path, strerror(errno));
goto done2;
}
- success = mzExtractZipEntryToFile(za, entry, fileno(f));
- fclose(f);
+ success = mzExtractZipEntryToFile(za, entry, fd);
+ if (fsync(fd) == -1) {
+ printf("fsync of \"%s\" failed: %s\n", dest_path, strerror(errno));
+ success = false;
+ }
+ if (close(fd) == -1) {
+ printf("close of \"%s\" failed: %s\n", dest_path, strerror(errno));
+ success = false;
+ }
}
done2: