lib: Wrap pthread_cond_timedwait for NULL abstime

We often have the pattern where we NULL-check abstime for
pthread_cond_timedwait to call pthread_cond_wait if it is.
Added a __timedwait function to wrap this.

Signed-off-by: Dimitri Staessens <[email protected]>
Signed-off-by: Sander Vrijders <[email protected]>

1 files changed, 3 insertions, 10 deletions

diff --git a/src/lib/shm_flow_set.c b/src/lib/shm_flow_set.c
index 9eb9bb65..75cf4b72 100644
--- a/src/lib/shm_flow_set.c
+++ b/src/lib/shm_flow_set.c
@@ -344,18 +344,11 @@ ssize_t shm_flow_set_wait(const struct shm_flow_set * set,
         pthread_cleanup_push(__cleanup_mutex_unlock, set->lock);
 
         while (set->heads[idx] == 0 && ret != -ETIMEDOUT) {
-                if (abstime != NULL) {
-                        ret = -pthread_cond_timedwait(set->conds + idx,
-                                                      set->lock,
-                                                      abstime);
+                ret = -__timedwait(set->conds + idx, set->lock, abstime);
 #ifdef HAVE_CANCEL_BUG
-                        if (ret == -ETIMEDOUT)
-                                pthread_testcancel();
+                if (ret == -ETIMEDOUT)
+                        pthread_testcancel();
 #endif
-                } else {
-                        ret = -pthread_cond_wait(set->conds + idx,
-                                                 set->lock);
-                }
 #ifdef HAVE_ROBUST_MUTEX
                 if (ret == -EOWNERDEAD)
                         pthread_mutex_consistent(set->lock);