"GOT", but the "O" is a cute, smiling pufferfish. Index | Thread | Search

From:
"Todd C. Miller" <millert@openbsd.org>
Subject:
Re: reuse tree entries array while parsing trees
To:
Stefan Sperling <stsp@stsp.name>
Cc:
gameoftrees@openbsd.org
Date:
Tue, 18 Oct 2022 14:42:09 -0600

Download raw body.

Thread
On Tue, 18 Oct 2022 22:15:57 +0200, Stefan Sperling wrote:

> This patch avoids a per-tree malloc/free dance in got-read-tree
> and got-read-pack while parsing trees.
>
> Instead of throwing the parsed-tree-entries array away after
> parsing a tree, we can leave its memory allocation in place,
> and keep extending it as needed with recallocarray().

Previously the newly-allocated memory was always zeroed out.  Now
that you are re-using the old allocation do you also need to zero
it before re-use?  I can't tell whether the callers really expect
zeroed memory or if this was just to be on the safe side.

 - todd