On Fri, Feb 8, 2019 at 4:22 PM Max Kirillov <max@xxxxxxxxxx> wrote:
> If packed-refs is marked as sorted but not really sorted it causes
> very hard to comprehend misbehavior of reference resolving - a reference
> is reported as not found, though it is listed by commands which output
> the references list.
>
> As the scope of the issue is not clear, make it visible by failing
> pack-refs command - the one which would not suffer performance penalty
> to verify the sortedness - when it encounters not really sorted existing
> data.
>
> Signed-off-by: Max Kirillov <max@xxxxxxxxxx>
> ---
> diff --git a/refs/packed-backend.c b/refs/packed-backend.c
> @@ -1137,6 +1138,21 @@ static int write_with_updates(struct packed_ref_store *refs,
> + if (iter)
> + {
> + if (prev_ref.len && strcmp(prev_ref.buf, iter->refname) > 0)
> + {
> + [...]
> + strbuf_release(&prev_ref);
> + goto error;
> + }
> +
> + strbuf_init(&prev_ref, 0);
> + strbuf_addstr(&prev_ref, iter->refname);
> + }
The call to strbuf_init() is leaking the allocated strbuf buffer each
time through the loop. The typical way to re-use a strbuf, and the way
you should do it here, is strbuf_reset().
