Commits


fix typo which caused a double-free in gotd repo_write_shutdown()


gotctl: simplify socket_path handling ok stsp@


remove umask TODO


respect umask when creating or changing files and directories This behaviour is already documented in got-worktree(5) but wasn't actually implemented. ok stsp@


man page fixes suggested by Josiah Frentsos


add gotctl(8); initially supported commands are 'info' and 'stop' This will be used by an upcoming regress test suite for gotd(8). ok tracey


fix syntax error in gotsh.1 sshd_config example


remove sendfd pledge promise from gotd repo_read process Have the parent process send one end of the pipe directly to gotsh(1), such that repo_write can run without "sendfd". Combining "sendfd" and "recvfd" in the same process is frowned upon. ok tracey


remove sendfd pledge promise from gotd repo_write process Have the parent process send one end of the pipe directly to gotsh(1), such that repo_write can run without "sendfd". Combining "sendfd" and "recvfd" in the same process is frowned upon. ok tracey


change add_delta to take an off_t instead of size_t. add_delta already calls got_delta_open casting the size_t to off_t, and all the add_delta callers pass an off_t anyway; fixes an implicit off_t truncation. ok stsp@


stop installing git-{receive,upload}-pack symlinks to gotsh in ~/bin Having those links in the user's PATH can make our send/fetch regression tests fail. We do want to talk to git-daemon during those tests, and these symlinks can get in the way of that.


convert got_pack' filesize to off_t; ok stsp@


plug a memory leak on error in got_gotconfig_read() ok op@


add missing check for errors from got_gotconfig_read() in open_worktree()


make got.conf(5) warn about remotes configured in locally-shared repositories


sort getopt() option lists and switch statements; patch by Josiah Frentsos


fix off_t type mismatch in gotd see e082ed67; ok stsp@


check size before calling mmap(2) It's only a preparatory step, as checking whether a size_t is less than SIZE_MAX is moot. In a follow-up commit, however, the `filesize' field of the struct got_pack will become off_t and these checks will kick in. This also makes consistent how we guard mmap(2) against empty files. ok and improvements stsp@


add bounds check when reading a delta offset from a packed object ok stsp@


add missing `goto done;' on error path; ok stsp@


fix spelling of "FastCGI"; patch by Josiah Frentsos; ok op@


fix off_t type mismatch off_t is a signed type and depending on the platform, it can be "long" or "long long", so cast to long long for printf().


rebase.sh: remove accidentally included absolute path to "got"


fix gotd(8) usage() string; patch by Josiah Frentsos


fix regression test failures with Git 3.30.5 / 2.38.1 or later installed The fix for CVE-2022-39253 in Git made our regression tests fail since creating submodules from local clones is no longer allowed by default. Add an override to our invocations of "git submodule add", the same which was added to Git's regression test suite as part of their fix. This CVE doesn't otherwise affect us. We do not implement submodules. Our use of them in regression tests exists only to ensure that Got does not freak out when it sees one. Precisely because automated support for nested repositories and/or working copies (as in Git Submodules, or Subversion Externals, etc.) have been an endless source of security problems in other systems. sudden test failures pointed out by naddy@