[PATCH] irmd: Don't destroy shm_flow_set from IRMd

  • From: Dimitri Staessens <dimitri@ouroboros.rocks>
  • To: ouroboros@xxxxxxxxxxxxx
  • Date: Tue, 12 Mar 2019 00:44:25 +0100

The shm_flow_set for the processes are now created and destroyed
together with the irm_flow objects. This old code causes stack
overflows in some (rare) conditions and must be removed.

Signed-off-by: Dimitri Staessens <dimitri@ouroboros.rocks>
---
 src/irmd/main.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/irmd/main.c b/src/irmd/main.c
index 5f9771a..eb9526b 100644
--- a/src/irmd/main.c
+++ b/src/irmd/main.c
@@ -1775,13 +1775,9 @@ void * irm_sanitize(void * o)
                         }
 
                         if (kill(f->n_pid, 0) < 0) {
-                                struct shm_flow_set * set;
                                 log_dbg("Process %d gone, deallocating "
                                         "flow %d.",
                                          f->n_pid, f->flow_id);
-                                set = shm_flow_set_open(f->n_pid);
-                                if (set != NULL)
-                                        shm_flow_set_destroy(set);
                                 f->n_pid = -1;
                                 irm_flow_set_state(f, FLOW_DEALLOC_PENDING);
                                 ipcpi   = f->n_1_pid;
-- 
2.21.0


Other related posts: