Game of Trees Repos

Convert from st.m_time (second-precision time_t) to st.m_tim (struct timespec). To compensate for the potential case where a filesystem provides resolution in seconds only, always read the directory if no pack files are known to exist. Otherwise, there is a race condition when gotd repo_write creates a new pack and a request arrives for repo_read shortly after. Caught by a regression test for gotd on empty repositories. Test failure pointed out by Mikhail. ok op@

diff | patch | tree

2 years ago c22ed3f5d5 Thomas Adam

make 'got clone -b' work for repositories which lack a HEAD reference

ok op@

diff | patch | tree

2 years ago c94b2859b3 Thomas Adam

upate number elements stored in the delta cache when removing an element

diff | patch | tree

2 years ago b1fad72fdc Thomas Adam

add missing free() of delta buffers in several error paths

diff | patch | tree

2 years ago 345f650943 Thomas Adam

reduce delta cache size to avoid running out of memory on large pack files

As reported by Jerome Kasper, got-index-pack would run out of memory while cloning projects like LLVM or the Linux kernel. The delta cache was pushed to its maximum size limit of ~2GB, which is much higher than the default user data limit on OpenBSD. Set the delta cache size limits to values which should result in the cache limiting itself to 128MB or less.

diff | patch | tree

2 years ago b5a8f7447c Thomas Adam

add dedicated compilation flag for the delta cache, separate from object cache

diff | patch | tree

2 years ago 0699dbc20f Thomas Adam

zero-initialize all of struct got_deflate_buf in got_deflate_init()

Fixes a potential bogus free on error. got_deflate_end() relies on zb.flags to be initialized. It will try to free a bogus zb.outbuf pointer if zb.flags happens to set by uninitialized stack memory while we are calling got_deflate_end() in an error path.

diff | patch | tree

2 years ago fedfac2c38 Thomas Adam

zero-initialize all of struct got_inflate_buf in got_inflate_init()

Fixes a potential bogus free on error. got_inflate_end() relies on zb.flags to be initialized. It will try to free a bogus zb.outbuf pointer if zb.flags happens to set by uninitialized stack memory while we are calling got_inflate_end() in an error path.

diff | patch | tree

2 years ago a7e9cbc619 Thomas Adam

make gotd work as intended on an empty repository; regression test is upcoming

diff | patch | tree

2 years ago ea1f058502 Thomas Adam

fix sending of capabilities from gotsh(1) in case repository has no refs

diff | patch | tree

2 years ago e28099979b Thomas Adam

fix crash in got-send-pack when server does not announce any capabilities

diff | patch | tree

2 years ago 946e0798ea Thomas Adam

remove trailing whitespace; patch by Josiah Frentsos