From 089c71cdb6bcb3eec696f64818377c80cde9b62f Mon Sep 17 00:00:00 2001
From: dimitri staessens <dimitri.staessens@intec.ugent.be>
Date: Wed, 8 Feb 2017 23:05:07 +0100
Subject: ipcpd: Fix boot order of normal

irm_bind has to be called after ipcp_init() since it targets the AP-I
and needs the pid to be registered with the IRMd. Also fixes missing
unbinding of the AP-I.
---
 src/ipcpd/normal/main.c | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/ipcpd/normal/main.c b/src/ipcpd/normal/main.c
index 5d8af688..2b30dba3 100644
--- a/src/ipcpd/normal/main.c
+++ b/src/ipcpd/normal/main.c
@@ -457,18 +457,20 @@ int main(int    argc,
         sigaction(SIGHUP,  &sig_act, NULL);
         sigaction(SIGPIPE, &sig_act, NULL);
 
-        if (irm_bind_api(getpid(), ipcpi.name)) {
-                log_err("Failed to bind AP name.");
+        if (ipcp_init(argc, argv, THIS_TYPE, &normal_ops) < 0) {
+                log_err("Failed to create instance.");
                 exit(EXIT_FAILURE);
         }
 
-        if (ipcp_init(argc, argv, THIS_TYPE, &normal_ops) < 0) {
-                log_err("Failed to create instance.");
+        if (irm_bind_api(getpid(), ipcpi.name)) {
+                log_err("Failed to bind AP name.");
+                ipcp_fini();
                 exit(EXIT_FAILURE);
         }
 
         if (rib_init()) {
                 log_err("Failed to initialize RIB.");
+                irm_unbind_api(getpid(), ipcpi.name);
                 ipcp_fini();
                 exit(EXIT_FAILURE);
         }
@@ -478,6 +480,7 @@ int main(int    argc,
         if (ipcp_boot() < 0) {
                 log_err("Failed to boot IPCP.");
                 rib_fini();
+                irm_unbind_api(getpid(), ipcpi.name);
                 ipcp_fini();
                 exit(EXIT_FAILURE);
         }
@@ -489,6 +492,7 @@ int main(int    argc,
                 ipcp_set_state(IPCP_NULL);
                 ipcp_shutdown();
                 rib_fini();
+                irm_unbind_api(getpid(), ipcpi.name);
                 ipcp_fini();
                 exit(EXIT_FAILURE);
         }
@@ -500,6 +504,8 @@ int main(int    argc,
 
         rib_fini();
 
+        irm_unbind_api(getpid(), ipcpi.name);
+
         ipcp_fini();
 
         exit(EXIT_SUCCESS);
-- 
cgit v1.2.3