Fix: Proper copy constructor for HTMLHRElement.

2 files changed, 41 insertions, 3 deletions

diff --git a/src/html/html_hr_element.c b/src/html/html_hr_element.c
index 6d96c84..e6086f5 100644
--- a/src/html/html_hr_element.c
+++ b/src/html/html_hr_element.c
@@ -137,10 +137,39 @@ void _dom_virtual_html_hr_element_destroy(dom_node_internal *node)
 }
 
 /* The virtual copy function, see src/core/node.c for detail */
-dom_exception _dom_html_hr_element_copy(dom_node_internal *old,
-		                dom_node_internal **copy)
+dom_exception _dom_html_hr_element_copy(
+		dom_node_internal *old, dom_node_internal **copy)
 {
-	        return _dom_html_element_copy(old, copy);
+	dom_html_hr_element *new_node;
+	dom_exception err;
+
+	new_node = malloc(sizeof(dom_html_hr_element));
+	if (new_node == NULL)
+		return DOM_NO_MEM_ERR;
+
+	err = dom_html_hr_element_copy_internal(old, new_node);
+	if (err != DOM_NO_ERR) {
+		free(new_node);
+		return err;
+	}
+
+	*copy = (dom_node_internal *) new_node;
+
+	return DOM_NO_ERR;
+}
+
+dom_exception _dom_html_hr_element_copy_internal(
+		dom_html_hr_element *old,
+		dom_html_hr_element *new)
+{
+	dom_exception err;
+
+	err = dom_html_element_copy_internal(old, new);
+	if (err != DOM_NO_ERR) {
+		return err;
+	}
+
+	return DOM_NO_ERR;
 }
 
 /*-----------------------------------------------------------------------*/
diff --git a/src/html/html_hr_element.h b/src/html/html_hr_element.h
index 63b5bbc..e0eb40b 100644
--- a/src/html/html_hr_element.h
+++ b/src/html/html_hr_element.h
@@ -49,4 +49,13 @@ dom_exception _dom_html_hr_element_copy(dom_node_internal *old,
 	_dom_virtual_html_hr_element_destroy, \
 	_dom_html_hr_element_copy
 
+/* Helper functions*/
+dom_exception _dom_html_hr_element_copy_internal(
+		dom_html_hr_element *old,
+		dom_html_hr_element *new);
+#define dom_html_hr_element_copy_internal(o, n) \
+		_dom_html_hr_element_copy_internal( \
+				(dom_html_hr_element *) (o), \
+				(dom_html_hr_element *) (n))
+
 #endif