Tests: fix: leak of trigger in trigger listing tests
authorJérémie Galarneau <jeremie.galarneau@efficios.com>
Fri, 15 Apr 2022 03:26:12 +0000 (23:26 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Wed, 18 May 2022 14:23:05 +0000 (10:23 -0400)
==1480456==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 112 byte(s) in 1 object(s) allocated from:
    #0 0x7fdb9260cfb9 in __interceptor_calloc /usr/src/debug/gcc/libsanitizer/asan/asan_malloc_linux.cpp:154
    #1 0x7fdb9242348d in zmalloc_internal ../../src/common/macros.hpp:60
    #2 0x7fdb924295a9 in lttng_trigger* zmalloc<lttng_trigger>() ../../src/common/macros.hpp:89
    #3 0x7fdb92423dbe in lttng_trigger_create /home/jgalar/EfficiOS/src/lttng-tools/src/common/trigger.cpp:58
    #4 0x56304832331f in register_trigger /home/jgalar/EfficiOS/src/lttng-tools/tests/regression/tools/trigger/utils/register-some-triggers.cpp:24
    #5 0x5630483233f1 in register_trigger_action_list_notify /home/jgalar/EfficiOS/src/lttng-tools/tests/regression/tools/trigger/utils/register-some-triggers.cpp:46
    #6 0x5630483239a0 in test_session_rotation_conditions /home/jgalar/EfficiOS/src/lttng-tools/tests/regression/tools/trigger/utils/register-some-triggers.cpp:246
    #7 0x563048323d4d in main /home/jgalar/EfficiOS/src/lttng-tools/tests/regression/tools/trigger/utils/register-some-triggers.cpp:309
    #8 0x7fdb91c6630f in __libc_start_call_main (/usr/lib/libc.so.6+0x2d30f)

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Change-Id: Ie163989a70f65f9c2c4e93c36cc9fc6ba6bdeeb5

tests/regression/tools/trigger/utils/register-some-triggers.c

index f4bba2df4488de26114dabe07eda609048debcee..113cd2caf6db05dfb5f4b310bebfd012379916cd 100644 (file)
@@ -25,6 +25,9 @@ static void register_trigger(const char *trigger_name,
        trigger = lttng_trigger_create(condition, action);
        ret = lttng_register_trigger_with_name(trigger, trigger_name);
        assert(ret == LTTNG_OK);
+       lttng_trigger_destroy(trigger);
+       lttng_condition_destroy(condition);
+       lttng_action_destroy(action);
 }
 
 /*
@@ -43,6 +46,7 @@ static void register_trigger_action_list_notify(
        action_status = lttng_action_list_add_action(
                        action_list, action_notify);
        assert(action_status == LTTNG_ACTION_STATUS_OK);
+       lttng_action_destroy(action_notify);
 
        register_trigger(trigger_name, condition, action_list);
 }
This page took 0.026025 seconds and 4 git commands to generate.