diff options
| author | Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> | 2023-06-16 00:29:26 +0000 |
|---|---|---|
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2023-06-16 00:29:26 +0000 |
| commit | f4333a2fa9beb0c9b6c47a7b9d3eef3f04657515 (patch) | |
| tree | 3c9ee91265f4ec3eaaa9c1587eba03dfdb5b8f2a | |
| parent | cc6af6925e49b0eec754ebfd72eacab2b3dbcab8 (diff) | |
| parent | 1eec465beb1c9f6c938cff0c959037f7d498c4bd (diff) | |
| download | bsdiff-f4333a2fa9beb0c9b6c47a7b9d3eef3f04657515.tar.gz | |
Merge "Exit loop earlier for bsdiff's pathological case" am: d3f0369732 am: 6fd01a93b5 am: 1eec465beb
Original change: https://android-review.googlesource.com/c/platform/external/bsdiff/+/2627658
Change-Id: Id2f674456eddf42346c44baf3203d9dcc7936bd5
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | bsdiff.cc | 2 | ||||
| -rw-r--r-- | patch_writer.cc | 5 |
2 files changed, 2 insertions, 5 deletions
@@ -119,7 +119,7 @@ int bsdiff(const uint8_t* old_buf, size_t oldsize, const uint8_t* new_buf, oldscore++; if(((len==oldscore) && (len!=0)) || - (len>oldscore+8 && len>=min_length)) break; + (len>=oldscore+8 && len>=min_length)) break; if((scan+lastoffset<oldsize) && (old_buf[scan+lastoffset] == new_buf[scan])) diff --git a/patch_writer.cc b/patch_writer.cc index b7d9b08..83bc531 100644 --- a/patch_writer.cc +++ b/patch_writer.cc @@ -13,10 +13,6 @@ #include "bsdiff/control_entry.h" #include "bsdiff/logging.h" -namespace { - - -} // namespace namespace bsdiff { @@ -43,6 +39,7 @@ bool BsdiffPatchWriter::InitializeCompressorList( LOG(ERROR) << "Patch writer expects at least one compressor."; return false; } + compressor_list->clear(); for (const auto& type : types_) { switch (type) { |