diff options
| author | Stefan Metzmacher <metze@samba.org> | 2018-07-31 09:40:20 +0200 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2018-07-31 11:26:16 +0200 |
| commit | 6da0d68f49bbd82f5a08427779c9f5ebd6f755aa (patch) | |
| tree | b26ae9a341cff79eb5f713705696c04c76ae35f7 /lib/pthreadpool | |
| parent | 80c9219d6079532f31c4e415906a606d395af6e8 (diff) | |
| download | samba-6da0d68f49bbd82f5a08427779c9f5ebd6f755aa.tar.gz | |
pthreadpool: ignore the return value of poll(NULL, 0UL, 1)
Otherwise Coverity reports this:
CID 1438160: (CHECKED_RETURN)
Calling "poll(NULL, 0UL, 1)" without checking return value. This
library function may fail and return an error code.
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Volker Lendecke <vl@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 b4a9786f231..7088202d97c 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) { - poll(NULL, 0, 1); + (void)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; } - poll(NULL, 0, 1); + (void)poll(NULL, 0, 1); PTHREAD_TEVENT_JOB_THREAD_FENCE(job); } |