From 19718b97ba13390ff5f2b7453d0297e936db5653 Mon Sep 17 00:00:00 2001 From: dimitri staessens Date: Tue, 9 May 2017 14:31:15 +0200 Subject: irmd: Fix unbinding AP and AP-Is from a name APs and AP-Is were not removed from the registry entry for that name. Reported by: loic.vervaeke@ugent.be Tested by: loic.vervaeke@ugent.be --- src/irmd/api_table.c | 2 +- src/irmd/api_table.h | 2 +- src/irmd/main.c | 16 ++++++++++++++-- 3 files changed, 16 insertions(+), 4 deletions(-) diff --git a/src/irmd/api_table.c b/src/irmd/api_table.c index 268f8231..580191da 100644 --- a/src/irmd/api_table.c +++ b/src/irmd/api_table.c @@ -142,7 +142,7 @@ int api_entry_add_name(struct api_entry * e, } void api_entry_del_name(struct api_entry * e, - char * name) + const char * name) { struct list_head * p = NULL; struct list_head * h = NULL; diff --git a/src/irmd/api_table.h b/src/irmd/api_table.h index f9c4d0aa..1bb73df7 100644 --- a/src/irmd/api_table.h +++ b/src/irmd/api_table.h @@ -67,7 +67,7 @@ int api_entry_add_name(struct api_entry * e, char * name); void api_entry_del_name(struct api_entry * e, - char * name); + const char * name); int api_table_add(struct list_head * api_table, struct api_entry * e); diff --git a/src/irmd/main.c b/src/irmd/main.c index c9c989c5..3331c754 100644 --- a/src/irmd/main.c +++ b/src/irmd/main.c @@ -630,6 +630,8 @@ static int bind_api(pid_t api, static int unbind_ap(char * ap, char * name) { + struct reg_entry * e; + if (ap == NULL) return -EINVAL; @@ -642,6 +644,10 @@ static int unbind_ap(char * ap, apn_entry_del_name(e, name); } + e = registry_get_entry(&irmd.registry, name); + if (e != NULL) + reg_entry_del_apn(e, ap); + pthread_rwlock_unlock(&irmd.reg_lock); if (name == NULL) @@ -652,9 +658,11 @@ static int unbind_ap(char * ap, return 0; } -static int unbind_api(pid_t api, - char * name) +static int unbind_api(pid_t api, + const char * name) { + struct reg_entry * e; + pthread_rwlock_wrlock(&irmd.reg_lock); if (name == NULL) @@ -664,6 +672,10 @@ static int unbind_api(pid_t api, api_entry_del_name(e, name); } + e = registry_get_entry(&irmd.registry, name); + if (e != NULL) + reg_entry_del_api(e, api); + pthread_rwlock_unlock(&irmd.reg_lock); if (name == NULL) -- cgit v1.2.3