From c85dac656cfea758ed582508d61a5d9eb6bd7bf9 Mon Sep 17 00:00:00 2001 From: dimitri staessens Date: Wed, 24 Aug 2016 22:13:10 +0200 Subject: irmd: Fix bad lock --- src/irmd/main.c | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-) (limited to 'src/irmd/main.c') diff --git a/src/irmd/main.c b/src/irmd/main.c index 8ac645df..5f793afd 100644 --- a/src/irmd/main.c +++ b/src/irmd/main.c @@ -425,7 +425,7 @@ static int enroll_ipcp(pid_t api, return -1; } - pthread_rwlock_rdlock(&irmd->reg_lock); + pthread_rwlock_wrlock(&irmd->reg_lock); entry = get_ipcp_entry_by_api(api); if (entry == NULL) { @@ -435,6 +435,13 @@ static int enroll_ipcp(pid_t api, return -1; } + if (entry->dif_name != NULL) { + pthread_rwlock_unlock(&irmd->reg_lock); + pthread_rwlock_unlock(&irmd->state_lock); + LOG_ERR("IPCP in wrong state"); + return -1; + } + entry->dif_name = strdup(dif_name); if (entry->dif_name == NULL) { pthread_rwlock_unlock(&irmd->reg_lock); @@ -443,7 +450,12 @@ static int enroll_ipcp(pid_t api, return -1; } + pthread_rwlock_unlock(&irmd->reg_lock); + pthread_rwlock_unlock(&irmd->state_lock); + if (ipcp_enroll(api, dif_name)) { + pthread_rwlock_rdlock(&irmd->state_lock); + pthread_rwlock_wrlock(&irmd->reg_lock); free(entry->dif_name); entry->dif_name = NULL; pthread_rwlock_unlock(&irmd->reg_lock); @@ -452,11 +464,8 @@ static int enroll_ipcp(pid_t api, return -1; } - pthread_rwlock_unlock(&irmd->reg_lock); - pthread_rwlock_unlock(&irmd->state_lock); - LOG_INFO("Enrolled IPCP %d in DIF %s.", - entry->api, dif_name); + api, dif_name); return 0; } -- cgit v1.2.3