[gnome-software/uajain/fixup-subsume-metadata: 2/2] lib/app: Set entire metadata GVariant in gs_app_subsume_metadata



commit e0bab44f941cb44f93d5309d7a5e336df46ce07b
Author: Umang Jain <umang endlessm com>
Date:   Thu Aug 15 23:27:31 2019 +0530

    lib/app: Set entire metadata GVariant in gs_app_subsume_metadata
    
    Set the entire metadata GVariant instead of setting each metadata
    component as a string value. The reason is some metadata components
    are not GVariant strings (e.g. "flatpak::RefKind"). Subsuming
    such a component will throw a critical as:
    
    ```
    07:02:57:0442 GLib g_variant_get_string:
    assertion 'g_variant_is_of_type (value, G_VARIANT_TYPE_STRING) ||
    g_variant_is_of_type (value, G_VARIANT_TYPE_OBJECT_PATH) ||
    g_variant_is_of_type (value, G_VARIANT_TYPE_SIGNATURE)' failed
    ```
    
    for gs_app_get_metadata_item.

 lib/gs-app.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
---
diff --git a/lib/gs-app.c b/lib/gs-app.c
index 7ec827a7..b07921ce 100644
--- a/lib/gs-app.c
+++ b/lib/gs-app.c
@@ -4540,10 +4540,10 @@ gs_app_subsume_metadata (GsApp *app, GsApp *donor)
        g_autoptr(GList) keys = g_hash_table_get_keys (priv->metadata);
        for (GList *l = keys; l != NULL; l = l->next) {
                const gchar *key = l->data;
-               const gchar *value = gs_app_get_metadata_item (donor, key);
-               if (gs_app_get_metadata_item (app, key) != NULL)
+               GVariant *tmp = gs_app_get_metadata_variant (donor, key);
+               if (gs_app_get_metadata_variant (app, key) != NULL)
                        continue;
-               gs_app_set_metadata (app, key, value);
+               gs_app_set_metadata_variant (app, key, tmp);
        }
 }
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]