journal: make mmap_cache_unref() a NOP when NULL is passed, like all other destructors
authorLennart Poettering <lennart@poettering.net>
Thu, 10 Dec 2015 10:25:14 +0000 (11:25 +0100)
committerLennart Poettering <lennart@poettering.net>
Thu, 10 Dec 2015 10:35:52 +0000 (11:35 +0100)
src/journal/journal-file.c
src/journal/mmap-cache.c

index f9ff954..6f09301 100644 (file)
@@ -169,8 +169,7 @@ JournalFile* journal_file_close(JournalFile *f) {
         safe_close(f->fd);
         free(f->path);
 
-        if (f->mmap)
-                mmap_cache_unref(f->mmap);
+        mmap_cache_unref(f->mmap);
 
         ordered_hashmap_free_free(f->chain_cache);
 
index 5a07ddd..eb4b092 100644 (file)
@@ -348,7 +348,10 @@ static void mmap_cache_free(MMapCache *m) {
 }
 
 MMapCache* mmap_cache_unref(MMapCache *m) {
-        assert(m);
+
+        if (!m)
+                return NULL;
+
         assert(m->n_ref > 0);
 
         m->n_ref --;