From 19718b97ba13390ff5f2b7453d0297e936db5653 Mon Sep 17 00:00:00 2001
From: dimitri staessens <dimitri.staessens@ugent.be>
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