Сериализация JSON (GLib) приводит к утечке памяти для GHashTable - PullRequest
0 голосов
/ 11 июля 2019

Я пытаюсь сериализовать и десериализовать GHashTable в и из JSON и valgrind сообщает о памяти, которая определенно теряется в результате этого.Есть подавления для g_hash_table_new в файле glib.suppressions, которые они распространяют, но нет для g_hash_table_new_full, что я вижу кучу.Моя хеш-таблица - это свойство GObject, которое было установлено с помощью g_param_spec_pointer, и функция для его сериализации:

static JsonNode *
foo_obj_serialize_property (JsonSerializable *serializable,
                            const gchar      *name,
                            const GValue     *value,
                            GParamSpec       *pspec)
{
  JsonNode *retval = NULL;

  if (g_strcmp0 (name, "list") == 0)
    {
      GHashTable *list = NULL;
      GHashTableIter iter;
      JsonArray *arr = NULL;
      gpointer key, val;

      retval = json_node_new (JSON_NODE_ARRAY);

      g_return_val_if_fail (value != NULL, retval);
      g_return_val_if_fail (G_VALUE_HOLDS_POINTER (value), retval);

      list = g_value_get_pointer (value);

      arr = json_array_new ();

      if (list != NULL)
        {
          g_hash_table_iter_init (&iter, list);
          while (g_hash_table_iter_next (&iter, &key, &val))
            {
              JsonNode *node = NULL;
              JsonObject *obj = NULL;
              FooItem *item;

              item = FOO_ITEM (val);
              node = json_gobject_serialize (G_OBJECT (item));

              if (JSON_NODE_HOLDS_OBJECT (node))
                {
                  obj = json_node_get_object (node);
                  json_array_add_object_element (arr, obj);
                }
            }
        }

      json_node_take_array (retval, arr);
    }

  return retval;
}

, десериализация:

static gboolean
foo_obj_deserialize_property (JsonSerializable *serializable,
                              const gchar      *name,
                              GValue           *value,
                              GParamSpec       *pspec,
                              JsonNode         *property_node)
{
  gboolean retval = FALSE;

  if (g_strcmp0 (name, "list") == 0)
    {
      GHashTable *list;
      JsonArray *arr;

      arr = json_node_get_array (property_node);
      list = g_hash_table_new_full (g_str_hash,
                                    g_str_equal,
                                    g_free,
                                    NULL);

      for (gint i = 0; i < json_array_get_length (arr); i++)
        {
          g_autoptr (FooItem) item = NULL;
          JsonNode *node = NULL;

          node = json_array_get_element (arr, i);

          item = FOO_ITEM (json_gobject_deserialize (FOO_TYPE_ITEM, node));
          g_return_val_if_fail (FOO_IS_ITEM (item), FALSE);
          g_object_ref (item);
          g_hash_table_insert (list,
                               g_strdup (foo_item_get_key (item)),
                               item);
        }

      g_value_set_pointer (value, list);

      retval = TRUE;
    }

  return retval;
}

my GObject Свойства класса get / set имеют следующие функции:

static void
foo_obj_get_property (GObject    *object,
                      guint       prop_id,
                      GValue     *value,
                      GParamSpec *pspec)
{
  FooObj *self = FOO_OBJ (object);

  switch (prop_id)
    {
    case PROP_LIST:
      g_value_set_pointer (value, self->list);
      break;

    default:
      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
    }
}

static void
foo_obj_set_property (GObject      *object,
                      guint         prop_id,
                      const GValue *value,
                      GParamSpec   *pspec)
{
  FooObj *self = FOO_OBJ (object);

  switch (prop_id)
    {
    case PROP_LIST:
      foo_obj_set_list (self, g_value_get_pointer (value));
      break;

    default:
      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
    }
}

и, наконец, класс getter / setter:

GHashTable *
foo_obj_get_list (FooObj *self)
{
  GHashTable *list;

  g_return_val_if_fail (FOO_IS_OBJ (self), NULL);

  g_object_get (self, "list", &list, NULL);

  return list;
}

void
foo_obj_set_list (FooObj     *self,
                  GHashTable *list)
{
  g_return_if_fail (FOO_IS_OBJ (self));

  if (self->list == list)
    return;

  if (list)
    g_hash_table_ref (list);

  if (self->list)
    g_hash_table_unref (self->list);

  self->list = list;

  g_object_notify_by_pspec (G_OBJECT (self), properties [PROP_LIST]);
}

Я запускаю проверку памяти, используя valgrind --tool=memcheck --leak-check=full --show-leak-kinds=all --leak-resolution=high --num-callers=20 --suppressions=../tests/glib.supp foo, и яполучить сообщения для потерянной памяти для сериализации, показывающей:

==24303== 32 bytes in 1 blocks are definitely lost in loss record 1,029 of 1,852
==24303==    at 0x483877F: malloc (vg_replace_malloc.c:299)
==24303==    by 0x494CAB1: g_malloc (in /usr/lib/libglib-2.0.so.0.6000.4)
==24303==    by 0x492C8E4: g_slice_alloc (in /usr/lib/libglib-2.0.so.0.6000.4)
==24303==    by 0x4933839: g_slice_alloc0 (in /usr/lib/libglib-2.0.so.0.6000.4)
==24303==    by 0x4D72CEE: json_node_alloc (in /usr/lib/libjson-glib-1.0.so.0.400.4)
==24303==    by 0x4D72D37: json_node_new (in /usr/lib/libjson-glib-1.0.so.0.400.4)
==24303==    by 0x4D77B37: json_gobject_serialize (in /usr/lib/libjson-glib-1.0.so.0.400.4)
==24303==    by 0x4867AC9: foo_item_serialize_property (foo-item.c:71)
==24303==    by 0x4D779C1: ??? (in /usr/lib/libjson-glib-1.0.so.0.400.4)
==24303==    by 0x4D77B42: json_gobject_serialize (in /usr/lib/libjson-glib-1.0.so.0.400.4)
==24303==    by 0x4861681: foo_obj_serialize_property (foo-obj.c:146)
==24303==    by 0x4D779C1: ??? (in /usr/lib/libjson-glib-1.0.so.0.400.4)
==24303==    by 0x4D77B42: json_gobject_serialize (in /usr/lib/libjson-glib-1.0.so.0.400.4)
==24303==    by 0x4D77B82: json_gobject_to_data (in /usr/lib/libjson-glib-1.0.so.0.400.4)

и для десериализации:

==24303== 184 (88 direct, 96 indirect) bytes in 1 blocks are definitely lost in loss record 1,774 of 1,852
==24303==    at 0x483877F: malloc (vg_replace_malloc.c:299)
==24303==    by 0x494CAB1: g_malloc (in /usr/lib/libglib-2.0.so.0.6000.4)
==24303==    by 0x492C8E4: g_slice_alloc (in /usr/lib/libglib-2.0.so.0.6000.4)
==24303==    by 0x4966C5E: g_hash_table_new_full (in /usr/lib/libglib-2.0.so.0.6000.4)
==24303==    by 0x4861AB2: foo_obj_deserialize_property (foo-obj.c:264)
==24303==    by 0x4D77F00: ??? (in /usr/lib/libjson-glib-1.0.so.0.400.4)
==24303==    by 0x4D7826B: json_gobject_from_data (in /usr/lib/libjson-glib-1.0.so.0.400.4)
==24303==    by 0x4862BEE: foo_obj_deserialize (foo-obj.c:472)

Мне трудно увидеть то, что я пропустил, и различные попытки очиститьуказатели приводят к двойному свободному или ошибочному подсчету ссылок == 0. Я знаю, что существует множество GLib подавлений из-за того, как он обрабатывает память, но я не знаю, подпадают ли они под это.

IВозможно, я смогу упаковать хэш-таблицу и упростить сериализацию, зарегистрировав функцию с json-glib, но я не хочу идти в этом направлении, если мне не нужно.

1 Ответ

0 голосов
/ 11 июля 2019
node = json_gobject_serialize (G_OBJECT (item));

После этой строки вы ничего не делаете с node, ни сохраняете ее где-либо, ни бесплатно. Это приводит к утечке памяти. Также обратите внимание, что JsonNode является GBoxed, а не GObject. Используйте json-node-free , чтобы освободить его.


Я бы также предложил вам прочитать о подсчете ссылок и анализе выходных данных valgrind .

Трассировка стека говорит вам, где была выделена утечка памяти. К сожалению, Memcheck не может сказать вам, почему утечка памяти.

Для метода deserialize он был размещен в (foo-obj.c: 264), то есть g_hash_table_new_full. Вы создаете хеш-таблицу, но не уничтожаете ее.

...