summaryrefslogtreecommitdiffstats
path: root/updater/blockimg.cpp
diff options
context:
space:
mode:
authorTao Bao <tbao@google.com>2018-06-07 18:56:19 +0200
committerTao Bao <tbao@google.com>2018-06-07 19:43:24 +0200
commitffede3e8d9b2a6cb6c431ccad69cb341c22f7e45 (patch)
tree2aa0d8763515e8f7706c8fd8b78a034f3bd564b4 /updater/blockimg.cpp
parentMerge "updater: Replace the reference arguments with pointers." (diff)
downloadandroid_bootable_recovery-ffede3e8d9b2a6cb6c431ccad69cb341c22f7e45.tar
android_bootable_recovery-ffede3e8d9b2a6cb6c431ccad69cb341c22f7e45.tar.gz
android_bootable_recovery-ffede3e8d9b2a6cb6c431ccad69cb341c22f7e45.tar.bz2
android_bootable_recovery-ffede3e8d9b2a6cb6c431ccad69cb341c22f7e45.tar.lz
android_bootable_recovery-ffede3e8d9b2a6cb6c431ccad69cb341c22f7e45.tar.xz
android_bootable_recovery-ffede3e8d9b2a6cb6c431ccad69cb341c22f7e45.tar.zst
android_bootable_recovery-ffede3e8d9b2a6cb6c431ccad69cb341c22f7e45.zip
Diffstat (limited to 'updater/blockimg.cpp')
-rw-r--r--updater/blockimg.cpp12
1 files changed, 3 insertions, 9 deletions
diff --git a/updater/blockimg.cpp b/updater/blockimg.cpp
index f2811bccf..1646b7c17 100644
--- a/updater/blockimg.cpp
+++ b/updater/blockimg.cpp
@@ -1628,9 +1628,10 @@ static Value* PerformBlockImageUpdate(const char* name, State* state,
}
}
+ static constexpr size_t kTransferListHeaderLines = 4;
std::vector<std::string> lines = android::base::Split(transfer_list_value->data, "\n");
- if (lines.size() < 2) {
- ErrorAbort(state, kArgsParsingFailure, "too few lines in the transfer list [%zd]",
+ if (lines.size() < kTransferListHeaderLines) {
+ ErrorAbort(state, kArgsParsingFailure, "too few lines in the transfer list [%zu]",
lines.size());
return StringValue("");
}
@@ -1654,12 +1655,6 @@ static Value* PerformBlockImageUpdate(const char* name, State* state,
return StringValue("t");
}
- if (lines.size() < 4) {
- ErrorAbort(state, kArgsParsingFailure, "too few lines in the transfer list [%zu]",
- lines.size());
- return StringValue("");
- }
-
// Third line is how many stash entries are needed simultaneously.
LOG(INFO) << "maximum stash entries " << lines[2];
@@ -1698,7 +1693,6 @@ static Value* PerformBlockImageUpdate(const char* name, State* state,
int rc = -1;
- static constexpr size_t kTransferListHeaderLines = 4;
// Subsequent lines are all individual transfer commands
for (size_t i = kTransferListHeaderLines; i < lines.size(); i++) {
const std::string& line = lines[i];