build: fix check_sec.sh warnings

check_sec.sh checks for calls to malloc(), free(), etc.

diff --git a/hcache-bdb.c b/hcache-bdb.c
index 3c291386c6e27b80715a22e2d551eb6e21a3e128..3c2b41454c2df08aa4eb04c62cb54b9ad55e424e 100644 (file)
--- a/hcache-bdb.c
+++ b/hcache-bdb.c
@@ -143,7 +143,7 @@ hcache_bdb_fetch(void *vctx, const char *key, size_t keylen)
 static void
 hcache_bdb_free(void *vctx, void **data)
 {
-    FREE(data);
+    FREE(data); /* __FREE_CHECKED__ */
 }
 
 static int

diff --git a/hcache-gdbm.c b/hcache-gdbm.c
index 6aa055c2b66fa0e8d7d366d8f642b744434b6f81..da27018bf73f41e685e22be207e9090ca8021071 100644 (file)
--- a/hcache-gdbm.c
+++ b/hcache-gdbm.c
@@ -63,7 +63,7 @@ hcache_gdbm_fetch(void *ctx, const char *key, size_t keylen)
 static void
 hcache_gdbm_free(void *vctx, void **data)
 {
-    FREE(data);
+    FREE(data); /* __FREE_CHECKED__ */
 }
 
 static int

diff --git a/hcache-kc.c b/hcache-kc.c
index ff07631867aa1775f1c58d16325e2300a45833ed..46a0968d0198fa59049cb61bc2ea4ca6b97704a5 100644 (file)
--- a/hcache-kc.c
+++ b/hcache-kc.c
@@ -73,7 +73,7 @@ hcache_kyotocabinet_fetch(void *ctx, const char *key, size_t keylen)
 static void
 hcache_kyotocabinet_free(void *vctx, void **data)
 {
-    FREE(data);
+    FREE(data); /* __FREE_CHECKED__ */
 }
 
 static int

diff --git a/hcache-qdbm.c b/hcache-qdbm.c
index 3de9cf4ab130b4ab6b05d0155588e38a393686e1..798d46156392954a12ceff0d17d89d7efb37d84e 100644 (file)
--- a/hcache-qdbm.c
+++ b/hcache-qdbm.c
@@ -53,7 +53,7 @@ hcache_qdbm_fetch(void *ctx, const char *key, size_t keylen)
 static void
 hcache_qdbm_free(void *ctx, void **data)
 {
-    FREE(data);
+    FREE(data); /* __FREE_CHECKED__ */
 }
 
 static int

diff --git a/hcache-tc.c b/hcache-tc.c
index 85b5ba8fc22347158b9d950236b49e9912d4c6ca..2824a916b953e134db2f4cea61a6eed670e2cbe9 100644 (file)
--- a/hcache-tc.c
+++ b/hcache-tc.c
@@ -63,7 +63,7 @@ hcache_tokyocabinet_fetch(void *ctx, const char *key, size_t keylen)
 static void
 hcache_tokyocabinet_free(void *ctx, void **data)
 {
-    FREE(data);
+    FREE(data); /* __FREE_CHECKED__ */
 }
 
 static int

diff --git a/hcache.c b/hcache.c
index 7627420ce7289ca086a0b8c92a5dc7b180c29088..85dde3a6895a1a3b1921f182519b7eea0c3bbf92 100644 (file)
--- a/hcache.c
+++ b/hcache.c
@@ -865,7 +865,7 @@ mutt_hcache_free(header_cache_t *h, void **data)
   if (!h || !ops)
     return;
 
-  ops->free(h->ctx, data);
+  ops->free(h->ctx, data); /* __MEM_CHECKED__ */
 }
 
 int
@@ -932,7 +932,7 @@ mutt_hcache_backend_list()
     len += snprintf(tmp+len, STRING-len, "%s", (*ops)->name);
   }
 
-  return strdup(tmp);
+  return safe_strdup(tmp);
 }
 
 int

diff --git a/hdrline.c b/hdrline.c
index 04159f2f51efa7cfb69cde082d82be160a9c6dc7..1997da2a4bca11e311562503befc11e8a4f4a3d5 100644 (file)
--- a/hdrline.c
+++ b/hdrline.c
@@ -189,7 +189,7 @@ enum FieldType
 /**
  * make_from_prefix - Create a prefix for an author field
  * @disp:   Type of field
- * @return: Prefix string (do not free() it)
+ * @return: Prefix string (do not free it)
  *
  * If $from_chars is set, pick an appropriate character from it.
  * If not, use the default prefix: "To", "Cc", etc

diff --git a/init.c b/init.c
index 27fff8798789116beb72930c546528d9a1ebe846..7058f5fbf655833c47b6993b839ed6668c51e487 100644 (file)
--- a/init.c
+++ b/init.c
@@ -2365,7 +2365,7 @@ static int parse_set (BUFFER *tmp, BUFFER *s, unsigned long data, BUFFER *err)
       mutt_extract_token (tmp, s, 0);
       if (mutt_hcache_is_valid_backend(tmp->data))
       {
-          FREE ((void *)MuttVars[idx].data);
+          FREE ((void *)MuttVars[idx].data); /* __FREE_CHECKED__ */
           *(char **)(MuttVars[idx].data) = safe_strdup(tmp->data);
       }
       else

diff --git a/strndup.c b/strndup.c
index f130730aa74e83d8d799a9132f250ac35577f69c..7e395389e05479517ed47c3c5d283dd2bf01c97b 100644 (file)
--- a/strndup.c
+++ b/strndup.c
@@ -11,7 +11,7 @@
 char *strndup(const char *s, size_t n)
 {
        size_t len = strnlen(s, n);
-       char *new = malloc((len + 1) * sizeof(char));
+       char *new = malloc((len + 1) * sizeof(char)); /* __MEM_CHECKED__ */
        if (!new)
                return NULL;
        new[len] = '\0';