diff options
| author |  Taylor Blau <me@ttaylorr.com> | 2021-02-26 11:31:02 -0500 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2021-02-26 14:42:27 -0800 |
| commit | 66f52fa26b4facbf79e3f74d8bc95f8ac8535c50 (patch) | |
| tree | eb7d524c705994d57fc3e63676155390ecf85d5f | |
| parent | t5325: check both on-disk and in-memory reverse index (diff) | |
| download | tgif-66f52fa26b4facbf79e3f74d8bc95f8ac8535c50.tar.xz | |
pack-revindex.c: don't close unopened file descriptors
When opening a reverse index, load_revindex_from_disk() jumps to the
'cleanup' label in case something goes wrong: the reverse index had the
wrong size, an unrecognized version, or similar.
It also jumps to this label when the reverse index couldn't be opened in
the first place, which will cause an error with the unguarded close()
call in the label.
Guard this call with "if (fd >= 0)" to make sure that we have a valid
file descriptor to close before attempting to close it.
Reported-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Signed-off-by: Taylor Blau <me@ttaylorr.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | pack-revindex.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/pack-revindex.c b/pack-revindex.c index 83fe4de773..4262530449 100644 --- a/pack-revindex.c +++ b/pack-revindex.c @@ -253,7 +253,8 @@ cleanup: *data_p = (const uint32_t *)data; } - close(fd); + if (fd >= 0) + close(fd); return ret; } |