X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=tests%2Funit%2Ftest_ust_data.c;h=7996b8e60e7d4e9ce60cea4220e44a357692d6ce;hp=dd00089266ad7770ce4c64e18b1340439f167c9b;hb=8273250bde9f8fa0cc2f6d8ea86aaf4b9fe83fc1;hpb=c7e35b037773dbbfe10178c946ba44feefb226e1 diff --git a/tests/unit/test_ust_data.c b/tests/unit/test_ust_data.c index dd0008926..7996b8e60 100644 --- a/tests/unit/test_ust_data.c +++ b/tests/unit/test_ust_data.c @@ -16,7 +16,6 @@ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. */ -#define _GNU_SOURCE #include #include #include @@ -24,6 +23,7 @@ #include #include #include +#include #include #include @@ -39,7 +39,7 @@ #define RANDOM_STRING_LEN 11 /* Number of TAP tests in this file */ -#define NUM_TESTS 10 +#define NUM_TESTS 16 /* For error.h */ int lttng_opt_quiet = 1; @@ -49,6 +49,9 @@ int lttng_opt_mi; int ust_consumerd32_fd; int ust_consumerd64_fd; +/* Global variable required by sessiond objects being linked-in */ +struct lttng_ht *agent_apps_ht_by_sock; + static const char alphanum[] = "0123456789" "ABCDEFGHIJKLMNOPQRSTUVWXYZ" @@ -82,6 +85,11 @@ static void test_create_one_ust_session(void) usess = trace_ust_create_session(42); ok(usess != NULL, "Create UST session"); + if (!usess) { + skip(1, "UST session is null"); + return; + } + ok(usess->id == 42 && usess->active == 0 && usess->domain_global.channels != NULL && @@ -99,11 +107,16 @@ static void test_create_ust_channel(void) memset(&attr, 0, sizeof(attr)); - strncpy(attr.name, "channel0", 8); - - uchan = trace_ust_create_channel(&attr); + ok(lttng_strncpy(attr.name, "channel0", sizeof(attr.name)) == 0, + "Validate channel name length"); + uchan = trace_ust_create_channel(&attr, LTTNG_DOMAIN_UST); ok(uchan != NULL, "Create UST channel"); + if (!usess) { + skip(1, "UST session is null"); + return; + } + ok(uchan->enabled == 0 && strncmp(uchan->name, "channel0", 8) == 0 && uchan->name[LTTNG_UST_SYM_NAME_LEN - 1] == '\0' && @@ -121,14 +134,21 @@ static void test_create_ust_event(void) struct lttng_event ev; memset(&ev, 0, sizeof(ev)); - strncpy(ev.name, get_random_string(), LTTNG_SYMBOL_NAME_LEN); + ok(lttng_strncpy(ev.name, get_random_string(), + LTTNG_SYMBOL_NAME_LEN) == 0, + "Validate string length"); ev.type = LTTNG_EVENT_TRACEPOINT; ev.loglevel_type = LTTNG_EVENT_LOGLEVEL_ALL; - event = trace_ust_create_event(&ev, NULL, NULL, NULL); + event = trace_ust_create_event(&ev, NULL, NULL, NULL, false); ok(event != NULL, "Create UST event"); + if (!event) { + skip(1, "UST event is null"); + return; + } + ok(event->enabled == 0 && event->attr.instrumentation == LTTNG_UST_TRACEPOINT && strcmp(event->attr.name, ev.name) == 0 && @@ -143,37 +163,77 @@ static void test_create_ust_event_exclusion(void) struct ltt_ust_event *event; struct lttng_event ev; char *name; + char *random_name; struct lttng_event_exclusion *exclusion; + const int exclusion_count = 2; memset(&ev, 0, sizeof(ev)); /* make a wildcarded event name */ name = get_random_string(); name[strlen(name) - 1] = '*'; - strncpy(ev.name, name, LTTNG_SYMBOL_NAME_LEN); + ok(lttng_strncpy(ev.name, name, LTTNG_SYMBOL_NAME_LEN) == 0, + "Validate string length"); ev.type = LTTNG_EVENT_TRACEPOINT; ev.loglevel_type = LTTNG_EVENT_LOGLEVEL_ALL; /* set up an exclusion set */ - exclusion = zmalloc(sizeof(*exclusion) + LTTNG_SYMBOL_NAME_LEN); - exclusion->count = 1; - strncpy((char *)(exclusion->names), get_random_string(), LTTNG_SYMBOL_NAME_LEN); + exclusion = zmalloc(sizeof(*exclusion) + + LTTNG_SYMBOL_NAME_LEN * exclusion_count); + ok(exclusion != NULL, "Create UST exclusion"); + if (!exclusion) { + skip(4, "zmalloc failed"); + goto end; + } - event = trace_ust_create_event(&ev, NULL, NULL, exclusion); + exclusion->count = exclusion_count; + random_name = get_random_string(); + strncpy(LTTNG_EVENT_EXCLUSION_NAME_AT(exclusion, 0), random_name, + LTTNG_SYMBOL_NAME_LEN); + strncpy(LTTNG_EVENT_EXCLUSION_NAME_AT(exclusion, 1), random_name, + LTTNG_SYMBOL_NAME_LEN); - ok(event != NULL, "Create UST event with exclusion"); + event = trace_ust_create_event(&ev, NULL, NULL, exclusion, false); + exclusion = NULL; + + ok(!event, "Create UST event with identical exclusion names fails"); + + exclusion = zmalloc(sizeof(*exclusion) + + LTTNG_SYMBOL_NAME_LEN * exclusion_count); + ok(exclusion != NULL, "Create UST exclusion"); + if (!exclusion) { + skip(2, "zmalloc failed"); + goto end; + } + + exclusion->count = exclusion_count; + strncpy(LTTNG_EVENT_EXCLUSION_NAME_AT(exclusion, 0), + get_random_string(), LTTNG_SYMBOL_NAME_LEN); + strncpy(LTTNG_EVENT_EXCLUSION_NAME_AT(exclusion, 1), + get_random_string(), LTTNG_SYMBOL_NAME_LEN); + + event = trace_ust_create_event(&ev, NULL, NULL, exclusion, false); + ok(event != NULL, "Create UST event with different exclusion names"); + + if (!event) { + skip(1, "UST event with exclusion is null"); + goto end; + } ok(event->enabled == 0 && event->attr.instrumentation == LTTNG_UST_TRACEPOINT && strcmp(event->attr.name, ev.name) == 0 && event->exclusion != NULL && - event->exclusion->count == 1 && - strcmp((char *)(event->exclusion->names), (char *)(exclusion->names)) == 0 && + event->exclusion->count == exclusion_count && + !memcmp(event->exclusion->names, exclusion->names, + LTTNG_SYMBOL_NAME_LEN * exclusion_count) && event->attr.name[LTTNG_UST_SYM_NAME_LEN - 1] == '\0', "Validate UST event and exclusion"); trace_ust_destroy_event(event); +end: + return; } @@ -198,11 +258,15 @@ int main(int argc, char **argv) diag("UST data structures unit test"); + rcu_register_thread(); + test_create_one_ust_session(); test_create_ust_channel(); test_create_ust_event(); test_create_ust_context(); test_create_ust_event_exclusion(); + rcu_unregister_thread(); + return exit_status(); }