Change-Id: I5b5c7c15e99bd9cc79c5e20ea68326a244490ce8
Signed-off-by: Simon Marchi <simon.marchi@efficios.com>
Reviewed-on: https://review.lttng.org/c/babeltrace/+/8449
Reviewed-by: Philippe Proulx <eeppeliteloop@gmail.com>
Reviewed-on: https://review.lttng.org/c/babeltrace/+/12363
Tested-by: jenkins <jenkins@lttng.org>
}
viewer_status = live_viewer_connection_create(
}
viewer_status = live_viewer_connection_create(
- lttng_live->params.url->str, false, lttng_live_msg_iter, lttng_live_msg_iter->logger,
+ lttng_live->params.url.c_str(), false, lttng_live_msg_iter, lttng_live_msg_iter->logger,
<tng_live_msg_iter->viewer_connection);
if (viewer_status != LTTNG_LIVE_VIEWER_STATUS_OK) {
if (viewer_status == LTTNG_LIVE_VIEWER_STATUS_ERROR) {
<tng_live_msg_iter->viewer_connection);
if (viewer_status != LTTNG_LIVE_VIEWER_STATUS_OK) {
if (viewer_status == LTTNG_LIVE_VIEWER_STATUS_ERROR) {
"Unable to connect to the requested live viewer session. "
"Keep trying to connect because of {}=\"{}\" component parameter: url=\"{}\"",
SESS_NOT_FOUND_ACTION_PARAM, SESS_NOT_FOUND_ACTION_CONTINUE_STR,
"Unable to connect to the requested live viewer session. "
"Keep trying to connect because of {}=\"{}\" component parameter: url=\"{}\"",
SESS_NOT_FOUND_ACTION_PARAM, SESS_NOT_FOUND_ACTION_CONTINUE_STR,
- lttng_live->params.url->str);
+ lttng_live->params.url);
break;
case SESSION_NOT_FOUND_ACTION_FAIL:
BT_CPPLOGE_APPEND_CAUSE_SPEC(
break;
case SESSION_NOT_FOUND_ACTION_FAIL:
BT_CPPLOGE_APPEND_CAUSE_SPEC(
"Fail the message iterator initialization because of {}=\"{}\" "
"component parameter: url =\"{}\"",
SESS_NOT_FOUND_ACTION_PARAM, SESS_NOT_FOUND_ACTION_FAIL_STR,
"Fail the message iterator initialization because of {}=\"{}\" "
"component parameter: url =\"{}\"",
SESS_NOT_FOUND_ACTION_PARAM, SESS_NOT_FOUND_ACTION_FAIL_STR,
- lttng_live->params.url->str);
+ lttng_live->params.url);
status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_ERROR;
goto error;
case SESSION_NOT_FOUND_ACTION_END:
status = BT_MESSAGE_ITERATOR_CLASS_INITIALIZE_METHOD_STATUS_ERROR;
goto error;
case SESSION_NOT_FOUND_ACTION_END:
"End gracefully at the first _next() call because of {}=\"{}\""
" component parameter: url=\"{}\"",
SESS_NOT_FOUND_ACTION_PARAM, SESS_NOT_FOUND_ACTION_END_STR,
"End gracefully at the first _next() call because of {}=\"{}\""
" component parameter: url=\"{}\"",
SESS_NOT_FOUND_ACTION_PARAM, SESS_NOT_FOUND_ACTION_END_STR,
- lttng_live->params.url->str);
+ lttng_live->params.url);
break;
default:
bt_common_abort();
break;
default:
bt_common_abort();
static void lttng_live_component_destroy_data(struct lttng_live_component *lttng_live)
{
static void lttng_live_component_destroy_data(struct lttng_live_component *lttng_live)
{
- if (!lttng_live) {
- return;
- }
-
- if (lttng_live->params.url) {
- g_string_free(lttng_live->params.url, TRUE);
- }
-
const bt_value *inputs_value;
const bt_value *url_value;
const bt_value *value;
const bt_value *inputs_value;
const bt_value *url_value;
const bt_value *value;
enum bt_param_validation_status validation_status;
gchar *validation_error = NULL;
bt_component_class_initialize_method_status status;
enum bt_param_validation_status validation_status;
gchar *validation_error = NULL;
bt_component_class_initialize_method_status status;
inputs_value = bt_value_map_borrow_entry_value_const(params, INPUTS_PARAM);
url_value = bt_value_array_borrow_element_by_index_const(inputs_value, 0);
inputs_value = bt_value_map_borrow_entry_value_const(params, INPUTS_PARAM);
url_value = bt_value_array_borrow_element_by_index_const(inputs_value, 0);
- url = bt_value_string_get(url_value);
-
- lttng_live->params.url = g_string_new(url);
- if (!lttng_live->params.url) {
- status = BT_COMPONENT_CLASS_INITIALIZE_METHOD_STATUS_MEMORY_ERROR;
- goto error;
- }
+ lttng_live->params.url = bt_value_string_get(url_value);
value = bt_value_map_borrow_entry_value_const(params, SESS_NOT_FOUND_ACTION_PARAM);
if (value) {
value = bt_value_map_borrow_entry_value_const(params, SESS_NOT_FOUND_ACTION_PARAM);
if (value) {
- GString *url = nullptr;
enum session_not_found_action sess_not_found_act = SESSION_NOT_FOUND_ACTION_CONTINUE;
} params;
enum session_not_found_action sess_not_found_act = SESSION_NOT_FOUND_ACTION_CONTINUE;
} params;