diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2016-03-14 09:15:06 +0100 |
---|---|---|
committer | Lukas Slebodnik <lslebodn@redhat.com> | 2016-03-14 11:43:56 +0100 |
commit | 265c7b5e5333befdadd4ac8d09f4147f211959fd (patch) | |
tree | 34130affd924707762ec91c638d77674afb0d54d /src | |
parent | de5fa34860886ad68fba5e739987e16c342e8f14 (diff) | |
download | sssd-265c7b5e5333befdadd4ac8d09f4147f211959fd.tar.gz sssd-265c7b5e5333befdadd4ac8d09f4147f211959fd.tar.xz sssd-265c7b5e5333befdadd4ac8d09f4147f211959fd.zip |
UTIL: Do not call stderr with negative number
The function connect return -1 in case of error
and we could call strerror with -1 instead of
errno.
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/util/sss_sockets.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/util/sss_sockets.c b/src/util/sss_sockets.c index ae259fae4..27be852eb 100644 --- a/src/util/sss_sockets.c +++ b/src/util/sss_sockets.c @@ -180,9 +180,11 @@ static void sssd_async_connect_done(struct tevent_context *ev, errno = 0; ret = connect(state->fd, (struct sockaddr *) &state->addr, state->addr_len); - if ((ret != EOK) && - (errno == EALREADY || errno == EINPROGRESS || errno == EINTR)) { - return; /* Try again later */ + if (ret == -1) { + ret = errno; + if (ret == EALREADY || ret == EINPROGRESS || ret == EINTR) { + return; /* Try again later */ + } } talloc_zfree(fde); |