From c2b5973f9ada481c7d623e388094dd58e7668211 Mon Sep 17 00:00:00 2001
From: Martin Schanzenbach <mschanzenbach@posteo.de>
Date: Sat, 15 May 2021 18:54:03 +0200
Subject: -coverity: nptr frees

---
 src/dht/dht_api.c                       |  3 ++-
 src/dht/gnunet-service-dht_clients.c    | 12 ++++++++----
 src/dht/gnunet-service-dht_hello.c      |  3 ++-
 src/dht/gnunet-service-dht_neighbours.c |  3 ++-
 4 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/src/dht/dht_api.c b/src/dht/dht_api.c
index 96399cb5a..19aa58a35 100644
--- a/src/dht/dht_api.c
+++ b/src/dht/dht_api.c
@@ -1258,7 +1258,8 @@ GNUNET_DHT_monitor_stop (struct GNUNET_DHT_MonitorHandle *mh)
   }
   GNUNET_MQ_send (handle->mq,
                   env);
-  GNUNET_free (mh->key);
+  if (NULL != mh->key)
+    GNUNET_free (mh->key);
   GNUNET_free (mh);
 }
 
diff --git a/src/dht/gnunet-service-dht_clients.c b/src/dht/gnunet-service-dht_clients.c
index cfcb25336..86b7653da 100644
--- a/src/dht/gnunet-service-dht_clients.c
+++ b/src/dht/gnunet-service-dht_clients.c
@@ -938,7 +938,8 @@ handle_dht_local_monitor_stop (void *cls,
       GNUNET_CONTAINER_DLL_remove (monitor_head,
                                    monitor_tail,
                                    r);
-      GNUNET_free (r->key);
+      if (NULL != r->key)
+        GNUNET_free (r->key);
       GNUNET_free (r);
       return;     /* Delete only ONE entry */
     }
@@ -1281,7 +1282,8 @@ GDS_CLIENTS_process_get (uint32_t options,
                       env);
     }
   }
-  GNUNET_free (cl);
+  if (NULL != cl)
+    GNUNET_free (cl);
 }
 
 
@@ -1363,7 +1365,8 @@ GDS_CLIENTS_process_get_resp (enum GNUNET_BLOCK_Type type,
                       env);
     }
   }
-  GNUNET_free (cl);
+  if (NULL != cl)
+    GNUNET_free (cl);
 }
 
 
@@ -1445,7 +1448,8 @@ GDS_CLIENTS_process_put (uint32_t options,
                       env);
     }
   }
-  GNUNET_free (cl);
+  if (NULL != cl)
+    GNUNET_free (cl);
 }
 
 
diff --git a/src/dht/gnunet-service-dht_hello.c b/src/dht/gnunet-service-dht_hello.c
index 906391334..7b22e928f 100644
--- a/src/dht/gnunet-service-dht_hello.c
+++ b/src/dht/gnunet-service-dht_hello.c
@@ -86,7 +86,8 @@ process_hello (void *cls,
                             gettext_noop ("# HELLOs obtained from peerinfo"), 1,
                             GNUNET_NO);
   hm = GNUNET_CONTAINER_multipeermap_get (peer_to_hello, peer);
-  GNUNET_free (hm);
+  if (NULL != hm)
+    GNUNET_free (hm);
   hm = GNUNET_malloc (GNUNET_HELLO_size (hello));
   GNUNET_memcpy (hm, hello, GNUNET_HELLO_size (hello));
   GNUNET_assert (GNUNET_SYSERR !=
diff --git a/src/dht/gnunet-service-dht_neighbours.c b/src/dht/gnunet-service-dht_neighbours.c
index 88b0c5d92..39b155152 100644
--- a/src/dht/gnunet-service-dht_neighbours.c
+++ b/src/dht/gnunet-service-dht_neighbours.c
@@ -1436,7 +1436,8 @@ GDS_NEIGHBOURS_handle_get (enum GNUNET_BLOCK_Type type,
   if (msize + sizeof(struct PeerGetMessage) >= GNUNET_MAX_MESSAGE_SIZE)
   {
     GNUNET_break (0);
-    GNUNET_free (reply_bf);
+    if (NULL != reply_bf)
+      GNUNET_free (reply_bf);
     GNUNET_free (targets);
     return GNUNET_NO;
   }
-- 
cgit v1.2.3