diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2022-03-04 21:32:13 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-03-05 00:24:19 +0300 |
commit | 51a94d8ffe57ff40e1db1d5e46a1864a5ded7f49 (patch) | |
tree | 3d8cb22238e053b92a88f622430efa03b430d324 /builtin/sparse-checkout.c | |
parent | 4a0479086a9bea5d31c4588b07bd45ae92a12b71 (diff) |
commit-graph: stop fill_oids_from_packs() progress on error and free()
Fix a bug in fill_oids_from_packs(), we should always stop_progress(),
but did not do so if we returned an error here. This also plugs a
memory leak in those cases by releasing the two "struct strbuf"
variables the function uses.
While I'm at it stop hardcoding "-1" here and just use the return
value of error() instead, which happens to be "-1".
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/sparse-checkout.c')
0 files changed, 0 insertions, 0 deletions