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

Re: [PATCH 0/2] some leak fixes on fs/ssh-signing-fix

From
Jeff King <peff@peff.net>
Date
Oct 19, 2021, 21:01 UTC
Message-ID
<YW8yP0mb7j15y6Yy@coredump.intra.peff.net>
In-Reply-To
<af0cf842-a16c-6cc3-fb61-b61422b24b17@gigacodes.de>
On Tue, Oct 19, 2021 at 10:16:41AM +0200, Fabian Stelzer wrote:
> Is coverity included in the ci/gh actions? Where would these notices
> show up?

Not currently. I've been playing with running it in an action, and may submit something to make it more official. There's some discussion here:

  https://lore.kernel.org/git/YV5dmkkuCqAY2qqG@coredump.intra.peff.net/

There's also some on-going work to make the test suite run without leak-checkers (I didn't dig them up, but you can find recent topics and commits from Ævar). But we've got a ways to go, so you'd likely find a bunch of existing leaks if you tried it.

So no, for now there was nothing obvious you could have seen that would have alerted you.

-Peff
Previous: Fabian Stelzer
Message 5 of 5 in “some leak fixes on fs/ssh-signing-fix”
  1. 0/2 some leak fixes on fs/ssh-signing-fixJeff King, Oct 18, 2021
  2. 1/2 gpg-interface: fix leak of "line" in parse_ssh_output()Jeff King, Oct 18, 2021
  3. 2/2 gpg-interface: fix leak of strbufs in get_ssh_key_fingerprint()Jeff King, Oct 18, 2021
  4. Fabian StelzerOct 19, 2021
  5. Jeff KingOct 19, 2021

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.