X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=tests%2Funit%2Ftest_session.c;h=cfbb52d048dd00820e6817a24dbe221a5a165716;hp=a6d76c73df49a96a8300f4ca3834811e9d87c441;hb=a9577b7628a85b064c3f249c3504bad989848b49;hpb=81d029da32ba3c502bdce6ea56b3603eed77e951 diff --git a/tests/unit/test_session.c b/tests/unit/test_session.c index a6d76c73d..cfbb52d04 100644 --- a/tests/unit/test_session.c +++ b/tests/unit/test_session.c @@ -16,7 +16,6 @@ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. */ -#define _GNU_SOURCE #include #include #include @@ -25,39 +24,36 @@ #include #include #include +#include + +#include #include +#include +#include +#include #include #include -#include "utils.h" - #define SESSION1 "test1" -/* This path will NEVER be created in this test */ -#define PATH1 "/tmp/.test-junk-lttng" - #define MAX_SESSIONS 10000 #define RANDOM_STRING_LEN 11 -/* - * String of 263 caracters. NAME_MAX + "OVERFLOW". If OVERFLOW appears in the - * session name, we have a problem. - * - * NAME_MAX = 255 - */ -#define OVERFLOW_SESSION_NAME \ - "abcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcd" \ - "abcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcd" \ - "abcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcd" \ - "abcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabcdabc" \ - "OVERFLOW" +/* Number of TAP tests in this file */ +#define NUM_TESTS 11 +struct health_app *health_sessiond; static struct ltt_session_list *session_list; +static pthread_t ht_cleanup_thread; -/* For lttngerr.h */ +/* For error.h */ int lttng_opt_quiet = 1; int lttng_opt_verbose = 0; +int lttng_opt_mi; + +int ust_consumerd32_fd; +int ust_consumerd64_fd; static const char alphanum[] = "0123456789" @@ -117,8 +113,7 @@ static void empty_session_list(void) struct ltt_session *iter, *tmp; cds_list_for_each_entry_safe(iter, tmp, &session_list->head, list) { - cds_list_del(&iter->list); - free(iter); + session_destroy(iter); } /* Session list must be 0 */ @@ -128,11 +123,11 @@ static void empty_session_list(void) /* * Test creation of 1 session */ -static int create_one_session(char *name, char *path) +static int create_one_session(char *name) { int ret; - ret = session_create(name, path, geteuid(), getegid()); + ret = session_create(name, geteuid(), getegid()); if (ret == LTTNG_OK) { /* Validate */ ret = find_session_name(name); @@ -160,15 +155,14 @@ static int create_one_session(char *name, char *path) static int destroy_one_session(struct ltt_session *session) { int ret; + char session_name[NAME_MAX]; - ret = session_destroy(session); + strncpy(session_name, session->name, sizeof(session_name)); + session_name[sizeof(session_name) - 1] = '\0'; + ret = session_destroy(session); if (ret == LTTNG_OK) { - /* Validate */ - if (session == NULL) { - return 0; - } - ret = find_session_name(session->name); + ret = find_session_name(session_name); if (ret < 0) { /* Success, -1 means that the sesion is NOT found */ return 0; @@ -181,28 +175,6 @@ static int destroy_one_session(struct ltt_session *session) return 0; } -static int fuzzing_create_args(void) -{ - int ret; - - ret = create_one_session(NULL, NULL); - if (ret > 0) { - printf("Session created with (null),(null)\n"); - return -1; - } - - ret = create_one_session(NULL, PATH1); - if (ret > 0) { - printf("Session created with (null), %s)\n", PATH1); - return -1; - } - - /* Session list must be 0 */ - assert(!session_list_count()); - - return 0; -} - /* * This test is supposed to fail at the second create call. If so, return 0 for * test success, else -1. @@ -212,7 +184,7 @@ static int two_session_same_name(void) int ret; struct ltt_session *sess; - ret = create_one_session(SESSION1, PATH1); + ret = create_one_session(SESSION1); if (ret < 0) { /* Fail */ return -1; @@ -228,103 +200,129 @@ static int two_session_same_name(void) return -1; } -int main(int argc, char **argv) +void test_session_list(void) { - int ret, i; - struct ltt_session *iter, *tmp; + session_list = session_get_list(); + ok(session_list != NULL, "Session list: not NULL"); +} - srand(time(NULL)); +void test_create_one_session(void) +{ + ok(create_one_session(SESSION1) == 0, + "Create session: %s", + SESSION1); +} - printf("\nTesting Sessions:\n-----------\n"); +void test_validate_session(void) +{ + struct ltt_session *tmp; - session_list = session_get_list(); - if (session_list == NULL) { - return -1; - } + tmp = session_find_by_name(SESSION1); - printf("Create 1 session %s: ", SESSION1); - fflush(stdout); - ret = create_one_session(SESSION1, PATH1); - if (ret < 0) { - return -1; - } - PRINT_OK(); + ok(tmp != NULL, + "Validating session: session found"); + + ok(tmp->kernel_session == NULL && + strlen(tmp->name), + "Validating session: basic sanity check"); - printf("Validating created session %s: ", SESSION1); - fflush(stdout); - tmp = session_find_by_name(SESSION1); - if (tmp == NULL) { - return -1; - } - /* Basic init session values */ - assert(tmp->kernel_session == NULL); - assert(strlen(tmp->path)); - assert(strlen(tmp->name)); session_lock(tmp); session_unlock(tmp); +} + +void test_destroy_session(void) +{ + struct ltt_session *tmp; - PRINT_OK(); + tmp = session_find_by_name(SESSION1); - printf("Destroy 1 session %s: ", SESSION1); - fflush(stdout); - ret = destroy_one_session(tmp); - if (ret < 0) { - return -1; - } - PRINT_OK(); + ok(tmp != NULL, + "Destroying session: session found"); - printf("Two session with same name: "); - fflush(stdout); - ret = two_session_same_name(); - if (ret < 0) { - return -1; - } - PRINT_OK(); + ok(destroy_one_session(tmp) == 0, + "Destroying session: %s destroyed", + SESSION1); +} - empty_session_list(); +void test_duplicate_session(void) +{ + ok(two_session_same_name() == 0, + "Duplicate session creation"); +} - printf("Fuzzing create_session arguments: "); - fflush(stdout); - ret = fuzzing_create_args(); - if (ret < 0) { - return -1; - } - PRINT_OK(); +void test_bogus_session_param(void) +{ + ok(create_one_session(NULL) < 0, + "Create session with bogus param: NULL should fail"); + + ok(session_list_count() == 0, + "Create session with bogus param: session list empty"); +} + +void test_large_session_number(void) +{ + int ret, i, failed = 0; + struct ltt_session *iter, *tmp; - printf("Creating %d sessions: ", MAX_SESSIONS); - fflush(stdout); for (i = 0; i < MAX_SESSIONS; i++) { char *tmp_name = get_random_string(); - - ret = create_one_session(tmp_name, PATH1); + ret = create_one_session(tmp_name); if (ret < 0) { - printf("session %d (name: %s) creation failed\n", i, tmp_name); - return -1; - } - - if ((i % 1000) == 0) { - fprintf(stdout, "%d..", i); - fflush(stdout); + diag("session %d (name: %s) creation failed", i, tmp_name); + ++failed; } } - PRINT_OK(); - printf("Destroying %d sessions: ", MAX_SESSIONS); - fflush(stdout); + ok(failed == 0, + "Large sessions number: created %u sessions", + MAX_SESSIONS); + + failed = 0; + for (i = 0; i < MAX_SESSIONS; i++) { cds_list_for_each_entry_safe(iter, tmp, &session_list->head, list) { ret = destroy_one_session(iter); if (ret < 0) { - printf("session %d (name: %s) creation failed\n", i, iter->name); - return -1; + diag("session %d destroy failed", i); + ++failed; } } } - PRINT_OK(); - /* Session list must be 0 */ - assert(!session_list_count()); + ok(failed == 0 && session_list_count() == 0, + "Large sessions number: destroyed %u sessions", + MAX_SESSIONS); +} - /* Success */ - return 0; +int main(int argc, char **argv) +{ + plan_tests(NUM_TESTS); + + health_sessiond = health_app_create(NR_HEALTH_SESSIOND_TYPES); + assert(!init_ht_cleanup_thread(&ht_cleanup_thread)); + + diag("Sessions unit tests"); + + rcu_register_thread(); + + test_session_list(); + + test_create_one_session(); + + test_validate_session(); + + test_destroy_session(); + + test_duplicate_session(); + + empty_session_list(); + + test_bogus_session_param(); + + test_large_session_number(); + + rcu_unregister_thread(); + assert(!fini_ht_cleanup_thread(&ht_cleanup_thread)); + + return exit_status(); }