From: William King <william.king@quentustech.com>
Date: Sat, 25 May 2013 02:49:20 +0000 (-0700)
Subject: More explicit and defensive code so if the init fails we know we handling this pointe... 
X-Git-Tag: v1.2.13~333
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=47c6981a9f7618e508a7064a1cd8a98b6db84f5a;p=thirdparty%2Ffreeswitch.git

More explicit and defensive code so if the init fails we know we handling this pointer safely, because  rc_config_init will free the pointer if there was a problem, but it will not set the pointer to NULL.
---

diff --git a/src/mod/xml_int/mod_xml_radius/mod_xml_radius.c b/src/mod/xml_int/mod_xml_radius/mod_xml_radius.c
index dc1c9fd6bd..00e0bc7507 100644
--- a/src/mod/xml_int/mod_xml_radius/mod_xml_radius.c
+++ b/src/mod/xml_int/mod_xml_radius/mod_xml_radius.c
@@ -55,6 +55,7 @@ switch_status_t mod_xml_radius_new_handle(rc_handle **new_handle, switch_xml_t x
 	}
 
 	if ( rc_config_init(*new_handle) == NULL ) {
+		*new_handle = NULL;
 		goto err;
 	}