diff options
author | Jin Qian <jinqian@google.com> | 2017-11-20 19:52:48 +0100 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2017-11-20 19:52:48 +0100 |
commit | 25e27f48db18831ba5fcfba65a67375f87944ed0 (patch) | |
tree | 7364554a491ad9e327206cccabaf718da88a6181 /roots.cpp | |
parent | Merge "Add libbrotli as a dependency when building applypatch binary" (diff) | |
parent | Merge "root: Fix an issue when volume length from fs_mgr is negative." (diff) | |
download | android_bootable_recovery-25e27f48db18831ba5fcfba65a67375f87944ed0.tar android_bootable_recovery-25e27f48db18831ba5fcfba65a67375f87944ed0.tar.gz android_bootable_recovery-25e27f48db18831ba5fcfba65a67375f87944ed0.tar.bz2 android_bootable_recovery-25e27f48db18831ba5fcfba65a67375f87944ed0.tar.lz android_bootable_recovery-25e27f48db18831ba5fcfba65a67375f87944ed0.tar.xz android_bootable_recovery-25e27f48db18831ba5fcfba65a67375f87944ed0.tar.zst android_bootable_recovery-25e27f48db18831ba5fcfba65a67375f87944ed0.zip |
Diffstat (limited to 'roots.cpp')
-rw-r--r-- | roots.cpp | 11 |
1 files changed, 7 insertions, 4 deletions
@@ -262,17 +262,20 @@ int format_volume(const char* volume, const char* directory) { } int64_t length = 0; - if (v->length != 0) { + if (v->length > 0) { length = v->length; - } else if (v->key_loc != nullptr && strcmp(v->key_loc, "footer") == 0) { + } else if (v->length < 0 || + (v->key_loc != nullptr && strcmp(v->key_loc, "footer") == 0)) { android::base::unique_fd fd(open(v->blk_device, O_RDONLY)); if (fd == -1) { PLOG(ERROR) << "format_volume: failed to open " << v->blk_device; return -1; } - length = get_file_size(fd.get(), CRYPT_FOOTER_OFFSET); + length = + get_file_size(fd.get(), v->length ? -v->length : CRYPT_FOOTER_OFFSET); if (length <= 0) { - LOG(ERROR) << "get_file_size: invalid size " << length << " for " << v->blk_device; + LOG(ERROR) << "get_file_size: invalid size " << length << " for " + << v->blk_device; return -1; } } |