summaryrefslogtreecommitdiffstats
path: root/fuse_sdcard_provider.cpp
diff options
context:
space:
mode:
authorTreehugger Robot <treehugger-gerrit@google.com>2016-09-01 19:48:39 +0200
committerGerrit Code Review <noreply-gerritcodereview@google.com>2016-09-01 19:48:39 +0200
commit3202b8faf408908456816b51e2f0faf2cd3ff746 (patch)
tree4f116fad73bb81476238deb9a70313f54cd68246 /fuse_sdcard_provider.cpp
parentMerge "minadbd: rename adb_server_main to minadbd_main." (diff)
parentCheck an edge case when read(2) returns 0 (diff)
downloadandroid_bootable_recovery-3202b8faf408908456816b51e2f0faf2cd3ff746.tar
android_bootable_recovery-3202b8faf408908456816b51e2f0faf2cd3ff746.tar.gz
android_bootable_recovery-3202b8faf408908456816b51e2f0faf2cd3ff746.tar.bz2
android_bootable_recovery-3202b8faf408908456816b51e2f0faf2cd3ff746.tar.lz
android_bootable_recovery-3202b8faf408908456816b51e2f0faf2cd3ff746.tar.xz
android_bootable_recovery-3202b8faf408908456816b51e2f0faf2cd3ff746.tar.zst
android_bootable_recovery-3202b8faf408908456816b51e2f0faf2cd3ff746.zip
Diffstat (limited to 'fuse_sdcard_provider.cpp')
-rw-r--r--fuse_sdcard_provider.cpp13
1 files changed, 5 insertions, 8 deletions
diff --git a/fuse_sdcard_provider.cpp b/fuse_sdcard_provider.cpp
index df9631272..b0ecf96be 100644
--- a/fuse_sdcard_provider.cpp
+++ b/fuse_sdcard_provider.cpp
@@ -23,6 +23,8 @@
#include <unistd.h>
#include <fcntl.h>
+#include <android-base/file.h>
+
#include "fuse_sideload.h"
struct file_data {
@@ -41,14 +43,9 @@ static int read_block_file(void* cookie, uint32_t block, uint8_t* buffer, uint32
return -EIO;
}
- while (fetch_size > 0) {
- ssize_t r = TEMP_FAILURE_RETRY(read(fd->fd, buffer, fetch_size));
- if (r == -1) {
- fprintf(stderr, "read on sdcard failed: %s\n", strerror(errno));
- return -EIO;
- }
- fetch_size -= r;
- buffer += r;
+ if (!android::base::ReadFully(fd->fd, buffer, fetch_size)) {
+ fprintf(stderr, "read on sdcard failed: %s\n", strerror(errno));
+ return -EIO;
}
return 0;