summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStephen Gallagher <sgallagh@redhat.com>2011-07-01 07:36:46 -0400
committerStephen Gallagher <sgallagh@redhat.com>2011-07-01 08:07:52 -0400
commita57cca518ba0b1dda8868448e176ab626272c84b (patch)
tree55420ac1796426a06eb09463e0b428c80fed465a
parente96c468ed06c3378e2aee6992dabe926d79e1a2d (diff)
downloadsssd-a57cca518ba0b1dda8868448e176ab626272c84b.tar.gz
sssd-a57cca518ba0b1dda8868448e176ab626272c84b.tar.xz
sssd-a57cca518ba0b1dda8868448e176ab626272c84b.zip
Do not attempt to close() a file descriptor < 0
Coverity 10886
-rw-r--r--src/util/sss_ldap.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/util/sss_ldap.c b/src/util/sss_ldap.c
index 9f76870f8..8d6660fab 100644
--- a/src/util/sss_ldap.c
+++ b/src/util/sss_ldap.c
@@ -359,7 +359,9 @@ struct tevent_req *sss_ldap_init_send(TALLOC_CTX *mem_ctx,
return req;
fail:
- close(state->sd);
+ if(state->sd >= 0) {
+ close(state->sd);
+ }
tevent_req_error(req, ret);
#else
DEBUG(3, ("ldap_init_fd not available, "