From: Nathan Neulinger <nneul@neulinger.org>
Date: Fri, 15 Apr 2016 13:24:28 +0000 (-0500)
Subject: FS-9074: [mod_skinny] Fix incorrect location of free causing memory leak of xml when... 
X-Git-Tag: v1.6.8~5^2~15
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=44258634e18dd6a1c3c2edc760895bb75eedef64;p=thirdparty%2Ffreeswitch.git

FS-9074: [mod_skinny] Fix incorrect location of free causing memory leak of xml when certain errors occur
---

diff --git a/src/mod/endpoints/mod_skinny/skinny_server.c b/src/mod/endpoints/mod_skinny/skinny_server.c
index 4b706c4f37..ad635ab2e2 100644
--- a/src/mod/endpoints/mod_skinny/skinny_server.c
+++ b/src/mod/endpoints/mod_skinny/skinny_server.c
@@ -1353,9 +1353,6 @@ switch_status_t skinny_handle_register(listener_t *listener, skinny_message_t *r
 			}
 		}
 	}
-	if (xroot) {
-		switch_xml_free(xroot);
-	}
 
 	status = SWITCH_STATUS_SUCCESS;
 
@@ -1372,6 +1369,10 @@ switch_status_t skinny_handle_register(listener_t *listener, skinny_message_t *r
 	keepalive_listener(listener, NULL);
 
 end:
+	if (xroot) {
+		switch_xml_free(xroot);
+	}
+
 	if(params) {
 		switch_event_destroy(&params);
 	}