From: Jérémie Galarneau Date: Fri, 24 Aug 2018 18:09:52 +0000 (-0400) Subject: Tests: possible NULL dereference in rotation notification test X-Git-Url: http://git.efficios.com/?p=lttng-tools.git;a=commitdiff_plain;h=c6795a46ffb3f62e9a4961a2be1de4a7e2ad7aa6 Tests: possible NULL dereference in rotation notification test Reported-by: Coverity Scan (1395166 Dereference after null check) Signed-off-by: Jérémie Galarneau --- diff --git a/tests/regression/tools/notification/rotation.c b/tests/regression/tools/notification/rotation.c index c9f5de497..fb516f31f 100644 --- a/tests/regression/tools/notification/rotation.c +++ b/tests/regression/tools/notification/rotation.c @@ -322,10 +322,11 @@ int test_notification( location_status = lttng_trace_archive_location_local_get_absolute_path( location, &chunk_path); - ok(location_status == LTTNG_TRACE_ARCHIVE_LOCATION_STATUS_OK, + ok(location_status == LTTNG_TRACE_ARCHIVE_LOCATION_STATUS_OK && chunk_path, "Retrieved path from location returned by the session rotation completed notification"); diag("Chunk available at %s", chunk_path ? chunk_path : "NULL"); - ok(!strncmp(session->output_path, chunk_path, strlen(session->output_path)), + + ok(chunk_path && !strncmp(session->output_path, chunk_path, strlen(session->output_path)), "Returned path from location starts with the output path"); end: