From 19a3277dbff7fc79c12be8a200ab6c8dfe6b50b9 Mon Sep 17 00:00:00 2001
From: dimitri staessens <dimitri.staessens@ugent.be>
Date: Tue, 21 Mar 2017 16:14:18 +0100
Subject: irmd: Fix timeouts in reg_entry

This fixes bad timedwaits for the state of the reg_entry. Also
slightly revised timedwaits throughout the prototype.
---
 src/irmd/api_table.c | 23 +++++++++++++++++++++--
 src/irmd/main.c      |  2 +-
 src/irmd/registry.c  |  6 +++---
 3 files changed, 25 insertions(+), 6 deletions(-)

(limited to 'src/irmd')

diff --git a/src/irmd/api_table.c b/src/irmd/api_table.c
index df300cea..83153aac 100644
--- a/src/irmd/api_table.c
+++ b/src/irmd/api_table.c
@@ -34,6 +34,7 @@
 struct api_entry * api_entry_create(pid_t api, char * apn)
 {
         struct api_entry * e;
+        pthread_condattr_t cattr;
 
         if (apn == NULL)
                 return NULL;
@@ -53,8 +54,26 @@ struct api_entry * api_entry_create(pid_t api, char * apn)
 
         e->state    = API_INIT;
 
-        pthread_mutex_init(&e->state_lock, NULL);
-        pthread_cond_init(&e->state_cond, NULL);
+        if (pthread_condattr_init(&cattr)) {
+                free(e);
+                return NULL;
+        }
+
+#ifndef __APPLE__
+        pthread_condattr_setclock(&cattr, PTHREAD_COND_CLOCK);
+#endif
+
+        if (pthread_mutex_init(&e->state_lock, NULL)) {
+                free(e);
+                return NULL;
+        }
+
+
+        if (pthread_cond_init(&e->state_cond, &cattr)) {
+                pthread_mutex_destroy(&e->state_lock);
+                free(e);
+                return NULL;
+        }
 
         return e;
 }
diff --git a/src/irmd/main.c b/src/irmd/main.c
index 658a08f6..cc13ccea 100644
--- a/src/irmd/main.c
+++ b/src/irmd/main.c
@@ -1360,7 +1360,7 @@ static struct irm_flow * flow_req_arr(pid_t     api,
         pid_t h_api = -1;
         int port_id = -1;
 
-        struct timespec wt = {IRMD_REQ_ARR_TIMEOUT % 1000,
+        struct timespec wt = {IRMD_REQ_ARR_TIMEOUT / 1000,
                               (IRMD_REQ_ARR_TIMEOUT % 1000) * MILLION};
 
         log_dbg("Flow req arrived from IPCP %d for %s on AE %s.",
diff --git a/src/irmd/registry.c b/src/irmd/registry.c
index 985ecda0..9512d664 100644
--- a/src/irmd/registry.c
+++ b/src/irmd/registry.c
@@ -76,7 +76,7 @@ static struct reg_entry * reg_entry_init(struct reg_entry * e,
         if (pthread_condattr_init(&cattr))
                 return NULL;
 
-#ifdef __APPLE__
+#ifndef __APPLE__
         pthread_condattr_setclock(&cattr, PTHREAD_COND_CLOCK);
 #endif
         if (pthread_cond_init(&e->state_cond, &cattr))
@@ -429,7 +429,7 @@ int reg_entry_leave_state(struct reg_entry *  e,
                 if (timeout)
                         ret = -pthread_cond_timedwait(&e->state_cond,
                                                       &e->state_lock,
-                                                      timeout);
+                                                      &abstime);
                 else
                         ret = -pthread_cond_wait(&e->state_cond,
                                                  &e->state_lock);
@@ -468,7 +468,7 @@ int reg_entry_wait_state(struct reg_entry *  e,
                 if (timeout)
                         ret = -pthread_cond_timedwait(&e->state_cond,
                                                       &e->state_lock,
-                                                      timeout);
+                                                      &abstime);
                 else
                         ret = -pthread_cond_wait(&e->state_cond,
                                                  &e->state_lock);
-- 
cgit v1.2.3