Nico Golde:

- don't allocate space for not used variable
- FREE does already do this sanity check ( both provided by Thomans Glanzmann)

git-svn-id: svn://svn.berlios.de/mutt-ng/trunk@298 e385b8ad-14ed-0310-8656-cc95a2468c6d

diff --git a/globals.h b/globals.h
index f938fe1..b34f747 100644 (file)
--- a/globals.h
+++ b/globals.h
@@ -70,8 +70,10 @@ WHERE char *Maildir;
 
 #if USE_HCACHE
 WHERE char *HeaderCache;
+#if HAVE_GDBM || HAVE_DB4
 WHERE char *HeaderCachePageSize;
-#endif
+#endif /* HAVE_GDBM || HAVE_DB4 */
+#endif /* USE_HCACHE */
 WHERE char *MhFlagged;
 WHERE char *MhReplied;
 WHERE char *MhUnseen;

diff --git a/hcache.c b/hcache.c
index 6db60e9..f6bcbf8 100644 (file)
--- a/hcache.c
+++ b/hcache.c
@@ -668,9 +668,7 @@ mutt_hcache_fetch(void *db, const char *filename,
 
   if (! crc32_matches(data, h->crc))
   {
-    if (data) { 
-      FREE(&data);
-    }
+    FREE(&data);
     return NULL;
   }