From 47c6981a9f7618e508a7064a1cd8a98b6db84f5a Mon Sep 17 00:00:00 2001 From: William King Date: Fri, 24 May 2013 19:49:20 -0700 Subject: [PATCH] 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. --- src/mod/xml_int/mod_xml_radius/mod_xml_radius.c | 1 + 1 file changed, 1 insertion(+) 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; }