diff options
| author | Stefan Metzmacher <metze@samba.org> | 2018-07-31 09:42:40 +0200 |
|---|---|---|
| committer | Volker Lendecke <vl@samba.org> | 2018-07-31 11:26:16 +0200 |
| commit | f68b5ee75f002ed542a8423070fb01e3b0e83f65 (patch) | |
| tree | f4e4f75324b818f5c03034f15d598c2e96cd32f7 /lib/pthreadpool | |
| parent | 6da0d68f49bbd82f5a08427779c9f5ebd6f755aa (diff) | |
| download | samba-f68b5ee75f002ed542a8423070fb01e3b0e83f65.tar.gz | |
pthreadpool: reset monitor_fd after calling tevent_fd_set_auto_close()
This tries to convince Coverity that we don't have a resource leak:
CID 1438157: (RESOURCE_LEAK)
Handle variable "monitor_fd" going out of scope leaks the handle.
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 | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/lib/pthreadpool/pthreadpool_tevent.c b/lib/pthreadpool/pthreadpool_tevent.c index 7088202d97c..12148f18123 100644 --- a/lib/pthreadpool/pthreadpool_tevent.c +++ b/lib/pthreadpool/pthreadpool_tevent.c @@ -719,6 +719,7 @@ static int pthreadpool_tevent_register_ev( return ENOMEM; } tevent_fd_set_auto_close(glue->fde); + monitor_fd = -1; } /* |