diff options
author | Ralph Boehme <slow@samba.org> | 2018-12-23 09:12:34 +0100 |
---|---|---|
committer | Stefan Metzmacher <metze@samba.org> | 2019-01-11 23:11:12 +0100 |
commit | a75f5de15dba1c9f41a2d86c4fa6573764d4cb8d (patch) | |
tree | 1114c07a11172f588caafc3c0db5de5209928b20 /lib/pthreadpool | |
parent | 57c7aaa36d2d146065cee78306cd38c0df34cd93 (diff) | |
download | samba-a75f5de15dba1c9f41a2d86c4fa6573764d4cb8d.tar.gz |
Revert "pthreadpool: ignore the return value of poll(NULL, 0UL, 1)"
This reverts commit 6da0d68f49bbd82f5a08427779c9f5ebd6f755aa.
See the discussion in
https://lists.samba.org/archive/samba-technical/2018-December/131731.html
for the reasoning behind this revert.
Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Diffstat (limited to 'lib/pthreadpool')
-rw-r--r-- | lib/pthreadpool/pthreadpool_tevent.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/pthreadpool/pthreadpool_tevent.c b/lib/pthreadpool/pthreadpool_tevent.c index 7088202d97c..b4a9786f231 100644 --- a/lib/pthreadpool/pthreadpool_tevent.c +++ b/lib/pthreadpool/pthreadpool_tevent.c @@ -873,7 +873,7 @@ static void pthreadpool_tevent_job_orphan(struct pthreadpool_tevent_job *job) */ PTHREAD_TEVENT_JOB_THREAD_FENCE(job); while (job->needs_fence.wrapper) { - (void)poll(NULL, 0, 1); + poll(NULL, 0, 1); PTHREAD_TEVENT_JOB_THREAD_FENCE(job); } job->wrapper = NULL; @@ -903,7 +903,7 @@ static void pthreadpool_tevent_job_orphan(struct pthreadpool_tevent_job *job) if (job->needs_fence.signaled) { break; } - (void)poll(NULL, 0, 1); + poll(NULL, 0, 1); PTHREAD_TEVENT_JOB_THREAD_FENCE(job); } |