Re: [PATCH 1/7] trace2: fix memory leak of thread name

From

Taylor Blau <me@ttaylorr.com>

Date

Sep 16, 2021, 05:43 UTC

Message-ID

<YULZbQgxuyw8iJ/R@nand.local>

In-Reply-To

<87fsu5m649.fsf@evledraar.gmail.com>

On Thu, Sep 16, 2021 at 07:35:59AM +0200, Ævar Arnfjörð Bjarmason wrote:

> So I think this patch can be dropped from this series, since it's exact
> duplicate of my 48f68715b14 (tr2: stop leaking "thread_name" memory,
> 2021-08-27) in ab/tr2-leaks-and-fixes, currently in "next" and marked
> for a merge with master.

I agree it can be dropped.

> When submitting a series that depends on another one it's best to rebase
> it on top of it & indicate it as such in the cover letter, Junio can
> queue such a series on top of another one.
>
> In this case I'm still not sure why this fix is here, i.e. surely
> nothing later in the series absolutely needs this stray memory leak
> fix...