From e7d7632549947fa867d6bec504442647862ad9b8 Mon Sep 17 00:00:00 2001 From: dimitri staessens Date: Thu, 16 Jun 2016 12:43:39 +0200 Subject: ipcpd: fixed locking and cleanup The ipcpds didn't unlock the flows lock when the dst_name is unbeknownst to the IRMd. --- src/ipcpd/local/main.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'src/ipcpd/local') diff --git a/src/ipcpd/local/main.c b/src/ipcpd/local/main.c index 6357487f..43ca14da 100644 --- a/src/ipcpd/local/main.c +++ b/src/ipcpd/local/main.c @@ -365,8 +365,10 @@ static int ipcp_local_flow_alloc(pid_t n_pid, } rb = shm_ap_rbuff_open(n_pid); - if (rb == NULL) + if (rb == NULL) { + rw_lock_unlock(&_ipcp->state_lock); return -1; /* -ENORBUFF */ + } rw_lock_wrlock(&_ap_instance->flows_lock); @@ -389,6 +391,7 @@ static int ipcp_local_flow_alloc(pid_t n_pid, src_ae_name); if (port_id < 0) { + rw_lock_unlock(&_ap_instance->flows_lock); rw_lock_unlock(&_ipcp->state_lock); LOG_ERR("Could not get port id from IRMd"); /* shm_ap_rbuff_close(n_pid); */ -- cgit v1.2.3