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>

diff --git a/reftable/reader.c b/reftable/reader.c
index 35781593a29579e8bbb70808f7ea6c72f75a921d..272378ed1d52627bd29332c3d42c6ea252c070c4 100644 (file)
--- 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);
 }

diff --git a/reftable/writer.c b/reftable/writer.c
index 3ca721e9f64c6a1d29e1efe333dff59894a928cd..9fd24fa93ce1f98528adfa5c7ecff060ea689fbe 100644 (file)
--- a/reftable/writer.c
+++ b/reftable/writer.c
@@ -150,6 +150,8 @@ void reftable_writer_set_limits(struct reftable_writer *w, uint64_t min,
 
 void reftable_writer_free(struct reftable_writer *w)
 {
+       if (!w)
+               return;
        reftable_free(w->block);
        reftable_free(w);
 }