diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2018-03-17 13:53:12 +0100 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2018-03-17 23:33:58 +0100 |
| commit | 26c58027fb9df068c927abe61336ff6364fe3d0e (patch) | |
| tree | 221ec0f15696a31524120034ec60485d7bc2717a | |
| parent | f331eb1c0d060529a3b5e6517c8ac423e21796d3 (diff) | |
| download | lvm2-26c58027fb9df068c927abe61336ff6364fe3d0e.tar.gz | |
coverity: validate descriptor
Since this function is called with 'fd == -1', but Coverity can't see
this path can't be visited with this argument, add explicit check for
valid descriptor.
| -rw-r--r-- | daemons/clvmd/clvmd.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/daemons/clvmd/clvmd.c b/daemons/clvmd/clvmd.c index e15a40a6d..829c5e5f8 100644 --- a/daemons/clvmd/clvmd.c +++ b/daemons/clvmd/clvmd.c @@ -1999,6 +1999,9 @@ static int send_message(void *buf, int msglen, const char *csid, int fd, return clops->cluster_send_message(buf, msglen, csid, errtext); } + if (fd < 0) + return 0; + /* Make sure it all goes */ for (ptr = 0; ptr < msglen;) { if ((len = write(fd, (char*)buf + ptr, msglen - ptr)) <= 0) { |