summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefan Metzmacher <metze@samba.org>2011-11-21 18:03:44 +0100
committerStefan Metzmacher <metze@samba.org>2011-11-21 19:58:57 +0100
commitaa2e415442fe121a3db5fa79e53ad732d78e9572 (patch)
treee7fc6c922abd6b9ed48586e48abe80817f558661
parentb895f1fb018405a5149c1c2a55e282bc88e6f8d3 (diff)
downloadsamba-aa2e415442fe121a3db5fa79e53ad732d78e9572.tar.gz
samba-aa2e415442fe121a3db5fa79e53ad732d78e9572.tar.bz2
samba-aa2e415442fe121a3db5fa79e53ad732d78e9572.zip
s3:winbindd_cm: close sockfd on error in cm_prepare_connection()
metze Autobuild-User: Stefan Metzmacher <metze@samba.org> Autobuild-Date: Mon Nov 21 19:58:58 CET 2011 on sn-devel-104
-rw-r--r--source3/winbindd/winbindd_cm.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/source3/winbindd/winbindd_cm.c b/source3/winbindd/winbindd_cm.c
index 7cfcaba545..cfb11039fb 100644
--- a/source3/winbindd/winbindd_cm.c
+++ b/source3/winbindd/winbindd_cm.c
@@ -802,6 +802,7 @@ static NTSTATUS cm_prepare_connection(const struct winbindd_domain *domain,
mutex = grab_named_mutex(talloc_tos(), controller,
WINBIND_SERVER_MUTEX_WAIT_TIME);
if (mutex == NULL) {
+ close(sockfd);
DEBUG(0,("cm_prepare_connection: mutex grab failed for %s\n",
controller));
result = NT_STATUS_POSSIBLE_DEADLOCK;
@@ -814,6 +815,7 @@ static NTSTATUS cm_prepare_connection(const struct winbindd_domain *domain,
controller, domain->alt_name,
SMB_SIGNING_DEFAULT, flags);
if (*cli == NULL) {
+ close(sockfd);
DEBUG(1, ("Could not cli_initialize\n"));
result = NT_STATUS_NO_MEMORY;
goto done;