diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2011-08-04 12:13:50 +0000 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2011-08-04 12:13:50 +0000 |
commit | fb2f985dfe55d3bd0522850899181ca92802469d (patch) | |
tree | 583969360c13b2667d28a67300df35e9fcbe475c /daemons/clvmd/clvmd-singlenode.c | |
parent | 765918a52869857c73838e2b432bdde0af04846d (diff) | |
download | lvm2-fb2f985dfe55d3bd0522850899181ca92802469d.tar.gz lvm2-fb2f985dfe55d3bd0522850899181ca92802469d.tar.xz lvm2-fb2f985dfe55d3bd0522850899181ca92802469d.zip |
Add test for fcntl error in singlenode client code.
Static analyzer noticed this check could be handy.
Diffstat (limited to 'daemons/clvmd/clvmd-singlenode.c')
-rw-r--r-- | daemons/clvmd/clvmd-singlenode.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/daemons/clvmd/clvmd-singlenode.c b/daemons/clvmd/clvmd-singlenode.c index 26c1f7ed..3a46fb60 100644 --- a/daemons/clvmd/clvmd-singlenode.c +++ b/daemons/clvmd/clvmd-singlenode.c @@ -55,7 +55,10 @@ static int init_comms(void) goto error; } /* Set Close-on-exec */ - fcntl(listen_fd, F_SETFD, 1); + if (fcntl(listen_fd, F_SETFD, 1)) { + DEBUGLOG("Setting CLOEXEC on client fd faile: %s\n", strerror(errno)); + goto error; + } memset(&addr, 0, sizeof(addr)); memcpy(addr.sun_path, SINGLENODE_CLVMD_SOCKNAME, |