Commits


CHANGES for 0.96


never write accum_buf on error in got_pack_dump_delta_chain_to_file() and also avoid to override `err' if fwrite fails. ok millert, stsp


fix use of uninitialized variable in update_blob spotted by smatch. ok millert@


plug memory leak in got_pack_dump_delta_chain_to_file() found by smatch. ok stsp@


fmt


pass the argument to calloc in the right order; no-op in practice reported by smatch.


fix logging during gotwebd shutdown During `rcctl stop gotwebd' the sockets process gets a SIGTERM and dies, and this makes the parent process assume it exited abnormally. Instead, catch SIGINT and SIGTERM and exit gracefully. Issue reported by xs on IRC. ok jamsek


gotwebd: fix colour of target lines in dark mode reported by xs on IRC, thanks!


Test log -b <path> traversing a merge commit. This tests for the bug fixed by c8255edc. ok stsp@


handle merge commits that are unrelated to requested changed path history Problem found and fix tested by James Cook.


plug some fd leaks in the fdopen{,dir} error paths There's also a memory leak fixed. ok stsp@


style: no-op change in practice change so it matches the style used in the rest of the tree. ok stsp


remove stray abort() call; ok stsp


remove unneded wbuf->fd = -1 There's no need to set the fd to -1 on ibufs created with imsg_create(3), and it was probably never needed.


convert to use imsg_get_fd() While here also fix a fd leak in got-read-pack. We were dup'ing imsg.fd without closing imsg.fd later; instead just use imsg_get_fd() to extract the file descriptor. Tested by falsifian and Kyle Ackerman, thanks! 'go ahead' stsp@