summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTao Bao <tbao@google.com>2018-06-09 14:28:40 +0200
committerandroid-build-merger <android-build-merger@google.com>2018-06-09 14:28:40 +0200
commite2777de4b06c0b4de86c6e85a6e99204a39bcfe7 (patch)
tree0c46583c7c19ada8f4d860986b166151781a02f2
parentMerge "Merge pi-dev-plus-aosp-without-vendor into stage-aosp-master" into stage-aosp-master (diff)
parentMerge "updater: Drop the 'overlap' parameter in LoadSrcTgtVersion3()." (diff)
downloadandroid_bootable_recovery-e2777de4b06c0b4de86c6e85a6e99204a39bcfe7.tar
android_bootable_recovery-e2777de4b06c0b4de86c6e85a6e99204a39bcfe7.tar.gz
android_bootable_recovery-e2777de4b06c0b4de86c6e85a6e99204a39bcfe7.tar.bz2
android_bootable_recovery-e2777de4b06c0b4de86c6e85a6e99204a39bcfe7.tar.lz
android_bootable_recovery-e2777de4b06c0b4de86c6e85a6e99204a39bcfe7.tar.xz
android_bootable_recovery-e2777de4b06c0b4de86c6e85a6e99204a39bcfe7.tar.zst
android_bootable_recovery-e2777de4b06c0b4de86c6e85a6e99204a39bcfe7.zip
-rw-r--r--updater/blockimg.cpp22
1 files changed, 10 insertions, 12 deletions
diff --git a/updater/blockimg.cpp b/updater/blockimg.cpp
index 1646b7c17..ff1d20a78 100644
--- a/updater/blockimg.cpp
+++ b/updater/blockimg.cpp
@@ -1091,9 +1091,8 @@ static int LoadSourceBlocks(CommandParameters& params, const RangeSet& tgt, size
* If the return value is 0, source blocks have expected content and the command can be performed.
*/
static int LoadSrcTgtVersion3(CommandParameters& params, RangeSet* tgt, size_t* src_blocks,
- bool onehash, bool* overlap) {
+ bool onehash) {
CHECK(src_blocks != nullptr);
- CHECK(overlap != nullptr);
if (params.cpos >= params.tokens.size()) {
LOG(ERROR) << "missing source hash";
@@ -1135,15 +1134,16 @@ static int LoadSrcTgtVersion3(CommandParameters& params, RangeSet* tgt, size_t*
}
// Load source blocks.
- if (LoadSourceBlocks(params, *tgt, src_blocks, overlap) == -1) {
+ bool overlap = false;
+ if (LoadSourceBlocks(params, *tgt, src_blocks, &overlap) == -1) {
return -1;
}
if (VerifyBlocks(srchash, params.buffer, *src_blocks, true) == 0) {
- // If source and target blocks overlap, stash the source blocks so we can
- // resume from possible write errors. In verify mode, we can skip stashing
- // because the source blocks won't be overwritten.
- if (*overlap && params.canwrite) {
+ // If source and target blocks overlap, stash the source blocks so we can resume from possible
+ // write errors. In verify mode, we can skip stashing because the source blocks won't be
+ // overwritten.
+ if (overlap && params.canwrite) {
LOG(INFO) << "stashing " << *src_blocks << " overlapping blocks to " << srchash;
bool stash_exists = false;
@@ -1164,7 +1164,7 @@ static int LoadSrcTgtVersion3(CommandParameters& params, RangeSet* tgt, size_t*
return 0;
}
- if (*overlap && LoadStash(params, srchash, true, &params.buffer, true) == 0) {
+ if (overlap && LoadStash(params, srchash, true, &params.buffer, true) == 0) {
// Overlapping source blocks were previously stashed, command can proceed. We are recovering
// from an interrupted command, so we don't know if the stash can safely be deleted after this
// command.
@@ -1182,9 +1182,8 @@ static int LoadSrcTgtVersion3(CommandParameters& params, RangeSet* tgt, size_t*
static int PerformCommandMove(CommandParameters& params) {
size_t blocks = 0;
- bool overlap = false;
RangeSet tgt;
- int status = LoadSrcTgtVersion3(params, &tgt, &blocks, true, &overlap);
+ int status = LoadSrcTgtVersion3(params, &tgt, &blocks, true);
if (status == -1) {
LOG(ERROR) << "failed to read blocks for move";
@@ -1382,8 +1381,7 @@ static int PerformCommandDiff(CommandParameters& params) {
RangeSet tgt;
size_t blocks = 0;
- bool overlap = false;
- int status = LoadSrcTgtVersion3(params, &tgt, &blocks, false, &overlap);
+ int status = LoadSrcTgtVersion3(params, &tgt, &blocks, false);
if (status == -1) {
LOG(ERROR) << "failed to read blocks for diff";