diff options
author | dimitri staessens <dimitri.staessens@intec.ugent.be> | 2017-02-04 20:05:20 +0100 |
---|---|---|
committer | dimitri staessens <dimitri.staessens@intec.ugent.be> | 2017-02-04 20:05:20 +0100 |
commit | 9301dcbc95c95a82eba9e831f49253dcad73abb2 (patch) | |
tree | 2881368b3fcb28c96c5b9dac6c524d709288f99c /src/lib/cdap_req.c | |
parent | a30e244407655d16429ef442ac23db43a548bf95 (diff) | |
download | ouroboros-9301dcbc95c95a82eba9e831f49253dcad73abb2.tar.gz ouroboros-9301dcbc95c95a82eba9e831f49253dcad73abb2.zip |
lib: Handle receiving CDAP reply before wait
In extreme cases, the response to a CDAP request can arrive before the
AE doing the request has called cdap_request_wait. This PR handles it
by waiting for the state to move away from INIT before responding.
Diffstat (limited to 'src/lib/cdap_req.c')
-rw-r--r-- | src/lib/cdap_req.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/lib/cdap_req.c b/src/lib/cdap_req.c index 565cafd0..2f55b107 100644 --- a/src/lib/cdap_req.c +++ b/src/lib/cdap_req.c @@ -108,6 +108,7 @@ int cdap_req_wait(struct cdap_req * creq) } creq->state = REQ_PENDING; + pthread_cond_broadcast(&creq->cond); while (creq->state == REQ_PENDING) { ret = -pthread_cond_timedwait(&creq->cond, @@ -131,13 +132,20 @@ int cdap_req_wait(struct cdap_req * creq) return ret; } -void cdap_req_respond(struct cdap_req * creq, int response, buffer_t data) +void cdap_req_respond(struct cdap_req * creq, + int response, + buffer_t data) { assert(creq); pthread_mutex_lock(&creq->lock); + while (creq->state == REQ_INIT) + pthread_cond_wait(&creq->cond, &creq->lock); + if (creq->state != REQ_PENDING) { + creq->state = REQ_NULL; + pthread_cond_broadcast(&creq->cond); pthread_mutex_unlock(&creq->lock); return; } |