Do not check `NULL` to call free(): free() accepts `NULL`

[babeltrace.git] / tests / plugins / flt.lttng-utils.debug-info / test_bin_info.c

diff --git a/tests/plugins/flt.lttng-utils.debug-info/test_bin_info.c b/tests/plugins/flt.lttng-utils.debug-info/test_bin_info.c
index c55201c116f389841bef0fc0db32d53b4ea1f093..6d29322498fa308114e240c87c2f7c6657df0464 100644 (file)
--- a/tests/plugins/flt.lttng-utils.debug-info/test_bin_info.c
+++ b/tests/plugins/flt.lttng-utils.debug-info/test_bin_info.c
@@ -21,7 +21,7 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
-#define BT_LOG_OUTPUT_LEVEL BT_LOG_WARN
+#define BT_LOG_OUTPUT_LEVEL BT_LOG_WARNING
 #define BT_LOG_TAG "TEST/BIN-INFO"
 #include "logging/log.h"
 
@@ -167,9 +167,7 @@ void subtest_lookup_function_name(struct bin_info *bin, uint64_t addr,
        ret = bin_info_lookup_function_name(bin, SO_INV_ADDR, &_func_name);
        ok(ret == -1 && _func_name == NULL,
           "bin_info_lookup_function_name - fail on invalid addr");
-       if (_func_name) {
-               free(_func_name);
-       }
+       free(_func_name);
 }
 
 static
@@ -234,7 +232,7 @@ void test_bin_info_build_id(const char *bin_info_dir)
        }
 
        bin = bin_info_create(&fdc, bin_path, SO_LOW_ADDR, SO_MEMSZ, true,
-                             data_dir, NULL);
+                             data_dir, NULL, BT_LOG_OUTPUT_LEVEL, NULL);
        ok(bin != NULL, "bin_info_create successful (%s)", bin_path);
 
        /* Test setting invalid build_id */
@@ -288,7 +286,7 @@ void test_bin_info_debug_link(const char *bin_info_dir)
        }
 
        bin = bin_info_create(&fdc, bin_path, SO_LOW_ADDR, SO_MEMSZ, true,
-                             data_dir, NULL);
+               data_dir, NULL, BT_LOG_OUTPUT_LEVEL, NULL);
        ok(bin != NULL, "bin_info_create successful (%s)", bin_path);
 
        /* Test setting debug link */
@@ -338,7 +336,7 @@ void test_bin_info_elf(const char *bin_info_dir)
        }
 
        bin = bin_info_create(&fdc, bin_path, SO_LOW_ADDR, SO_MEMSZ, true,
-                             data_dir, NULL);
+               data_dir, NULL, BT_LOG_OUTPUT_LEVEL, NULL);
        ok(bin != NULL, "bin_info_create successful (%s)", bin_path);
 
        /* Test bin_info_has_address */
@@ -386,7 +384,7 @@ void test_bin_info_bundled(const char *bin_info_dir)
        }
 
        bin = bin_info_create(&fdc, bin_path, SO_LOW_ADDR, SO_MEMSZ, true,
-                             data_dir, NULL);
+               data_dir, NULL, BT_LOG_OUTPUT_LEVEL, NULL);
        ok(bin != NULL, "bin_info_create successful (%s)", bin_path);
 
        /* Test bin_info_has_address */
@@ -439,7 +437,7 @@ int main(int argc, char **argv)
 
        plan_tests(NR_TESTS);
 
-       ret = bin_info_init();
+       ret = bin_info_init(BT_LOG_OUTPUT_LEVEL, NULL);
        ok(ret == 0, "bin_info_init successful");
 
        test_bin_info_elf(opt_debug_info_dir);