From: James Rouzier <jrouzier@inverse.ca>
Date: Tue, 1 Nov 2016 17:13:53 +0000 (-0400)
Subject: Check if info.ldapai_extensions is not NULL before freeing it
X-Git-Tag: release_3_0_13~123^2~1
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a583d7aaccb2096fb6ce0d79adf1bc139cbb31e9;p=thirdparty%2Ffreeradius-server.git

Check if info.ldapai_extensions is not NULL before freeing it
---

diff --git a/src/modules/rlm_ldap/rlm_ldap.c b/src/modules/rlm_ldap/rlm_ldap.c
index 15f3368c9b3..4b19da17585 100644
--- a/src/modules/rlm_ldap/rlm_ldap.c
+++ b/src/modules/rlm_ldap/rlm_ldap.c
@@ -693,10 +693,12 @@ static int mod_instantiate(CONF_SECTION *conf, void *instance)
 			INFO("rlm_ldap: libldap vendor: %s, version: %i", info.ldapai_vendor_name,
 			     info.ldapai_vendor_version);
 
-			for ( i = 0; info.ldapai_extensions[i] != NULL; i++) {
-				ldap_memfree(info.ldapai_extensions[i]);
+			if (info.ldapai_extensions != NULL ) {
+				for ( i = 0; info.ldapai_extensions[i] != NULL; i++) {
+					ldap_memfree(info.ldapai_extensions[i]);
+				}
+				ldap_memfree(info.ldapai_extensions);
 			}
-			ldap_memfree(info.ldapai_extensions);
 			ldap_memfree(info.ldapai_vendor_name);
 		} else {
 			DEBUG("rlm_ldap: Falling back to build time libldap version info.  Query for LDAP_OPT_API_INFO "