Skip to content

Commit

Permalink
Return BSPatch value (previously, was ignored).
Browse files Browse the repository at this point in the history
BUG=705036

Review-Url: https://codereview.chromium.org/2769353003
Cr-Commit-Position: refs/heads/master@{#461326}
  • Loading branch information
waffles authored and Commit bot committed Apr 1, 2017
1 parent 42820a4 commit 761ab5e
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions courgette/third_party/bsdiff/bsdiff_apply.cc
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,7 @@ BSDiffStatus MBS_ApplyPatch(const MBSPatchHeader* header,
const uint8_t* extra_start = extra_bytes->Buffer();
const uint8_t* extra_end = extra_start + extra_bytes->Remaining();
const uint8_t* extra_position = extra_start;
extra_bytes->Skip(extra_bytes->Remaining());

const uint8_t* old_position = old_start;

Expand Down Expand Up @@ -185,9 +186,7 @@ BSDiffStatus ApplyBinaryPatch(SourceStream* old_stream,
if (CalculateCrc(old_start, old_size) != header.scrc32)
return CRC_ERROR;

MBS_ApplyPatch(&header, patch_stream, old_start, old_size, new_stream);

return OK;
return MBS_ApplyPatch(&header, patch_stream, old_start, old_size, new_stream);
}

BSDiffStatus ApplyBinaryPatch(base::File old_file,
Expand Down

0 comments on commit 761ab5e

Please sign in to comment.