diff options
| author | Han-Wen Nienhuys <hanwen@google.com> | 2022-01-20 15:12:06 +0000 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2022-01-20 11:31:52 -0800 |
| commit | 33e922432016f614327bcf5624c084b7c02fe9c3 (patch) | |
| tree | 790704f2146d071dd15929d545da7f5c40fea2f6 /reftable/reader.c | |
| parent | b20aab501788a076b7118946afac4a460bf19d68 (diff) | |
| download | git-33e922432016f614327bcf5624c084b7c02fe9c3.tar.xz | |
reftable: all xxx_free() functions accept NULL arguments
This fixes NULL derefs in error paths. Spotted by Coverity.
Signed-off-by: Han-Wen Nienhuys <hanwen@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'reftable/reader.c')
| -rw-r--r-- | reftable/reader.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/reftable/reader.c b/reftable/reader.c index 35781593a2..272378ed1d 100644 --- a/reftable/reader.c +++ b/reftable/reader.c @@ -646,6 +646,8 @@ int reftable_new_reader(struct reftable_reader **p, void reftable_reader_free(struct reftable_reader *r) { + if (!r) + return; reader_close(r); reftable_free(r); } |
