I don't think any of these is a triggerable bug. They're just cleanups to make it more obvious that the code is doing the right thing (and making it harder to do the wrong thing). [1/4]: fast-import: use xsnprintf for writing sha1s [2/4]: fast-import: use xsnprintf for formatting headers [3/4]: encode_in_pack_object_header: respect output buffer length [4/4]: pack.h: define largest possible encoded object size builtin/pack-objects.c | 12 ++++++++---- bulk-checkin.c | 2 +- fast-import.c | 16 +++++++--------- pack-write.c | 5 ++++- pack.h | 9 ++++++++- 5 files changed, 28 insertions(+), 16 deletions(-) -Peff