Attachment 14220 Details for Bug 14429 – ignore namespaced nodes and attributes

[patch] ignore namespaced nodes and attributes

dbus-glib-ignore-namespaced-nodes-2.patch (text/plain), 2.25 KB, created by Dan Williams on 2008-02-08 11:47:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Dan Williams

Created: 2008-02-08 11:47:44 UTC

Size: 2.25 KB

patch

obsolete

>diff --git a/dbus/dbus-gparser.c b/dbus/dbus-gparser.c
>index f296f96..cdd4e53 100644
>--- a/dbus/dbus-gparser.c
>+++ b/dbus/dbus-gparser.c
>@@ -128,13 +128,17 @@ locate_attributes (const char  *element_name,
> 
>       if (!found)
>         {
>-          g_set_error (error,
>-                       G_MARKUP_ERROR,
>-                       G_MARKUP_ERROR_PARSE,
>-                       _("Attribute \"%s\" is invalid on <%s> element in this context"),
>-                       attribute_names[i], element_name);
>-          retval = FALSE;
>-          goto out;
>+          /* We want to passthrough namespaced XML nodes that we don't know anything about. */
>+          if (strchr (attribute_names[i], ':') == NULL)
>+            {
>+              g_set_error (error,
>+                           G_MARKUP_ERROR,
>+                           G_MARKUP_ERROR_PARSE,
>+                           _("Attribute \"%s\" is invalid on <%s> element in this context"),
>+                           attribute_names[i], element_name);
>+              retval = FALSE;
>+              goto out;
>+            }
>         }
> 
>       ++i;
>@@ -177,6 +181,7 @@ struct Parser
>   PropertyInfo *property;
>   ArgInfo *arg;
>   gboolean in_annotation;
>+  guint unknown_namespaced_depth;
> };
> 
> Parser*
>@@ -791,10 +796,14 @@ parser_start_element (Parser      *parser,
>     }
>   else
>     {
>-      g_set_error (error, G_MARKUP_ERROR,
>-                   G_MARKUP_ERROR_PARSE,
>-                   _("Element <%s> not recognized"),
>-                   element_name);
>+      if (strchr (element_name, ':') != NULL)
>+        /* Passthrough XML-namespaced nodes */
>+        parser->unknown_namespaced_depth += 1;
>+      else
>+        g_set_error (error, G_MARKUP_ERROR,
>+                     G_MARKUP_ERROR_PARSE,
>+                     _("Element <%s> not recognized"),
>+                     element_name);
>     }
>   
>   return TRUE;
>@@ -844,6 +853,11 @@ parser_end_element (Parser      *parser,
>       if (parser->node_stack == NULL)
>         parser->result = top; /* We are done, store the result */      
>     }
>+  else if (strchr (element_name, ':') != NULL)
>+    {
>+      /* Passthrough XML-namespaced nodes */
>+      parser->unknown_namespaced_depth -= 1;
>+    }
>   else
>     g_assert_not_reached (); /* should have had an error on start_element */
>   
>

Actions: View | Splinter Review

Attachments on bug 14429: 14220 | 15088

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.