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.

2013-05-24 19:49:20 -07:00 · 2013-05-24 19:49:20 -07:00 · 47c6981a9f
parent 6320042f6b
commit 47c6981a9f
1 changed files with 1 additions and 0 deletions
--- 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;
 	}