summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorLukas Slebodnik <lslebodn@redhat.com>2016-03-14 09:15:06 +0100
committerLukas Slebodnik <lslebodn@redhat.com>2016-03-14 11:43:56 +0100
commit265c7b5e5333befdadd4ac8d09f4147f211959fd (patch)
tree34130affd924707762ec91c638d77674afb0d54d /src
parentde5fa34860886ad68fba5e739987e16c342e8f14 (diff)
downloadsssd-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.c8
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);