external/cde
1
0
Fork 0
mirror of git://git.code.sf.net/p/cdesktopenv/code synced 2025-03-09 15:50:02 +00:00
Code Issues Releases Wiki Activity

libtt: client init(): use fcntl F_SETFD correctly.

Browse source
This commit is contained in:
Jon Trulson 2012-07-27 19:17:14 -06:00
parent 2968429c0b
commit 5e6b400c3c
1 changed files with 24 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

24
cde/lib/tt/lib/mp/mp_rpc_client.C
View file

@ -207,10 +207,34 @@ init(_Tt_host_ptr &host, int program, int version,
// Set close-on-exec bit so a libtt client which forks and execs won't
// be short some fd's in the child.
#if defined(linux)
// JET - for linux, we need to do this properly - I don't know
// how the original code below can be correct, so we'll do it
// differently.
{
long flags;
if ((flags = fcntl(_socket, F_GETFD)) == -1)
{
_tt_syslog( 0, LOG_ERR, "_Tt_rpc_client::init(): "
"fcntl(F_GETFD): %m");
}
else
{
if (fcntl(_socket, F_SETFD, flags | FD_CLOEXEC) == -1)
_tt_syslog( 0, LOG_ERR, "_Tt_rpc_client::init(): "
"fcntl(F_SETFD): %m");
}
}
#else
if (-1==fcntl(_socket, F_SETFD, 1)) {
_tt_syslog( 0, LOG_ERR, "_Tt_rpc_client::init(): "
"fcntl(F_SETFD): %m");
}
#endif // linux
return(1);
}