Merge pull request #1 from brianmichel/bmichel/fix-potential-leak

Fix potential leak

HTMLNode.m

@@ -29,6 +29,8 @@ void setAttributeNamed(xmlNode * node, const char * nameStr, const char * value)
 	char * newVal = (char *)malloc(strlen(value)+1);
 	memcpy (newVal, value, strlen(value)+1);
 
+	bool copyUsed = false;
+
 	for(xmlAttrPtr attr = node->properties; NULL != attr; attr = attr->next)
 	{
 		if (strcmp((char*)attr->name, nameStr) == 0)
@@ -37,12 +39,21 @@ void setAttributeNamed(xmlNode * node, const char * nameStr, const char * value)
 			{
 				free(child->content);
 				child->content = (xmlChar*)newVal;
+
+				if (!copyUsed)
+				{
+					copyUsed = true;
+				}
 				break;
 			}
 			break;
 		}
 	}
 
+	if (!copyUsed)
+	{
+		free(newVal);
+	}
 
 }