[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[gnunet] 02/02: -coverity: use after free
From: |
gnunet |
Subject: |
[gnunet] 02/02: -coverity: use after free |
Date: |
Sat, 15 May 2021 18:49:01 +0200 |
This is an automated email from the git hooks/post-receive script.
martin-schanzenbach pushed a commit to branch master
in repository gnunet.
commit 168e501991b9506b588e1180527dad9c9662ae0c
Author: Martin Schanzenbach <mschanzenbach@posteo.de>
AuthorDate: Sat May 15 18:46:35 2021 +0200
-coverity: use after free
---
src/datacache/plugin_datacache_sqlite.c | 3 ++-
src/datastore/datastore_api.c | 2 +-
src/datastore/gnunet-service-datastore.c | 6 ++++--
src/datastore/plugin_datastore_sqlite.c | 3 ++-
4 files changed, 9 insertions(+), 5 deletions(-)
diff --git a/src/datacache/plugin_datacache_sqlite.c
b/src/datacache/plugin_datacache_sqlite.c
index a7da6b068..bade15479 100644
--- a/src/datacache/plugin_datacache_sqlite.c
+++ b/src/datacache/plugin_datacache_sqlite.c
@@ -764,7 +764,8 @@ libgnunet_plugin_datacache_sqlite_done (void *cls)
#if ! WINDOWS || defined(__CYGWIN__)
if ((NULL != plugin->fn) && (0 != unlink (plugin->fn)))
LOG_STRERROR_FILE (GNUNET_ERROR_TYPE_WARNING, "unlink", plugin->fn);
- GNUNET_free (plugin->fn);
+ if (NULL != plugin->fn)
+ GNUNET_free (plugin->fn);
#endif
sqlite3_finalize (plugin->insert_stmt);
sqlite3_finalize (plugin->get_count_stmt);
diff --git a/src/datastore/datastore_api.c b/src/datastore/datastore_api.c
index d658b9c85..cb21c9308 100644
--- a/src/datastore/datastore_api.c
+++ b/src/datastore/datastore_api.c
@@ -1430,7 +1430,7 @@ GNUNET_DATASTORE_get_key (struct GNUNET_DATASTORE_Handle
*h,
{
LOG (GNUNET_ERROR_TYPE_DEBUG,
"Could not queue request for `%s'\n",
- GNUNET_h2s (key));
+ (NULL == key) ? "NULL" : GNUNET_h2s (key));
return NULL;
}
#if INSANE_STATISTICS
diff --git a/src/datastore/gnunet-service-datastore.c
b/src/datastore/gnunet-service-datastore.c
index 97888ce03..7d07ba3b3 100644
--- a/src/datastore/gnunet-service-datastore.c
+++ b/src/datastore/gnunet-service-datastore.c
@@ -1556,7 +1556,8 @@ run (void *cls,
5); /* approx. 3% false
positives at max use */
refresh_bf = GNUNET_YES;
}
- GNUNET_free (pfn);
+ if (NULL != pfn)
+ GNUNET_free (pfn);
}
else
{
@@ -1566,7 +1567,8 @@ run (void *cls,
5); /* approx. 3% false positives at
max use */
refresh_bf = GNUNET_YES;
}
- GNUNET_free (fn);
+ if (NULL != fn)
+ GNUNET_free (fn);
if (NULL == filter)
{
GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
diff --git a/src/datastore/plugin_datastore_sqlite.c
b/src/datastore/plugin_datastore_sqlite.c
index f58955b7b..40f955312 100644
--- a/src/datastore/plugin_datastore_sqlite.c
+++ b/src/datastore/plugin_datastore_sqlite.c
@@ -729,7 +729,8 @@ sqlite_plugin_put (void *cls,
}
GNUNET_SQ_reset (plugin->dbh, stmt);
cont (cont_cls, key, size, ret, msg);
- GNUNET_free (msg);
+ if (NULL != msg)
+ GNUNET_free (msg);
}
--
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.