lib: trace IR, values: reset pointers to `NULL` on destruction
[babeltrace.git] / lib / trace-ir / fields.c
index 376bc5d66c60d80ecbaa17738cd1122aa92a9121..8eadc75711f9e902fbe187ae9e90a99dacec7865 100644 (file)
@@ -1,8 +1,4 @@
 /*
- * fields.c
- *
- * Babeltrace trace IR - Event Fields
- *
  * Copyright 2013, 2014 Jérémie Galarneau <jeremie.galarneau@efficios.com>
  *
  * Author: Jérémie Galarneau <jeremie.galarneau@efficios.com>
 #include <babeltrace/lib-logging-internal.h>
 
 #include <babeltrace/assert-pre-internal.h>
+#include <babeltrace/trace-ir/private-fields.h>
 #include <babeltrace/trace-ir/fields.h>
 #include <babeltrace/trace-ir/fields-internal.h>
 #include <babeltrace/trace-ir/field-classes-internal.h>
 #include <babeltrace/object-internal.h>
-#include <babeltrace/ref.h>
+#include <babeltrace/object.h>
 #include <babeltrace/compiler-internal.h>
 #include <babeltrace/compat/fcntl-internal.h>
 #include <babeltrace/align-internal.h>
@@ -142,16 +139,16 @@ struct bt_field *create_variant_field(struct bt_field_class *);
 
 static
 struct bt_field *(* const field_create_funcs[])(struct bt_field_class *) = {
-       [BT_FIELD_CLASS_ID_UNSIGNED_INTEGER]    = create_integer_field,
-       [BT_FIELD_CLASS_ID_SIGNED_INTEGER]      = create_integer_field,
-       [BT_FIELD_CLASS_ID_UNSIGNED_ENUMERATION] = create_integer_field,
-       [BT_FIELD_CLASS_ID_SIGNED_ENUMERATION]  = create_integer_field,
-       [BT_FIELD_CLASS_ID_REAL]                        = create_real_field,
-       [BT_FIELD_CLASS_ID_STRING]              = create_string_field,
-       [BT_FIELD_CLASS_ID_STRUCTURE]           = create_structure_field,
-       [BT_FIELD_CLASS_ID_STATIC_ARRAY]                = create_static_array_field,
-       [BT_FIELD_CLASS_ID_DYNAMIC_ARRAY]       = create_dynamic_array_field,
-       [BT_FIELD_CLASS_ID_VARIANT]             = create_variant_field,
+       [BT_FIELD_CLASS_TYPE_UNSIGNED_INTEGER]  = create_integer_field,
+       [BT_FIELD_CLASS_TYPE_SIGNED_INTEGER]    = create_integer_field,
+       [BT_FIELD_CLASS_TYPE_UNSIGNED_ENUMERATION] = create_integer_field,
+       [BT_FIELD_CLASS_TYPE_SIGNED_ENUMERATION]        = create_integer_field,
+       [BT_FIELD_CLASS_TYPE_REAL]                      = create_real_field,
+       [BT_FIELD_CLASS_TYPE_STRING]            = create_string_field,
+       [BT_FIELD_CLASS_TYPE_STRUCTURE]         = create_structure_field,
+       [BT_FIELD_CLASS_TYPE_STATIC_ARRAY]              = create_static_array_field,
+       [BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY]     = create_dynamic_array_field,
+       [BT_FIELD_CLASS_TYPE_VARIANT]           = create_variant_field,
 };
 
 static
@@ -174,16 +171,16 @@ void destroy_variant_field(struct bt_field *field);
 
 static
 void (* const field_destroy_funcs[])(struct bt_field *) = {
-       [BT_FIELD_CLASS_ID_UNSIGNED_INTEGER]    = destroy_integer_field,
-       [BT_FIELD_CLASS_ID_SIGNED_INTEGER]      = destroy_integer_field,
-       [BT_FIELD_CLASS_ID_UNSIGNED_ENUMERATION] = destroy_integer_field,
-       [BT_FIELD_CLASS_ID_SIGNED_ENUMERATION]  = destroy_integer_field,
-       [BT_FIELD_CLASS_ID_REAL]                        = destroy_real_field,
-       [BT_FIELD_CLASS_ID_STRING]              = destroy_string_field,
-       [BT_FIELD_CLASS_ID_STRUCTURE]           = destroy_structure_field,
-       [BT_FIELD_CLASS_ID_STATIC_ARRAY]                = destroy_array_field,
-       [BT_FIELD_CLASS_ID_DYNAMIC_ARRAY]       = destroy_array_field,
-       [BT_FIELD_CLASS_ID_VARIANT]             = destroy_variant_field,
+       [BT_FIELD_CLASS_TYPE_UNSIGNED_INTEGER]  = destroy_integer_field,
+       [BT_FIELD_CLASS_TYPE_SIGNED_INTEGER]    = destroy_integer_field,
+       [BT_FIELD_CLASS_TYPE_UNSIGNED_ENUMERATION] = destroy_integer_field,
+       [BT_FIELD_CLASS_TYPE_SIGNED_ENUMERATION]        = destroy_integer_field,
+       [BT_FIELD_CLASS_TYPE_REAL]                      = destroy_real_field,
+       [BT_FIELD_CLASS_TYPE_STRING]            = destroy_string_field,
+       [BT_FIELD_CLASS_TYPE_STRUCTURE]         = destroy_structure_field,
+       [BT_FIELD_CLASS_TYPE_STATIC_ARRAY]              = destroy_array_field,
+       [BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY]     = destroy_array_field,
+       [BT_FIELD_CLASS_TYPE_VARIANT]           = destroy_variant_field,
 };
 
 struct bt_field_class *bt_field_borrow_class(struct bt_field *field)
@@ -192,10 +189,16 @@ struct bt_field_class *bt_field_borrow_class(struct bt_field *field)
        return field->class;
 }
 
-enum bt_field_class_id bt_field_get_class_id(struct bt_field *field)
+struct bt_private_field_class *bt_private_field_borrow_class(
+               struct bt_private_field *field)
+{
+       return (void *) bt_field_borrow_class((void *) field);
+}
+
+enum bt_field_class_type bt_field_get_class_type(struct bt_field *field)
 {
        BT_ASSERT_PRE_NON_NULL(field, "Field");
-       return field->class->id;
+       return field->class->type;
 }
 
 BT_HIDDEN
@@ -204,8 +207,8 @@ struct bt_field *bt_field_create(struct bt_field_class *fc)
        struct bt_field *field = NULL;
 
        BT_ASSERT_PRE_NON_NULL(fc, "Field class");
-       BT_ASSERT(bt_field_class_has_known_id(fc));
-       field = field_create_funcs[fc->id](fc);
+       BT_ASSERT(bt_field_class_has_known_type(fc));
+       field = field_create_funcs[fc->type](fc);
        if (!field) {
                BT_LIB_LOGE("Cannot create field object from field classe: "
                        "%![fc-]+F", fc);
@@ -224,7 +227,7 @@ void init_field(struct bt_field *field, struct bt_field_class *fc,
        BT_ASSERT(fc);
        bt_object_init_unique(&field->base);
        field->methods = methods;
-       field->class = bt_get(fc);
+       field->class = bt_object_get_ref(fc);
 }
 
 static
@@ -282,7 +285,7 @@ struct bt_field *create_string_field(struct bt_field_class *fc)
                sizeof(char), 1);
        if (!string_field->buf) {
                BT_LOGE_STR("Failed to allocate a GArray.");
-               BT_PUT(string_field);
+               BT_OBJECT_PUT_REF_AND_RESET(string_field);
                goto end;
        }
 
@@ -350,7 +353,7 @@ struct bt_field *create_structure_field(struct bt_field_class *fc)
                        &struct_field->fields)) {
                BT_LIB_LOGE("Cannot create structure member fields: "
                        "%![fc-]+F", fc);
-               BT_PUT(struct_field);
+               BT_OBJECT_PUT_REF_AND_RESET(struct_field);
                goto end;
        }
 
@@ -378,7 +381,7 @@ struct bt_field *create_variant_field(struct bt_field_class *fc)
                        &var_field->fields)) {
                BT_LIB_LOGE("Cannot create variant member fields: "
                        "%![fc-]+F", fc);
-               BT_PUT(var_field);
+               BT_OBJECT_PUT_REF_AND_RESET(var_field);
                goto end;
        }
 
@@ -442,7 +445,7 @@ struct bt_field *create_static_array_field(struct bt_field_class *fc)
        if (init_array_field_fields(array_field)) {
                BT_LIB_LOGE("Cannot create static array fields: "
                        "%![fc-]+F", fc);
-               BT_PUT(array_field);
+               BT_OBJECT_PUT_REF_AND_RESET(array_field);
                goto end;
        }
 
@@ -469,7 +472,7 @@ struct bt_field *create_dynamic_array_field(struct bt_field_class *fc)
        if (init_array_field_fields(array_field)) {
                BT_LIB_LOGE("Cannot create dynamic array fields: "
                        "%![fc-]+F", fc);
-               BT_PUT(array_field);
+               BT_OBJECT_PUT_REF_AND_RESET(array_field);
                goto end;
        }
 
@@ -489,8 +492,10 @@ int64_t bt_field_signed_integer_get_value(struct bt_field *field)
        return int_field->value.i;
 }
 
-void bt_field_signed_integer_set_value(struct bt_field *field, int64_t value)
+void bt_private_field_signed_integer_set_value(
+               struct bt_private_field *priv_field, int64_t value)
 {
+       struct bt_field *field = (void *) priv_field;
        struct bt_field_integer *int_field = (void *) field;
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
@@ -514,9 +519,10 @@ uint64_t bt_field_unsigned_integer_get_value(struct bt_field *field)
        return int_field->value.u;
 }
 
-void bt_field_unsigned_integer_set_value(struct bt_field *field,
-               uint64_t value)
+void bt_private_field_unsigned_integer_set_value(
+               struct bt_private_field *priv_field, uint64_t value)
 {
+       struct bt_field *field = (void *) priv_field;
        struct bt_field_integer *int_field = (void *) field;
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
@@ -536,16 +542,18 @@ double bt_field_real_get_value(struct bt_field *field)
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
        BT_ASSERT_PRE_FIELD_IS_SET(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field, BT_FIELD_CLASS_ID_REAL, "Field");
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field, BT_FIELD_CLASS_TYPE_REAL, "Field");
        return real_field->value;
 }
 
-void bt_field_real_set_value(struct bt_field *field, double value)
+void bt_private_field_real_set_value(struct bt_private_field *priv_field,
+               double value)
 {
+       struct bt_field *field = (void *) priv_field;
        struct bt_field_real *real_field = (void *) field;
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field, BT_FIELD_CLASS_ID_REAL, "Field");
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field, BT_FIELD_CLASS_TYPE_REAL, "Field");
        BT_ASSERT_PRE_FIELD_HOT(field, "Field");
        BT_ASSERT_PRE(
                !((struct bt_field_class_real *) field->class)->is_single_precision ||
@@ -566,8 +574,8 @@ int bt_field_unsigned_enumeration_get_mapping_labels(struct bt_field *field,
        BT_ASSERT_PRE_NON_NULL(label_array, "Label array (output)");
        BT_ASSERT_PRE_NON_NULL(label_array, "Count (output)");
        BT_ASSERT_PRE_FIELD_IS_SET(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field,
-               BT_FIELD_CLASS_ID_UNSIGNED_ENUMERATION, "Field");
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field,
+               BT_FIELD_CLASS_TYPE_UNSIGNED_ENUMERATION, "Field");
        return bt_field_class_unsigned_enumeration_get_mapping_labels_by_value(
                field->class, int_field->value.u, label_array, count);
 }
@@ -582,8 +590,8 @@ int bt_field_signed_enumeration_get_mapping_labels(struct bt_field *field,
        BT_ASSERT_PRE_NON_NULL(label_array, "Label array (output)");
        BT_ASSERT_PRE_NON_NULL(label_array, "Count (output)");
        BT_ASSERT_PRE_FIELD_IS_SET(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field,
-               BT_FIELD_CLASS_ID_SIGNED_ENUMERATION, "Field");
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field,
+               BT_FIELD_CLASS_TYPE_SIGNED_ENUMERATION, "Field");
        return bt_field_class_signed_enumeration_get_mapping_labels_by_value(
                field->class, int_field->value.i, label_array, count);
 }
@@ -594,7 +602,7 @@ const char *bt_field_string_get_value(struct bt_field *field)
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
        BT_ASSERT_PRE_FIELD_IS_SET(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field, BT_FIELD_CLASS_ID_STRING,
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field, BT_FIELD_CLASS_TYPE_STRING,
                "Field");
        return (const char *) string_field->buf->data;
 }
@@ -605,32 +613,38 @@ uint64_t bt_field_string_get_length(struct bt_field *field)
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
        BT_ASSERT_PRE_FIELD_IS_SET(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field, BT_FIELD_CLASS_ID_STRING,
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field, BT_FIELD_CLASS_TYPE_STRING,
                "Field");
        return string_field->length;
 }
 
-int bt_field_string_set_value(struct bt_field *field, const char *value)
+int bt_private_field_string_set_value(struct bt_private_field *priv_field,
+               const char *value)
 {
+       struct bt_field *field = (void *) priv_field;
+
        BT_ASSERT_PRE_NON_NULL(field, "Field");
        BT_ASSERT_PRE_NON_NULL(value, "Value");
        BT_ASSERT_PRE_FIELD_HOT(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field, BT_FIELD_CLASS_ID_STRING,
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field, BT_FIELD_CLASS_TYPE_STRING,
                "Field");
-       bt_field_string_clear(field);
-       return bt_field_string_append_with_length(field, value,
+       bt_private_field_string_clear(priv_field);
+       return bt_private_field_string_append_with_length(priv_field, value,
                (uint64_t) strlen(value));
 }
 
-int bt_field_string_append(struct bt_field *field, const char *value)
+int bt_private_field_string_append(struct bt_private_field *field,
+               const char *value)
 {
-       return bt_field_string_append_with_length(field, value,
-               (uint64_t) strlen(value));
+       return bt_private_field_string_append_with_length(field,
+               value, (uint64_t) strlen(value));
 }
 
-int bt_field_string_append_with_length(struct bt_field *field,
+int bt_private_field_string_append_with_length(
+               struct bt_private_field *priv_field,
                const char *value, uint64_t length)
 {
+       struct bt_field *field = (void *) priv_field;
        struct bt_field_string *string_field = (void *) field;
        char *data;
        uint64_t new_length;
@@ -638,8 +652,8 @@ int bt_field_string_append_with_length(struct bt_field *field,
        BT_ASSERT_PRE_NON_NULL(field, "Field");
        BT_ASSERT_PRE_NON_NULL(value, "Value");
        BT_ASSERT_PRE_FIELD_HOT(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field,
-               BT_FIELD_CLASS_ID_STRING, "Field");
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field,
+               BT_FIELD_CLASS_TYPE_STRING, "Field");
 
        /* Make sure no null bytes are appended */
        BT_ASSERT_PRE(memchr(value, '\0', length) == NULL,
@@ -660,14 +674,15 @@ int bt_field_string_append_with_length(struct bt_field *field,
        return 0;
 }
 
-int bt_field_string_clear(struct bt_field *field)
+int bt_private_field_string_clear(struct bt_private_field *priv_field)
 {
+       struct bt_field *field = (void *) priv_field;
        struct bt_field_string *string_field = (void *) field;
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
        BT_ASSERT_PRE_FIELD_HOT(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field,
-               BT_FIELD_CLASS_ID_STRING, "Field");
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field,
+               BT_FIELD_CLASS_TYPE_STRING, "Field");
        string_field->length = 0;
        bt_field_set_single(field, true);
        return 0;
@@ -682,15 +697,16 @@ uint64_t bt_field_array_get_length(struct bt_field *field)
        return array_field->length;
 }
 
-int bt_field_dynamic_array_set_length(struct bt_field *field,
-               uint64_t length)
+int bt_private_field_dynamic_array_set_length(
+               struct bt_private_field *priv_field, uint64_t length)
 {
        int ret = 0;
+       struct bt_field *field = (void *) priv_field;
        struct bt_field_array *array_field = (void *) field;
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field,
-               BT_FIELD_CLASS_ID_DYNAMIC_ARRAY, "Field");
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field,
+               BT_FIELD_CLASS_TYPE_DYNAMIC_ARRAY, "Field");
        BT_ASSERT_PRE_FIELD_HOT(field, "Field");
 
        if (unlikely(length > array_field->fields->len)) {
@@ -737,18 +753,34 @@ struct bt_field *bt_field_array_borrow_element_field_by_index(
        return array_field->fields->pdata[index];
 }
 
+struct bt_private_field *
+bt_private_field_array_borrow_element_field_by_index(
+               struct bt_private_field *field, uint64_t index)
+{
+       return (void *) bt_field_array_borrow_element_field_by_index(
+               (void *) field, index);
+}
+
 struct bt_field *bt_field_structure_borrow_member_field_by_index(
                struct bt_field *field, uint64_t index)
 {
        struct bt_field_structure *struct_field = (void *) field;
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field,
-               BT_FIELD_CLASS_ID_STRUCTURE, "Field");
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field,
+               BT_FIELD_CLASS_TYPE_STRUCTURE, "Field");
        BT_ASSERT_PRE_VALID_INDEX(index, struct_field->fields->len);
        return struct_field->fields->pdata[index];
 }
 
+struct bt_private_field *
+bt_private_field_structure_borrow_member_field_by_index(
+               struct bt_private_field *field, uint64_t index)
+{
+       return (void *) bt_field_structure_borrow_member_field_by_index(
+               (void *) field, index);
+}
+
 struct bt_field *bt_field_structure_borrow_member_field_by_name(
                struct bt_field *field, const char *name)
 {
@@ -760,8 +792,8 @@ struct bt_field *bt_field_structure_borrow_member_field_by_name(
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
        BT_ASSERT_PRE_NON_NULL(name, "Field name");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field,
-               BT_FIELD_CLASS_ID_STRUCTURE, "Field");
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field,
+               BT_FIELD_CLASS_TYPE_STRUCTURE, "Field");
        struct_fc = (void *) field->class;
 
        if (!g_hash_table_lookup_extended(struct_fc->common.name_to_index, name,
@@ -776,27 +808,44 @@ end:
        return ret_field;
 }
 
+struct bt_private_field *
+bt_private_field_structure_borrow_member_field_by_name(
+               struct bt_private_field *field, const char *name)
+{
+       return (void *) bt_field_structure_borrow_member_field_by_name(
+               (void *) field, name);
+}
+
 struct bt_field *bt_field_variant_borrow_selected_option_field(
                struct bt_field *field)
 {
        struct bt_field_variant *var_field = (void *) field;
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field,
-               BT_FIELD_CLASS_ID_VARIANT, "Field");
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field,
+               BT_FIELD_CLASS_TYPE_VARIANT, "Field");
        BT_ASSERT_PRE(var_field->selected_field,
                "Variant field has no selected field: %!+f", field);
        return var_field->selected_field;
 }
 
-int bt_field_variant_select_option_field(struct bt_field *field,
-               uint64_t index)
+struct bt_private_field *
+bt_private_field_variant_borrow_selected_option_field(
+               struct bt_private_field *field)
+{
+       return (void *) bt_field_variant_borrow_selected_option_field(
+               (void *) field);
+}
+
+int bt_private_field_variant_select_option_field(
+               struct bt_private_field *priv_field, uint64_t index)
 {
+       struct bt_field *field = (void *) priv_field;
        struct bt_field_variant *var_field = (void *) field;
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field,
-               BT_FIELD_CLASS_ID_VARIANT, "Field");
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field,
+               BT_FIELD_CLASS_TYPE_VARIANT, "Field");
        BT_ASSERT_PRE_FIELD_HOT(field, "Field");
        BT_ASSERT_PRE_VALID_INDEX(index, var_field->fields->len);
        var_field->selected_field = var_field->fields->pdata[index];
@@ -810,8 +859,8 @@ uint64_t bt_field_variant_get_selected_option_field_index(
        struct bt_field_variant *var_field = (void *) field;
 
        BT_ASSERT_PRE_NON_NULL(field, "Field");
-       BT_ASSERT_PRE_FIELD_HAS_CLASS_ID(field,
-               BT_FIELD_CLASS_ID_VARIANT, "Field");
+       BT_ASSERT_PRE_FIELD_HAS_CLASS_TYPE(field,
+               BT_FIELD_CLASS_TYPE_VARIANT, "Field");
        BT_ASSERT_PRE(var_field->selected_field,
                "Variant field has no selected field: %!+f", field);
        return var_field->selected_index;
@@ -822,7 +871,7 @@ void bt_field_finalize(struct bt_field *field)
 {
        BT_ASSERT(field);
        BT_LOGD_STR("Putting field's class.");
-       bt_put(field->class);
+       BT_OBJECT_PUT_REF_AND_RESET(field->class);
 }
 
 static
@@ -854,6 +903,7 @@ void destroy_structure_field(struct bt_field *field)
 
        if (struct_field->fields) {
                g_ptr_array_free(struct_field->fields, TRUE);
+               struct_field->fields = NULL;
        }
 
        g_free(field);
@@ -870,6 +920,7 @@ void destroy_variant_field(struct bt_field *field)
 
        if (var_field->fields) {
                g_ptr_array_free(var_field->fields, TRUE);
+               var_field->fields = NULL;
        }
 
        g_free(field);
@@ -886,6 +937,7 @@ void destroy_array_field(struct bt_field *field)
 
        if (array_field->fields) {
                g_ptr_array_free(array_field->fields, TRUE);
+               array_field->fields = NULL;
        }
 
        g_free(field);
@@ -902,6 +954,7 @@ void destroy_string_field(struct bt_field *field)
 
        if (string_field->buf) {
                g_array_free(string_field->buf, TRUE);
+               string_field->buf = NULL;
        }
 
        g_free(field);
@@ -911,8 +964,8 @@ BT_HIDDEN
 void bt_field_destroy(struct bt_field *field)
 {
        BT_ASSERT(field);
-       BT_ASSERT(bt_field_class_has_known_id(field->class));
-       field_destroy_funcs[field->class->id](field);
+       BT_ASSERT(bt_field_class_has_known_type(field->class));
+       field_destroy_funcs[field->class->type](field);
 }
 
 static
This page took 0.03257 seconds and 4 git commands to generate.