Commits
- Commit:
43be1edb0aebe761af9941921b0d17db40d98979
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
gotwebd: fix crash handling -D
Avoid crashing by corrupting the pointer instead of writing to it and
fix how sym is generated: it shouldn't include the '=' sign.
ok / improvement tb@
- Commit:
41861f2b47879010d252aa5a6d8a7f8f734041f0
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
avoid a pointless malloc/free in got_reflist_insert() in my previous fix
suggested by op@
- Commit:
21de8138fee88f812099d8b0b71d8280f4f6ba4e
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
detect packed vs. on-disk reference collision when listing refs
ok op@
- Commit:
d68b973725ce0a2c7b1f5094448a444d4f2effdc
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
plug leak in the commit graph iterator
We fail to release the memory for the nodes. To fix it however, we some
consumer of the commit graph iterator need to be corrected: the returned
pointer is safe to be used only up until the next iter_next call;
save a copy it if it's needed afterwards too.
ok stsp@
- Commit:
95326260eb5a3786dbd04e735b6c8c9313523ff2
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
gotwebd: fix usage of the commit graph iter
got_get_repo_commits mixes ids allocated on the heap and the one
returned by the iter_next in the same variable, and then even attemp to
free it.
This is both a leak (we loose the pointer to the previously allocated
id) and a possible invalid free since the object id pointer returned by
the graph iterator is not to be passed to free(3).
part of a bigger diff that's ok stsp@
- Commit:
7ee8c11a1c0b35faced7b53a32b5a43b0f45b764
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
add missing freaddrinfo
also spotted by valgrind. ok stsp@
- Commit:
e476ee5e274eada26700de434b1500a7d8f71b97
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
release the memory used by the object cache
Leak spotted by valgrind. ok stsp@
- Commit:
99c6f580a3ccd12811e726f6773d468eb0952427
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
replace an indirect call to idset_add(), via add_node(), with a direct call
Avoids leaking the node structure returned from add_node().
ok op@
- Commit:
e4ca14132ba73f03dbfbdc465d31619781b465f3
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
got: fflush(stdout) after asking questions
Not everywhere getline does an implicit fflush, so we might end up not
showing the prompt to the user. Spotted on alpine with muslc.
ok Thomas Adams
- Commit:
07894e99b111edc91d20565e0982344b52b32095
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
free commit on error path
while here drop the ternary too: if add_node succeeds `new_node' is
non-NULL. ok stsp@
- Commit:
f3933be66b2ab372f0f0b48b3dec7bc6d21b170a
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
plug leak: release nodes not added to the iter list
ok stsp@
- Commit:
599785e85d854563801d9640012f93e799ff87ea
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
got_commit_graph_close: free tailq contents
The tailq should ideally be empty when we reach got_commit_graph_close,
but consumer code may exit the commit graph loop earlier.
spotted by and ok stsp@
- Commit:
8dbcf3885ae920e0fd38a455e7577824989a605f
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
forbid rebase of references outside the refs/heads/ namespace
ok jrick
- Commit:
35eabca99227d5245619972d35e5229c63c248db
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
plug leak in resolve_symlink
ok stsp@
- Commit:
2ac684a4e1b8a971689edc8c5543832dd1b9db0e
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
gotwebd: percent-decode the received query string
ok stsp@
- Commit:
a6fd182ad357eeabb88cffe83d799b412e4cdab7
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
plug mem leak in got_repo_pack_fds_open
pack_fds_tmp is not always free'd. However, while here, simplify it to
not use a temporary array.
ok stsp@
- Commit:
aefd24c9f8aab7588c262b45581331004a959717
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
gotwebd: free obj_id and reflist in got_output_file_blame
ok stsp@
- Commit:
d38635a394cab631d6a100b34ae0bd6f9b651bba
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
gotwebd: got_output_file_blame: free lines on error path
ok stsp@
- Commit:
d3a6241d268d5433724e871be827d2327de510a8
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
gotwebd: free eline in got_gotweb_blame_cb loop
ok stsp@
- Commit:
d5a6a0f390343f68393d08e55305180b9d81e68b
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
gotwebd: drop superfluous check
commit_id is not NULL if got_repo_match_object_id returned
successfully. ok stsp@
- Commit:
7a800a02c49f1b97dfd6344a10df6e741894de56
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
sync file with diff.git 279d77aa1e4b921090e702a6a8a5a94a98a062b1 and
use the faster headers-only mode for diff3
- Commit:
43896ae629e895c4f9fda63d34496408db649225
- From:
- Stefan Hagen <sh+got@codevoid.de>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
got: don't search through ignored paths and files on commit
- Commit:
063253307314ac1b90e8cd14b32cd11f93d31177
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
gotwebd: kill unused id_str in got_output_repo_tree
ok stsp@
- Commit:
ccff619c1232894de361a4f436f9695a57d7f401
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
gotwebd: avoid extra strdups in got_output_repo_tree
ok stsp@
- Commit:
557d32ee82573eead5f6556008c70a5997bf5d8a
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
gotwebd: avoid extra variable in got_output_repo_tree
in_repo_path was used only to build a path, it's then assigned to path
(which is NULL at that point.) Just asprintf into path.
ok stsp@