Thread (2 messages) 2 messages, 2 authors, 2025-09-23

Re: [PATCH 1/6] stash: tell setup_revisions() to free our allocated strings

From: Jeff King <hidden>
Date: 2025-09-23 00:48:11

On Mon, Sep 22, 2025 at 02:26:44PM -0700, Junio C Hamano wrote:
By the way this also solves the original "git stash show -p --invalid";
do we want to credit the bug reporter and mention that issue as well?
That's not handled by this patch, but rather by patch 2 ("manage memory
ownership..."). I put this one first because until we start setting the
free_removed_argv_elements flag, that second patch won't do anything for
git-stash (and I don't think there was a way to trigger the problems
from the other callers which did set it).

That patch discusses the original issue, but yeah, it could probably
stand to have a:

  Reported-by: Lauri Niskanen [off-list ref]

trailer tacked on.

-Peff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help