[PATCH 3/9] midx: don't free midx_name early
- From
Taylor Blau <me@ttaylorr.com>
- Date
- Feb 10, 2021, 23:02 UTC
- Message-ID
- <218474158a56d3638290989cfda3db92dd2ca606.1612998106.git.me@ttaylorr.com>
- In-Reply-To
- <cover.1612998106.git.me@ttaylorr.com>
A subsequent patch will need to refer back to 'midx_name' later on in the function. In fact, this variable is already free()'d later on, so this makes the later free() no longer redundant.
Signed-off-by: Taylor Blau <me@ttaylorr.com> --- midx.c | 1 - 1 file changed, 1 deletion(-)
diff --git a/midx.c b/midx.c index 064670c0c0..34fb9de3f3 100644 --- a/midx.c +++ b/midx.c @@ -995,7 +995,6 @@ static int write_midx_internal(const char *object_dir, struct multi_pack_index * hold_lock_file_for_update(&lk, midx_name, LOCK_DIE_ON_ERROR); f = hashfd(get_lock_file_fd(&lk), get_lock_file_path(&lk)); - FREE_AND_NULL(midx_name); if (packs.m) close_midx(packs.m);
-- 2.30.0.667.g81c0cbc6fd