appdata: exclude <name> element inside <developer>

<developer_name> has been deprecated in favor of <developer> with a
<name> child. We need to ensure that this developer name isn't parsed
as the application name.

Fixes: #5700
This commit is contained in:
Chris Williams
2024-02-21 13:24:26 -05:00
committed by Patrick Griffis
parent 17455f97fc
commit 2cba9e3db3

View File

@@ -43,6 +43,7 @@ typedef struct
gboolean in_text;
gboolean in_component;
gboolean in_content_rating;
gboolean in_developer;
char *lang;
guint64 timestamp;
const char *id; /* interned */
@@ -119,7 +120,7 @@ start_element (GMarkupParseContext *context,
{
data->in_text = TRUE;
}
else if (g_str_equal (element_name, "name") ||
else if ((!data->in_developer && g_str_equal (element_name, "name")) ||
g_str_equal (element_name, "summary"))
{
const char *lang = NULL;
@@ -259,6 +260,10 @@ start_element (GMarkupParseContext *context,
g_warning ("Ignoring content attribute missing id attribute");
}
}
else if (g_str_equal (element_name, "developer"))
{
data->in_developer = TRUE;
}
}
static void
@@ -294,7 +299,7 @@ end_element (GMarkupParseContext *context,
{
component->id = g_steal_pointer (&text);
}
else if (g_str_equal (element_name, "name"))
else if (!data->in_developer && g_str_equal (element_name, "name"))
{
g_hash_table_insert (component->names, g_steal_pointer (&data->lang), g_steal_pointer (&text));
}
@@ -316,6 +321,10 @@ end_element (GMarkupParseContext *context,
g_assert (component->content_rating != NULL);
g_hash_table_insert (component->content_rating, (gpointer) data->id, (gpointer) g_intern_string (text));
}
else if (g_str_equal (element_name, "developer"))
{
data->in_developer = FALSE;
}
}
static void