[PATCH 2/2] fix memory leaks in xml_prepare_inform_message

[Jonas Gorski <jonas.gorski@gmail.com>]

Always free allocated value strings regardless of success of node
creation.

Contributed by T-Labs, Deutsche Telekom Innovation Laboratories

Signed-off-by: Jonas Gorski <jonas.gorski@gmail.com>
---
 src/xml/xml.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/xml/xml.c b/src/xml/xml.c
index 38f8583..8e46ee6 100644
--- a/src/xml/xml.c
+++ b/src/xml/xml.c
@@ -282,9 +282,9 @@ xml_prepare_inform_message(char **msg_out)
                goto error;
        if (c) {
                busy_node = mxmlNewText(busy_node, 0, c);
+               free(c);
                if (!busy_node)
                        goto error;
-               free(c);
        }
 
        tmp = "InternetGatewayDevice.ManagementServer.ConnectionRequestURL";
@@ -300,9 +300,9 @@ xml_prepare_inform_message(char **msg_out)
                goto error;
        if (c) {
                busy_node = mxmlNewText(busy_node, 0, c);
+               free(c);
                if (!busy_node)
                        goto error;
-               free(c);
        }
 
        *msg_out = mxmlSaveAllocString(tree, MXML_NO_CALLBACK);
-- 
1.7.10