#include <babeltrace2/babeltrace.h>
-#define BT_LOG_OUTPUT_LEVEL ((enum bt_log_level) log_level)
-#define BT_LOG_TAG "PLUGIN/SRC.CTF.FS/QUERY"
-#include "logging/comp-logging.h"
-#include "logging/log.h"
+#include "cpp-common/bt2/exc.hpp"
+#include "cpp-common/bt2c/glib-up.hpp"
+#include "cpp-common/bt2c/libc-up.hpp"
-#include "common/assert.h"
+#include "plugins/common/param-validation/param-validation.h"
#include "../common/src/metadata/tsdl/decoder.hpp"
#include "fs.hpp"
-#include "metadata.hpp"
#include "query.hpp"
#define METADATA_TEXT_SIG "/* CTF 1.8"
bool set = false;
};
-bt_component_class_query_method_status
-metadata_info_query(bt_self_component_class_source *self_comp_class_src, const bt_value *params,
- bt_logging_level log_level, const bt_value **user_result)
-{
- bt_component_class_query_method_status status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_OK;
- bt_self_component_class *self_comp_class =
- bt_self_component_class_source_as_self_component_class(self_comp_class_src);
- bt_value *result = NULL;
- const bt_value *path_value = NULL;
- FILE *metadata_fp = NULL;
- int ret;
- int bo;
- const char *path;
- bool is_packetized;
- struct ctf_metadata_decoder *decoder = NULL;
- ctf_metadata_decoder_config decoder_cfg;
- enum ctf_metadata_decoder_status decoder_status;
- GString *g_metadata_text = NULL;
- const char *plaintext;
-
- result = bt_value_map_create();
- if (!result) {
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_MEMORY_ERROR;
- goto error;
- }
+static bt_param_validation_map_value_entry_descr metadataInfoQueryParamsDesc[] = {
+ {"path", BT_PARAM_VALIDATION_MAP_VALUE_ENTRY_MANDATORY,
+ bt_param_validation_value_descr::makeString()},
+ BT_PARAM_VALIDATION_MAP_VALUE_ENTRY_END};
- BT_ASSERT(params);
+bt2::Value::Shared metadata_info_query(const bt2::ConstMapValue params, const bt2c::Logger& logger)
+{
+ gchar *validateError = nullptr;
+ const auto validationStatus = bt_param_validation_validate(
+ params.libObjPtr(), metadataInfoQueryParamsDesc, &validateError);
- if (!bt_value_is_map(params)) {
- BT_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp_class,
- "Query parameters is not a map value object.");
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_ERROR;
- goto error;
- }
+ if (validationStatus == BT_PARAM_VALIDATION_STATUS_MEMORY_ERROR) {
+ throw bt2::MemoryError {};
+ } else if (validationStatus == BT_PARAM_VALIDATION_STATUS_VALIDATION_ERROR) {
+ const bt2c::GCharUP deleter {validateError};
- path_value = bt_value_map_borrow_entry_value_const(params, "path");
- if (!path_value) {
- BT_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp_class, "Mandatory `path` parameter missing");
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_ERROR;
- goto error;
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(logger, bt2::Error, "{}", validateError);
}
- if (!bt_value_is_string(path_value)) {
- BT_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp_class,
- "`path` parameter is required to be a string value");
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_ERROR;
- goto error;
+ const auto path = params["path"]->asString().value();
+ bt2c::FileUP metadataFp {ctf_fs_metadata_open_file(path, logger)};
+ if (!metadataFp) {
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(logger, bt2::Error,
+ "Cannot open trace metadata: path=\"{}\".", path);
}
- path = bt_value_string_get(path_value);
-
- BT_ASSERT(path);
- metadata_fp = ctf_fs_metadata_open_file(path, log_level, self_comp_class);
- if (!metadata_fp) {
- BT_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp_class, "Cannot open trace metadata: path=\"%s\".",
- path);
- goto error;
- }
+ bool is_packetized;
+ int bo;
+ int ret = ctf_metadata_decoder_is_packetized(metadataFp.get(), &is_packetized, &bo, logger);
- ret = ctf_metadata_decoder_is_packetized(metadata_fp, &is_packetized, &bo, log_level, NULL);
if (ret) {
- BT_COMP_CLASS_LOGE_APPEND_CAUSE(
- self_comp_class,
- "Cannot check whether or not the metadata stream is packetized: path=\"%s\".", path);
- goto error;
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(
+ logger, bt2::Error,
+ "Cannot check whether or not the metadata stream is packetized: path=\"{}\".", path);
}
- decoder_cfg.log_level = log_level;
- decoder_cfg.self_comp_class = self_comp_class;
+ ctf_metadata_decoder_config decoder_cfg {logger};
decoder_cfg.keep_plain_text = true;
- decoder = ctf_metadata_decoder_create(&decoder_cfg);
+ ctf_metadata_decoder_up decoder = ctf_metadata_decoder_create(&decoder_cfg);
if (!decoder) {
- BT_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp_class,
- "Cannot create metadata decoder: path=\"%s\".", path);
- goto error;
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(
+ logger, bt2::Error, "Cannot create metadata decoder: path=\"{}}\".", path);
}
- rewind(metadata_fp);
- decoder_status = ctf_metadata_decoder_append_content(decoder, metadata_fp);
+ rewind(metadataFp.get());
+ ctf_metadata_decoder_status decoder_status =
+ ctf_metadata_decoder_append_content(decoder.get(), metadataFp.get());
if (decoder_status) {
- BT_COMP_CLASS_LOGE_APPEND_CAUSE(
- self_comp_class, "Cannot update metadata decoder's content: path=\"%s\".", path);
- goto error;
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(
+ logger, bt2::Error, "Cannot update metadata decoder's content: path=\"{}\".", path);
}
- plaintext = ctf_metadata_decoder_get_text(decoder);
- g_metadata_text = g_string_new(NULL);
+ const char *plain_text = ctf_metadata_decoder_get_text(decoder.get());
+ std::string metadata_text;
- if (!g_metadata_text) {
- goto error;
+ if (strncmp(plain_text, METADATA_TEXT_SIG, sizeof(METADATA_TEXT_SIG) - 1) != 0) {
+ metadata_text = METADATA_TEXT_SIG;
+ metadata_text += " */\n\n";
}
- if (strncmp(plaintext, METADATA_TEXT_SIG, sizeof(METADATA_TEXT_SIG) - 1) != 0) {
- g_string_assign(g_metadata_text, METADATA_TEXT_SIG);
- g_string_append(g_metadata_text, " */\n\n");
- }
-
- g_string_append(g_metadata_text, plaintext);
-
- ret = bt_value_map_insert_string_entry(result, "text", g_metadata_text->str);
- if (ret) {
- BT_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp_class,
- "Cannot insert metadata text into query result.");
- goto error;
- }
+ metadata_text += plain_text;
- ret = bt_value_map_insert_bool_entry(result, "is-packetized", is_packetized);
- if (ret) {
- BT_COMP_CLASS_LOGE_APPEND_CAUSE(
- self_comp_class, "Cannot insert \"is-packetized\" attribute into query result.");
- goto error;
- }
-
- goto end;
+ const auto result = bt2::MapValue::create();
+ result->insert("text", metadata_text);
+ result->insert("is-packetized", is_packetized);
-error:
- BT_VALUE_PUT_REF_AND_RESET(result);
- result = NULL;
-
- if (status >= 0) {
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_ERROR;
- }
-
-end:
- if (g_metadata_text) {
- g_string_free(g_metadata_text, TRUE);
- }
- ctf_metadata_decoder_destroy(decoder);
-
- if (metadata_fp) {
- ret = fclose(metadata_fp);
- if (ret) {
- BT_LOGE_ERRNO("Cannot close metadata file stream", ": path=\"%s\"", path);
- }
- }
-
- *user_result = result;
- return status;
+ return result;
}
-static int add_range(bt_value *info, struct range *range, const char *range_name)
+static void add_range(const bt2::MapValue info, struct range *range, const char *range_name)
{
- int ret = 0;
- bt_value_map_insert_entry_status status;
- bt_value *range_map;
-
if (!range->set) {
/* Not an error. */
- goto end;
+ return;
}
- status = bt_value_map_insert_empty_map_entry(info, range_name, &range_map);
- if (status != BT_VALUE_MAP_INSERT_ENTRY_STATUS_OK) {
- ret = -1;
- goto end;
- }
-
- status = bt_value_map_insert_signed_integer_entry(range_map, "begin", range->begin_ns);
- if (status != BT_VALUE_MAP_INSERT_ENTRY_STATUS_OK) {
- ret = -1;
- goto end;
- }
-
- status = bt_value_map_insert_signed_integer_entry(range_map, "end", range->end_ns);
- if (status != BT_VALUE_MAP_INSERT_ENTRY_STATUS_OK) {
- ret = -1;
- goto end;
- }
-
-end:
- return ret;
+ const auto rangeMap = info.insertEmptyMap(range_name);
+ rangeMap.insert("begin", range->begin_ns);
+ rangeMap.insert("end", range->end_ns);
}
-static int populate_stream_info(struct ctf_fs_ds_file_group *group, bt_value *group_info,
- struct range *stream_range)
+static void populate_stream_info(struct ctf_fs_ds_file_group *group, const bt2::MapValue groupInfo,
+ struct range *stream_range, const bt2c::Logger& logger)
{
- int ret = 0;
- bt_value_map_insert_entry_status insert_status;
- struct ctf_fs_ds_index_entry *first_ds_index_entry, *last_ds_index_entry;
- gchar *port_name = NULL;
-
/*
* Since each `struct ctf_fs_ds_file_group` has a sorted array of
* `struct ctf_fs_ds_index_entry`, we can compute the stream range from
BT_ASSERT(group->index->entries->len > 0);
/* First entry. */
- first_ds_index_entry =
+ ctf_fs_ds_index_entry *first_ds_index_entry =
(struct ctf_fs_ds_index_entry *) g_ptr_array_index(group->index->entries, 0);
/* Last entry. */
- last_ds_index_entry = (struct ctf_fs_ds_index_entry *) g_ptr_array_index(
+ ctf_fs_ds_index_entry *last_ds_index_entry = (struct ctf_fs_ds_index_entry *) g_ptr_array_index(
group->index->entries, group->index->entries->len - 1);
stream_range->begin_ns = first_ds_index_entry->timestamp_begin_ns;
stream_range->set =
stream_range->begin_ns != UINT64_C(-1) && stream_range->end_ns != UINT64_C(-1);
- ret = add_range(group_info, stream_range, "range-ns");
- if (ret) {
- goto end;
- }
-
- port_name = ctf_fs_make_port_name(group);
- if (!port_name) {
- ret = -1;
- goto end;
- }
+ add_range(groupInfo, stream_range, "range-ns");
- insert_status = bt_value_map_insert_string_entry(group_info, "port-name", port_name);
- if (insert_status != BT_VALUE_MAP_INSERT_ENTRY_STATUS_OK) {
- ret = -1;
- goto end;
+ bt2c::GCharUP portName = ctf_fs_make_port_name(group);
+ if (!portName) {
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(logger, bt2::Error, "Failed to make port name");
}
-end:
- g_free(port_name);
- return ret;
+ groupInfo.insert("port-name", portName.get());
}
-static int populate_trace_info(const struct ctf_fs_trace *trace, bt_value *trace_info,
- bt_logging_level log_level, bt_self_component_class *self_comp_class)
+static void populate_trace_info(const struct ctf_fs_trace *trace, const bt2::MapValue traceInfo,
+ const bt2c::Logger& logger)
{
- int ret = 0;
- size_t group_idx;
- bt_value_map_insert_entry_status insert_status;
- bt_value_array_append_element_status append_status;
- bt_value *file_groups = NULL;
-
BT_ASSERT(trace->ds_file_groups);
/* Add trace range info only if it contains streams. */
if (trace->ds_file_groups->len == 0) {
- ret = -1;
- BT_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp_class, "Trace has no streams: trace-path=%s",
- trace->path->str);
- goto end;
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(
+ logger, bt2::Error, "Trace has no streams: trace-path={}", trace->path->str);
}
- insert_status = bt_value_map_insert_empty_array_entry(trace_info, "stream-infos", &file_groups);
- if (insert_status != BT_VALUE_MAP_INSERT_ENTRY_STATUS_OK) {
- ret = -1;
- goto end;
- }
+ const auto fileGroups = traceInfo.insertEmptyArray("stream-infos");
/* Find range of all stream groups, and of the trace. */
- for (group_idx = 0; group_idx < trace->ds_file_groups->len; group_idx++) {
- bt_value *group_info;
+ for (size_t group_idx = 0; group_idx < trace->ds_file_groups->len; group_idx++) {
range group_range;
ctf_fs_ds_file_group *group =
(ctf_fs_ds_file_group *) g_ptr_array_index(trace->ds_file_groups, group_idx);
- append_status = bt_value_array_append_empty_map_element(file_groups, &group_info);
- if (append_status != BT_VALUE_ARRAY_APPEND_ELEMENT_STATUS_OK) {
- ret = -1;
- goto end;
- }
-
- ret = populate_stream_info(group, group_info, &group_range);
- if (ret) {
- goto end;
- }
+ const auto groupInfo = fileGroups.appendEmptyMap();
+ populate_stream_info(group, groupInfo, &group_range, logger);
}
-
-end:
- return ret;
}
-bt_component_class_query_method_status
-trace_infos_query(bt_self_component_class_source *self_comp_class_src, const bt_value *params,
- bt_logging_level log_level, const bt_value **user_result)
+bt2::Value::Shared trace_infos_query(const bt2::ConstMapValue params, const bt2c::Logger& logger)
{
- struct ctf_fs_component *ctf_fs = NULL;
- bt_component_class_query_method_status status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_OK;
- bt_self_component_class *self_comp_class =
- bt_self_component_class_source_as_self_component_class(self_comp_class_src);
- bt_value *result = NULL;
- const bt_value *inputs_value = NULL;
- const bt_value *trace_name_value;
- int ret = 0;
- bt_value *trace_info = NULL;
- bt_value_array_append_element_status append_status;
-
- BT_ASSERT(params);
-
- if (!bt_value_is_map(params)) {
- BT_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp_class,
- "Query parameters is not a map value object.");
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_ERROR;
- goto error;
- }
-
- ctf_fs = ctf_fs_component_create(log_level);
+ ctf_fs_component::UP ctf_fs = ctf_fs_component_create(logger);
if (!ctf_fs) {
- goto error;
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(logger, bt2::Error,
+ "Cannot create ctf_fs_component");
}
- if (!read_src_fs_parameters(params, &inputs_value, &trace_name_value, ctf_fs, NULL,
- self_comp_class)) {
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_ERROR;
- goto error;
- }
+ const bt_value *inputs_value = NULL;
+ const bt_value *trace_name_value;
- if (ctf_fs_component_create_ctf_fs_trace(ctf_fs, inputs_value, trace_name_value, NULL,
- self_comp_class)) {
- goto error;
+ if (!read_src_fs_parameters(params.libObjPtr(), &inputs_value, &trace_name_value,
+ ctf_fs.get())) {
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(logger, bt2::Error, "Failed to read parameters");
}
- result = bt_value_array_create();
- if (!result) {
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_MEMORY_ERROR;
- goto error;
+ if (ctf_fs_component_create_ctf_fs_trace(ctf_fs.get(), inputs_value, trace_name_value, NULL)) {
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(logger, bt2::Error, "Failed to create trace");
}
- append_status = bt_value_array_append_empty_map_element(result, &trace_info);
- if (append_status != BT_VALUE_ARRAY_APPEND_ELEMENT_STATUS_OK) {
- BT_COMP_CLASS_LOGE_APPEND_CAUSE(self_comp_class, "Failed to create trace info map.");
- goto error;
- }
+ const auto result = bt2::ArrayValue::create();
+ const auto traceInfo = result->appendEmptyMap();
+ populate_trace_info(ctf_fs->trace.get(), traceInfo, logger);
- ret = populate_trace_info(ctf_fs->trace, trace_info, log_level, self_comp_class);
- if (ret) {
- goto error;
- }
+ return result;
+}
- goto end;
+static bt_param_validation_map_value_entry_descr supportInfoQueryParamsDesc[] = {
+ {"type", BT_PARAM_VALIDATION_MAP_VALUE_ENTRY_MANDATORY,
+ bt_param_validation_value_descr::makeString()},
+ {"input", BT_PARAM_VALIDATION_MAP_VALUE_ENTRY_MANDATORY,
+ bt_param_validation_value_descr::makeString()},
+ BT_PARAM_VALIDATION_MAP_VALUE_ENTRY_END};
-error:
- BT_VALUE_PUT_REF_AND_RESET(result);
+bt2::Value::Shared support_info_query(const bt2::ConstMapValue params, const bt2c::Logger& logger)
+{
+ gchar *validateError = NULL;
+ const auto validationStatus = bt_param_validation_validate(
+ params.libObjPtr(), supportInfoQueryParamsDesc, &validateError);
- if (status >= 0) {
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_ERROR;
- }
+ if (validationStatus == BT_PARAM_VALIDATION_STATUS_MEMORY_ERROR) {
+ throw bt2::MemoryError {};
+ } else if (validationStatus == BT_PARAM_VALIDATION_STATUS_VALIDATION_ERROR) {
+ const bt2c::GCharUP deleter {validateError};
-end:
- if (ctf_fs) {
- ctf_fs_destroy(ctf_fs);
- ctf_fs = NULL;
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(logger, bt2::Error, "{}", validateError);
}
- *user_result = result;
- return status;
-}
-
-bt_component_class_query_method_status
-support_info_query(bt_self_component_class_source *comp_class, const bt_value *params,
- bt_logging_level log_level, const bt_value **user_result)
-{
- const bt_value *input_type_value;
- const char *input_type;
- bt_component_class_query_method_status status;
- bt_value_map_insert_entry_status insert_entry_status;
- double weight = 0;
- gchar *metadata_path = NULL;
- bt_value *result = NULL;
- struct ctf_metadata_decoder *metadata_decoder = NULL;
- FILE *metadata_file = NULL;
- char uuid_str[BT_UUID_STR_LEN + 1];
- bool has_uuid = false;
- const bt_value *input_value;
- const char *input;
-
- input_type_value = bt_value_map_borrow_entry_value_const(params, "type");
- BT_ASSERT(input_type_value);
- BT_ASSERT(bt_value_get_type(input_type_value) == BT_VALUE_TYPE_STRING);
- input_type = bt_value_string_get(input_type_value);
+ const auto type = params["type"]->asString().value();
- if (strcmp(input_type, "directory") != 0) {
- goto create_result;
+ if (strcmp(type, "directory") != 0) {
+ /*
+ * The input type is not a directory so we are 100% sure it's not a CTF
+ * 1.8 trace as it would need a directory with at least 1 metadata file
+ * and 1 data stream file.
+ */
+ const auto result = bt2::MapValue::create();
+ result->insert("weight", 0.0f);
+ return result;
}
- input_value = bt_value_map_borrow_entry_value_const(params, "input");
- BT_ASSERT(input_value);
- BT_ASSERT(bt_value_get_type(input_value) == BT_VALUE_TYPE_STRING);
- input = bt_value_string_get(input_value);
+ const auto input = params["input"]->asString().value();
- metadata_path = g_build_filename(input, CTF_FS_METADATA_FILENAME, NULL);
- if (!metadata_path) {
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_MEMORY_ERROR;
- goto end;
+ bt2c::GCharUP metadataPath {g_build_filename(input, CTF_FS_METADATA_FILENAME, NULL)};
+ if (!metadataPath) {
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(logger, bt2::Error, "Failed to read parameters");
}
- metadata_file = g_fopen(metadata_path, "rb");
- if (metadata_file) {
+ double weight = 0;
+ char uuid_str[BT_UUID_STR_LEN + 1];
+ bool has_uuid = false;
+ bt2c::FileUP metadataFile {g_fopen(metadataPath.get(), "rb")};
+ if (metadataFile) {
enum ctf_metadata_decoder_status decoder_status;
bt_uuid_t uuid;
- ctf_metadata_decoder_config metadata_decoder_config;
- metadata_decoder_config.log_level = log_level;
- metadata_decoder_config.self_comp_class =
- bt_self_component_class_source_as_self_component_class(comp_class);
+ ctf_metadata_decoder_config metadata_decoder_config {logger};
- metadata_decoder = ctf_metadata_decoder_create(&metadata_decoder_config);
+ ctf_metadata_decoder_up metadata_decoder =
+ ctf_metadata_decoder_create(&metadata_decoder_config);
if (!metadata_decoder) {
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_ERROR;
- goto end;
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(logger, bt2::Error,
+ "Failed to create metadata decoder");
}
- decoder_status = ctf_metadata_decoder_append_content(metadata_decoder, metadata_file);
+ decoder_status =
+ ctf_metadata_decoder_append_content(metadata_decoder.get(), metadataFile.get());
if (decoder_status != CTF_METADATA_DECODER_STATUS_OK) {
- BT_LOGW("cannot append metadata content: metadata-decoder-status=%d", decoder_status);
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_ERROR;
- goto end;
+ BT_CPPLOGE_APPEND_CAUSE_AND_THROW_SPEC(
+ logger, bt2::Error, "Failed to append metadata content: metadata-decoder-status={}",
+ decoder_status);
}
/*
weight = 0.75;
/* If the trace has a UUID, return the stringified UUID as the group. */
- if (ctf_metadata_decoder_get_trace_class_uuid(metadata_decoder, uuid) == 0) {
+ if (ctf_metadata_decoder_get_trace_class_uuid(metadata_decoder.get(), uuid) == 0) {
bt_uuid_to_str(uuid, uuid_str);
has_uuid = true;
}
}
-create_result:
- result = bt_value_map_create();
- if (!result) {
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_MEMORY_ERROR;
- goto end;
- }
-
- insert_entry_status = bt_value_map_insert_real_entry(result, "weight", weight);
- if (insert_entry_status != BT_VALUE_MAP_INSERT_ENTRY_STATUS_OK) {
- status = (bt_component_class_query_method_status) insert_entry_status;
- goto end;
- }
+ const auto result = bt2::MapValue::create();
+ result->insert("weight", weight);
/* We are not supposed to have weight == 0 and a UUID. */
BT_ASSERT(weight > 0 || !has_uuid);
if (weight > 0 && has_uuid) {
- insert_entry_status = bt_value_map_insert_string_entry(result, "group", uuid_str);
- if (insert_entry_status != BT_VALUE_MAP_INSERT_ENTRY_STATUS_OK) {
- status = (bt_component_class_query_method_status) insert_entry_status;
- goto end;
- }
+ result->insert("group", uuid_str);
}
- *user_result = result;
- result = NULL;
- status = BT_COMPONENT_CLASS_QUERY_METHOD_STATUS_OK;
-
-end:
- g_free(metadata_path);
- bt_value_put_ref(result);
- ctf_metadata_decoder_destroy(metadata_decoder);
-
- return status;
+ return result;
}