[4/6] webservices: Fix duplicating attributes.

Hans Leidekker hans at codeweavers.com
Mon May 15 02:59:22 CDT 2017


Signed-off-by: Hans Leidekker <hans at codeweavers.com>
---
 dlls/webservices/reader.c | 22 +++++++++++++++++-----
 1 file changed, 17 insertions(+), 5 deletions(-)

diff --git a/dlls/webservices/reader.c b/dlls/webservices/reader.c
index 85a6dc9..4e7211f 100644
--- a/dlls/webservices/reader.c
+++ b/dlls/webservices/reader.c
@@ -151,16 +151,28 @@ void destroy_nodes( struct node *node )
 static WS_XML_ATTRIBUTE *dup_attribute( const WS_XML_ATTRIBUTE *src )
 {
     WS_XML_ATTRIBUTE *dst;
-    const WS_XML_STRING *prefix = (src->prefix && src->prefix->length) ? src->prefix : NULL;
+    const WS_XML_STRING *prefix = src->prefix;
     const WS_XML_STRING *localname = src->localName;
     const WS_XML_STRING *ns = src->localName;
+    const WS_XML_TEXT *text = src->value;
 
     if (!(dst = heap_alloc( sizeof(*dst) ))) return NULL;
     dst->singleQuote = src->singleQuote;
     dst->isXmlNs     = src->isXmlNs;
-    if (prefix && !(dst->prefix = alloc_xml_string( prefix->bytes, prefix->length ))) goto error;
-    if (localname && !(dst->localName = alloc_xml_string( localname->bytes, localname->length ))) goto error;
-    if (ns && !(dst->ns = alloc_xml_string( ns->bytes, ns->length ))) goto error;
+
+    if (!prefix) dst->prefix = NULL;
+    else if (!(dst->prefix = alloc_xml_string( prefix->bytes, prefix->length ))) goto error;
+    if (!(dst->localName = alloc_xml_string( localname->bytes, localname->length ))) goto error;
+    if (!(dst->ns = alloc_xml_string( ns->bytes, ns->length ))) goto error;
+
+    if (text)
+    {
+        WS_XML_UTF8_TEXT *utf8;
+        const WS_XML_UTF8_TEXT *utf8_src = (const WS_XML_UTF8_TEXT *)text;
+        if (!(utf8 = alloc_utf8_text( utf8_src->value.bytes, utf8_src->value.length ))) goto error;
+        dst->value = &utf8->text;
+    }
+
     return dst;
 
 error:
@@ -223,7 +235,7 @@ static struct node *dup_text_node( const WS_XML_TEXT_NODE *src )
     if (src->text)
     {
         WS_XML_UTF8_TEXT *utf8;
-        const WS_XML_UTF8_TEXT *utf8_src = (WS_XML_UTF8_TEXT *)src->text;
+        const WS_XML_UTF8_TEXT *utf8_src = (const WS_XML_UTF8_TEXT *)src->text;
         if (!(utf8 = alloc_utf8_text( utf8_src->value.bytes, utf8_src->value.length )))
         {
             free_node( node );
-- 
2.1.4




More information about the wine-patches mailing list