diff options
| author |   Dave Wallace <dwallacelf@gmail.com> | 2017-10-26 16:12:04 -0400 |
|---|---|---|
| committer |   Chris Luke <chris_luke@comcast.com> | 2017-10-26 21:25:26 +0000 |
| commit | faf9d7730828b80814a233eeecf7affc046193d3 (patch) | |
| tree | 8d2dbf8c03b9fc117ba4c42f897c43cda7997d64 /src/vcl/vcom.c | |
| parent | 617dffaee47fc6fbc0dca85dc07c53ca9b57bf90 (diff) | |
VCL-LDPRELOAD: Fix more coverity warnings
vcom.c:
CID 178227: Logically dead code in vcom.c
vcom_socket.c:
CID 178254: Dereference after null check
CID 178250: Out-of-bounds access
vppcom.c:
CID 178252: Unused value
Suppress vppcom_session_attr() debug output.
Change-Id: I1d47bafb84fc0ad00c642392ae3cb6761fd3fb17
Signed-off-by: Dave Wallace <dwallacelf@gmail.com>
Diffstat (limited to 'src/vcl/vcom.c')
| -rw-r--r-- | src/vcl/vcom.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/vcl/vcom.c b/src/vcl/vcom.c index 9bd5e0e54a2..be1e4c5b47a 100644 --- a/src/vcl/vcom.c +++ b/src/vcl/vcom.c @@ -1283,11 +1283,7 @@ vcom_select (int __nfds, fd_set * __restrict __readfds, vcom_fd_set_merge ( /* dest, select sets */ &new_nfds, - __readfds, - __writefds, - __exceptfds, - __readfds || __writefds || __exceptfds ? - &new_nfd : NULL, + __readfds, __writefds, __exceptfds, &new_nfd, /* src1, vcom sets */ vcom_nfds, __readfds ? &vcom_readfds : NULL, |