[PATCH] ipcpd: Fix free of uninitialed variable in graph.c

  • From: Dimitri Staessens <dimitri@ouroboros.rocks>
  • To: ouroboros@xxxxxxxxxxxxx
  • Date: Mon, 17 May 2021 16:50:03 +0200

The new GCC 11.1 compiler discovered s_dist would be uninitialized
with an unknown policy, so it doesn't need to be free'd. Also removes
some unneeded includes in broadcast dt.c that I had pending.

Signed-off-by: Dimitri Staessens <dimitri@ouroboros.rocks>
---
 src/ipcpd/broadcast/dt.c      | 2 --
 src/ipcpd/unicast/pol/graph.c | 4 +---
 2 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/ipcpd/broadcast/dt.c b/src/ipcpd/broadcast/dt.c
index c3f4090..8b15008 100644
--- a/src/ipcpd/broadcast/dt.c
+++ b/src/ipcpd/broadcast/dt.c
@@ -33,7 +33,6 @@
 #define DT               "dt"
 #define OUROBOROS_PREFIX DT
 
-#include <ouroboros/endian.h>
 #include <ouroboros/dev.h>
 #include <ouroboros/errno.h>
 #include <ouroboros/fqueue.h>
@@ -45,7 +44,6 @@
 #include "common/comp.h"
 #include "common/connmgr.h"
 #include "dt.h"
-#include "ipcp.h"
 
 #include <assert.h>
 #include <stdlib.h>
diff --git a/src/ipcpd/unicast/pol/graph.c b/src/ipcpd/unicast/pol/graph.c
index 51fd0c4..6ea5c50 100644
--- a/src/ipcpd/unicast/pol/graph.c
+++ b/src/ipcpd/unicast/pol/graph.c
@@ -834,7 +834,7 @@ int graph_routing_table(struct graph *     graph,
                 break;
         default:
                 log_err("Unsupported algorithm.");
-                goto fail_algo;
+                goto fail_table;
         }
 
         pthread_mutex_unlock(&graph->lock);
@@ -843,8 +843,6 @@ int graph_routing_table(struct graph *     graph,
 
         return 0;
 
- fail_algo:
-        free(s_dist);
  fail_table:
         pthread_mutex_unlock(&graph->lock);
         return -1;
-- 
2.31.1


Other related posts:

  • » [PATCH] ipcpd: Fix free of uninitialed variable in graph.c - Dimitri Staessens