X-Git-Url: http://git.lttng.org./?a=blobdiff_plain;f=src%2Fcommon%2Factions%2Fpath.c;h=9e1b448a6e20a9a6a6ac48ea1965c608bbc82900;hb=a0377dfefe40662ba7d68617bce6ff467114136c;hp=dbae34b9eb44f2e8609c6adc6b7e4d8cb3e9999a;hpb=27993cc2e9edb3f5a59bc2c16439e063462235e5;p=lttng-tools.git diff --git a/src/common/actions/path.c b/src/common/actions/path.c index dbae34b9e..9e1b448a6 100644 --- a/src/common/actions/path.c +++ b/src/common/actions/path.c @@ -29,10 +29,6 @@ struct lttng_action_path *lttng_action_path_create( } lttng_dynamic_array_init(&path->indexes, sizeof(uint64_t), NULL); - ret = lttng_dynamic_array_set_count(&path->indexes, index_count); - if (ret) { - goto error; - } for (i = 0; i < index_count; i++) { ret = lttng_dynamic_array_add_element( @@ -106,17 +102,12 @@ int lttng_action_path_copy(const struct lttng_action_path *src, int ret; size_t i, src_count; - assert(src); - assert(dst); + LTTNG_ASSERT(src); + LTTNG_ASSERT(dst); lttng_dynamic_array_init(&dst->indexes, sizeof(uint64_t), NULL); src_count = lttng_dynamic_array_get_count(&src->indexes); - ret = lttng_dynamic_array_set_count(&dst->indexes, src_count); - if (ret) { - goto error; - } - for (i = 0; i < src_count; i++) { const void *index = lttng_dynamic_array_get_element( &src->indexes, i); @@ -152,6 +143,13 @@ ssize_t lttng_action_path_create_from_payload( header = (typeof(header)) header_view.buffer.data; consumed_size += header_view.buffer.size; + + /* + * An action path of size 0 can exist and represents a trigger with a + * single non-list action. Handle it differently since a payload view of + * size 0 is considered invalid. + */ + if (header->index_count != 0) { const struct lttng_payload_view indexes_view = lttng_payload_view_from_view(view, @@ -170,9 +168,15 @@ ssize_t lttng_action_path_create_from_payload( if (!action_path) { goto end; } + } else { + action_path = lttng_action_path_create(NULL, 0); + if (!action_path) { + goto end; + } } ret = consumed_size; + *_action_path = action_path; end: return ret; }