diff options
author | dimitri staessens <dimitri.staessens@intec.ugent.be> | 2016-05-14 11:41:16 +0200 |
---|---|---|
committer | dimitri staessens <dimitri.staessens@intec.ugent.be> | 2016-05-14 11:41:16 +0200 |
commit | aff55f153ddb247357f59b3740477e90eba527bd (patch) | |
tree | 94494b4589c1ae41b913976566812eaeb8b3e068 | |
parent | fe4eb1fe61042a08686be54a37f72de72bef70f8 (diff) | |
parent | 182e493a5668b66d3017d92130b5fb31d325796d (diff) | |
download | ouroboros-aff55f153ddb247357f59b3740477e90eba527bd.tar.gz ouroboros-aff55f153ddb247357f59b3740477e90eba527bd.zip |
Merged in dstaesse/ouroboros/be (pull request #83)
lib: shm_ap_rbuff: removed bad memory allocation
-rw-r--r-- | src/lib/shm_ap_rbuff.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/src/lib/shm_ap_rbuff.c b/src/lib/shm_ap_rbuff.c index 6c977cbb..6c04ccc5 100644 --- a/src/lib/shm_ap_rbuff.c +++ b/src/lib/shm_ap_rbuff.c @@ -245,18 +245,17 @@ struct rb_entry * shm_ap_rbuff_read(struct shm_ap_rbuff * rb) if (rb == NULL) return NULL; - e = malloc(sizeof(*e)); - if (e == NULL) - return NULL; - pthread_mutex_lock(rb->shm_mutex); if (shm_rbuff_used(rb) == 0) { pthread_mutex_unlock(rb->shm_mutex); - free(e); return NULL; } + e = malloc(sizeof(*e)); + if (e == NULL) + return NULL; + *e = *(rb->shm_base + *rb->ptr_tail); *rb->ptr_tail = (*rb->ptr_tail + 1) & (SHM_RBUFF_SIZE -1); |