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>
Han-Wen Nienhuys committed
Jan 20, 2022 at 15:12 UTC
33e922432016f614327bcf5624c084b7c02fe9c3
2 files changed
+4
reftable/reader.c
+2
@@ -646,6 +646,8 @@ int reftable_new_reader(struct reftable_reader **p,
646
647
void reftable_reader_free(struct reftable_reader *r)
648
{
649
+ if (!r)
650
+ return;
651
reader_close(r);
652
reftable_free(r);
653
}
reftable/writer.c
+2
@@ -150,6 +150,8 @@ void reftable_writer_set_limits(struct reftable_writer *w, uint64_t min,
150
151
void reftable_writer_free(struct reftable_writer *w)
152
{
153
+ if (!w)
154
+ return;
155
reftable_free(w->block);
156
reftable_free(w);
157
}