Skip to content
This repository was archived by the owner on May 13, 2025. It is now read-only.

Bugfixes #2

Open
jzebedee wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Bugfixes #2

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
c563cb5
Added Yuta Mori's induced sort
jzebedee Jul 22, 2014
20ff65a
Revert "Added Yuta Mori's induced sort"
jzebedee Jul 22, 2014
1c2cbc1
Fixed db/eb being allocated with an extra byte
jzebedee Jul 22, 2014
f4adb84
Fixed an erroneous throw if a patch completes without any extra data
jzebedee Jul 22, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 6 additions & 3 deletions src/bsdiff/BinaryPatchUtility.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,8 +79,8 @@ 0 32 Header

int[] I = SuffixSort(oldData);

byte[] db = new byte[newData.Length + 1];
byte[] eb = new byte[newData.Length + 1];
byte[] db = new byte[newData.Length];
byte[] eb = new byte[newData.Length];

int dblen = 0;
int eblen = 0;
Expand Down Expand Up @@ -348,7 +348,10 @@ with control block a set of triples (x,y,z) meaning "add x bytes
bytesToCopy -= actualBytesToCopy;
}

// sanity-check
if (newPosition == newSize)
return;

// sanity-check
if (newPosition + control[1] > newSize)
throw new InvalidOperationException("Corrupt patch.");

Expand Down