git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: What's cooking in git.git (Sep 2024, #01; Thu, 5)

From
Patrick Steinhardt <ps@pks.im>
Date
Sep 6, 2024, 09:18 UTC
Message-ID
<ZtrIxtNJcE15BY-Z@pks.im>
In-Reply-To
<xmqq5xr9zow7.fsf@gitster.g>
On Thu, Sep 05, 2024 at 04:01:12PM -0700, Junio C Hamano wrote:
Show 28 quoted lines
> * ps/leakfixes-part-6 (2024-09-05) 22 commits
>  - builtin/repack: fix leaking keep-pack list
>  - merge-ort: fix two leaks when handling directory rename modifications
>  - match-trees: fix leaking prefixes in `shift_tree()`
>  - builtin/fmt-merge-msg: fix leaking buffers
>  - builtin/grep: fix leaking object context
>  - builtin/pack-objects: plug leaking list of keep-packs
>  - builtin/repack: fix leaking line buffer when packing promisors
>  - negotiator/skipping: fix leaking commit entries
>  - shallow: fix leaking members of `struct shallow_info`
>  - shallow: free grafts when unregistering them
>  - object: clear grafts when clearing parsed object pool
>  - gpg-interface: fix misdesigned signing key interfaces
>  - send-pack: fix leaking push cert nonce
>  - remote: fix leak in reachability check of a remote-tracking ref
>  - remote: fix leaking tracking refs
>  - builtin/submodule--helper: fix leaking refs on push-check
>  - submodule: fix leaking fetch task data
>  - upload-pack: fix leaking child process data on reachability checks
>  - builtin/push: fix leaking refspec query result
>  - send-pack: fix leaking common object IDs
>  - fetch-pack: fix memory leaks on fetch negotiation
>  - t/test-lib: allow skipping leak checks for passing tests
> 
>  More leakfixes.
> 
>  Will merge to 'next'?
>  source: <cover.1725530720.git.ps@pks.im>

There were multiple ACKs on the first version, and the second version only addressed some feedback for trivial things. So I think this one should be ready.

Show 13 quoted lines
> * sj/ref-contents-check (2024-09-03) 4 commits
>  - ref: add symlink ref content check for files backend
>  - ref: add symref content check for files backend
>  - ref: add regular ref content check for files backend
>  - ref: initialize "fsck_ref_report" with zero
> 
>  "git fsck" learned to issue warnings on "curiously formatted" ref
>  contents that have always been taken valid but something Git
>  wouldn't have written itself (e.g., missing terminating end-of-line
>  after the full object name).
> 
>  Ready?
>  source: <Ztb-mgl50cwGVO8A@ArchLinux>
I'll do another review soonish, either today or on Monday.
Patrick
Previous: Junio C Hamano
Message 2 of 2 in “What's cooking in git.git (Sep 2024, #01; Thu, 5)”
  1. Junio C HamanoSep 5, 2024
  2. Patrick SteinhardtSep 6, 2024

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.