[389-devel] Please review: fix mem leak in admldapBuildInfoSSL when there is no password

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




From 41a05f031694a46786ffbd29c61b122fe2d56e3b Mon Sep 17 00:00:00 2001
From: Rich Megginson <rmeggins@xxxxxxxxxx>
Date: Tue, 13 Aug 2013 10:05:11 -0600
Subject: [PATCH 1/3] fix mem leak in admldapBuildInfoSSL when there is no password

---
 lib/libadmsslutil/admsslutil.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/lib/libadmsslutil/admsslutil.c b/lib/libadmsslutil/admsslutil.c
index 7ea9eb0..a86c37c 100644
--- a/lib/libadmsslutil/admsslutil.c
+++ b/lib/libadmsslutil/admsslutil.c
@@ -106,6 +106,7 @@ admldapBuildInfoSSL(AdmldapInfo info, int *errorcode)
   } else {
       /* no password means just punt rather than do anon bind */
       /* this mimics the same logic in admldapBuildInfoCbk() */
+      ldap_unbind_ext(ld, NULL, NULL);
       *errorcode = ADMUTIL_LDAP_ERR;
       return 1; /* have to return true here to mimic admldapBuildInfoCbk() */
   }
-- 
1.7.1

--
389-devel mailing list
389-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/389-devel

[Index of Archives]     [Fedora Directory Announce]     [Fedora Users]     [Older Fedora Users Mail]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Review]     [Fedora Art]     [Fedora Music]     [Fedora Packaging]     [CentOS]     [Fedora SELinux]     [Big List of Linux Books]     [KDE Users]     [Fedora Art]     [Fedora Docs]

  Powered by Linux