From: Kinsey Moore <kmoore@digium.com>
Date: Tue, 20 Aug 2013 14:25:30 +0000 (+0000)
Subject: Fix xmldoc memory leak
X-Git-Tag: 11.6.0-rc1~3^2~55
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e71f01d16c12d7c4b45ff0b29cc5aa3d1f4a5499;p=thirdparty%2Fasterisk.git

Fix xmldoc memory leak

This fixes a single-attribute memory leak that was occurring when the
"required" attribute was not true.

(closes issue ASTERISK-22249)
Reported by: Corey Farrell
Tested by: Corey Farrell
Patches:
    xmldoc-free_attr_required.patch uploaded by Corey Farrell
........

Merged revisions 397064 from http://svn.asterisk.org/svn/asterisk/branches/1.8


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/11@397067 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---

diff --git a/main/xmldoc.c b/main/xmldoc.c
index f537e06c82..18afbb6368 100644
--- a/main/xmldoc.c
+++ b/main/xmldoc.c
@@ -759,6 +759,7 @@ static char *xmldoc_get_syntax_fun(struct ast_xml_node *rootnode, const char *ro
 				if ((paramtype = ast_xml_get_attribute(node, "required"))) {
 					if (!ast_true(paramtype)) {
 						optmidnode = 1;
+						ast_xml_free_attr(paramtype);
 						break;
 					}
 					ast_xml_free_attr(paramtype);