Commits

Commit:
5de743f8fddcaaf2912ffc92dce239aa6227d6d0
From:
Stefan Sperling <stsp@stsp.name>
Date:
fix seek to incorrect offset in the delta base when creating deltas The stretchblk() function needs to compare data located after the block which has just been matched. However, upon entry it was resetting the file pointer of the delta base to the beginning(!) of the block. The other file is correctly positioned after the block. In many cases the data won't match and stretchblk() will not stretch the matched block. But when the data did happen to match this resulted in a bogus delta, and wrong file contents when the delta was applied. Fix this by setting the delta base file pointer to end of the block. Problem reported by naddy after our server refused a pack file which was sent by 'got send'. I could reproduce the issue by running the 'gotadmin pack' command on a copy of naddy's repository. ok naddy
diff | patch | tree
Commit:
0af64e86449b8d836b04b25ece0bbc5543a75238
From:
Stefan Sperling <stsp@stsp.name>
Date:
plug a memory leak in an error path of got_deltify()
diff | patch | tree
Commit:
dd29967c8be9311a99ae3310d49789c65989498e
From:
Stefan Sperling <stsp@stsp.name>
Date:
make got_deltify() rellocate the deltas array less often
diff | patch | tree
Commit:
9a8dc2b3ec216fd01b3c33137eb92d98ddadb63e
From:
Stefan Sperling <stsp@stsp.name>
Date:
fix deltas with trailing data that is smaller than the minimum chunk size
diff | patch | tree
Commit:
740bba1c3179a597c83f7dd3a23bffb50a494bdf
From:
Stefan Sperling <stsp@stsp.name>
Date:
allow the delta base file to lose its header between deltify_init and deltify This simplifies pack file creation. A delta base could be read from a loose object, a packfile, or it might be available in a temporary file. All these cases can now be handled the same way. We may need to open, close, and re-open a given delta base multiple times while packing.
diff | patch | tree
Commit:
7550e799ee994b0b74689a6895f84d8aaec86f49
From:
Stefan Sperling <stsp@stsp.name>
Date:
check for errors from emitdelta() in got_deltify()
diff | patch | tree
Commit:
aa51f4a4acac901a4f1bf4062664644ce95d3e8c
From:
Stefan Sperling <stsp@stsp.name>
Date:
handle fseek in got_deltify() instead of in stretchblk(); simplifies the code
diff | patch | tree
Commit:
f34b169e54fc4d4960f06b804cabe1aeec70e07d
From:
Stefan Sperling <stsp@stsp.name>
Date:
Allow for skipping the base object header in got_deltify().
diff | patch | tree
Commit:
0d15f6dcf929ae42606d3ca046621aee79e45890
From:
Stefan Sperling <stsp@stsp.name>
Date:
in addblk(), only read data into buffer1 if we will compare it to buffer2 suggested by and ok naddy@
diff | patch | tree
Commit:
68bdcdc2f5d3c37d918f85368c2537a8aa7d90eb
From:
Stefan Sperling <stsp@stsp.name>
Date:
addblk() may seek in its input file; reposition the file pointer afterwards
diff | patch | tree
Commit:
a893025fd207950945eed1482170223a2d3b9ce3
From:
Stefan Sperling <stsp@stsp.name>
Date:
addblk: iterate over the correct number of entries after growing the array ok naddy
diff | patch | tree
Commit:
e89540a95a268f47ef2d1b24c41fbb72a1f0bdc9
From:
Stefan Sperling <stsp@stsp.name>
Date:
addblk: be more careful about expanding the blocks array when we outgrow it fixes + ok naddy
diff | patch | tree
Commit:
51a494da48acb57ed84501a6d10f39ed624c711e
From:
Stefan Sperling <stsp@stsp.name>
Date:
check a block's hash as well as its length before expensive comparisons suggested by + ok naddy, and Ori agrees
diff | patch | tree
Commit:
dbbf4a5f0cfb712c5970dcb79a65c5dd2e62b19a
From:
Stefan Sperling <stsp@stsp.name>
Date:
allow got_deltify_free(NULL); will be needed by 'gotadmin pack'
diff | patch | tree
Commit:
22403ab7111504d4c33872ec85a46d2eaf21a95b
From:
Christian Weisgerber <naddy@mips.inka.de>
Date:
deltify.c: explicitly include <endian.h> for be64toh()
diff | patch | tree