summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTao Bao <tbao@google.com>2015-12-15 00:53:25 +0100
committerTao Bao <tbao@google.com>2015-12-15 02:13:34 +0100
commitd3cac3443096c105cf1a1985677659105902cbdd (patch)
treeebcae32374c84258018e995b10224c0f0eb6a78e
parentMerge "update_verifier: Track the API change for isSlotBootable()." (diff)
downloadandroid_bootable_recovery-d3cac3443096c105cf1a1985677659105902cbdd.tar
android_bootable_recovery-d3cac3443096c105cf1a1985677659105902cbdd.tar.gz
android_bootable_recovery-d3cac3443096c105cf1a1985677659105902cbdd.tar.bz2
android_bootable_recovery-d3cac3443096c105cf1a1985677659105902cbdd.tar.lz
android_bootable_recovery-d3cac3443096c105cf1a1985677659105902cbdd.tar.xz
android_bootable_recovery-d3cac3443096c105cf1a1985677659105902cbdd.tar.zst
android_bootable_recovery-d3cac3443096c105cf1a1985677659105902cbdd.zip
-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: