tequilaOS/platform_system_core
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

am ccb8abf5: am 63177cbf: Merge "Use base::WriteFully in zip_archive."

Browse source 
* commit 'ccb8abf5de28edd4325ff201dfd471bf91d032fe':
  Use base::WriteFully in zip_archive.
This commit is contained in:
Narayan Kamath 2015-04-29 15:06:05 +00:00 committed by Android Git Automerger
commit 22313a75b3
2 changed files with 9 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
fastboot/Android.mk
View file

@ -58,7 +58,8 @@ LOCAL_STATIC_LIBRARIES := \
libsparse_host \
libutils \
liblog \
libz
libz \
libbase
ifneq ($(HOST_OS),windows)
LOCAL_STATIC_LIBRARIES += libselinux

23
libziparchive/zip_archive.cc
View file

@ -30,6 +30,7 @@
#include <memory>
#include <vector>
#include "base/file.h"
#include "base/macros.h" // TEMP_FAILURE_RETRY may or may not be in unistd
#include "base/memory.h"
#include "log/log.h"
@ -1033,24 +1034,14 @@ class FileWriter : public Writer {
return false;
}
// Keep track of the start position so we can calculate the
// total number of bytes written.
const uint8_t* const start = buf;
while (buf_size > 0) {
ssize_t bytes_written = TEMP_FAILURE_RETRY(write(fd_, buf, buf_size));
if (bytes_written == -1) {
ALOGW("Zip: unable to write " ZD " bytes to file; %s", buf_size, strerror(errno));
return false;
}
buf_size -= bytes_written;
buf += bytes_written;
const bool result = android::base::WriteFully(fd_, buf, buf_size);
if (result) {
total_bytes_written_ += buf_size;
} else {
ALOGW("Zip: unable to write " ZD " bytes to file; %s", buf_size, strerror(errno));
}
total_bytes_written_ += static_cast<size_t>(
reinterpret_cast<uintptr_t>(buf) - reinterpret_cast<uintptr_t>(start));
return true;
return result;
}
private:
FileWriter(const int fd, const size_t declared_length) :