diff options
| author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2022-03-04 19:32:13 +0100 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2022-03-04 13:24:19 -0800 |
| commit | 51a94d8ffe57ff40e1db1d5e46a1864a5ded7f49 (patch) | |
| tree | 3d8cb22238e053b92a88f622430efa03b430d324 /builtin/index-pack.c | |
| parent | commit-graph: fix memory leak in misused string_list API (diff) | |
| download | git-51a94d8ffe57ff40e1db1d5e46a1864a5ded7f49.tar.gz git-51a94d8ffe57ff40e1db1d5e46a1864a5ded7f49.zip | |
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/index-pack.c')
0 files changed, 0 insertions, 0 deletions