diff options
| author | Michael Scherer <misc@redhat.com> | 2017-02-19 15:18:56 +0100 |
|---|---|---|
| committer | Shyamsundar Ranganathan <srangana@redhat.com> | 2017-02-23 06:59:10 -0500 |
| commit | afe3a0d2869b706a3d5257f39d6acb9cbe856b87 (patch) | |
| tree | d88cb6087c307d6d2bfb07c638fb5407b5528a76 /libglusterfs | |
| parent | a88cb01c78c9a13d2879917b54ead26dce8cc63c (diff) | |
Properly verify the return code of fcntl
Coverty scan complain about it, even if in practice, this
doesn't seems to have much impact.
Change-Id: I513f7d393889625d22dded25ef4c7477f68d1064
BUG: 1424793
Signed-off-by: Michael Scherer <misc@redhat.com>
Reviewed-on: https://review.gluster.org/16671
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
Tested-by: Nigel Babu <nigelb@redhat.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Reviewed-by: Niels de Vos <ndevos@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Poornima G <pgurusid@redhat.com>
Diffstat (limited to 'libglusterfs')
| -rw-r--r-- | libglusterfs/src/event.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libglusterfs/src/event.c b/libglusterfs/src/event.c index 09ecce1599d..6aaa53499df 100644 --- a/libglusterfs/src/event.c +++ b/libglusterfs/src/event.c @@ -219,7 +219,7 @@ event_dispatch_destroy (struct event_pool *event_pool) /* Make the write end of the pipe nonblocking */ flags = fcntl(fd[1], F_GETFL); flags |= O_NONBLOCK; - fcntl(fd[1], F_SETFL, flags); + ret = fcntl(fd[1], F_SETFL, flags); if (ret < 0) goto out; |