From: Mathieu Desnoyers Date: Tue, 17 May 2016 13:20:17 +0000 (-0400) Subject: Fix: tests: incorrect uri index X-Git-Tag: v2.9.0-rc1~199 X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=commitdiff_plain;h=ee059b98e72ba715a13f32b5b4c43158db795c71 Fix: tests: incorrect uri index Initial issue found by Coverity: CID 1322864 (#1 of 1): Constant expression result (CONSTANT_EXPRESSION_RESULT)impossible_and: The "and" condition size == 2L && (uri + 0).dtype == LTTNG_DST_IPV4 && (uri + 0).utype == LTTNG_URI_DST && (uri + 0).stype == 0U && (uri + 0).port == 8989 && strcmp((uri + 0).subdir, "my/test/path") == 0 && strcmp((uri + 0).dst.ipv4, "127.0.0.1") == 0 && (uri + 1).dtype == LTTNG_DST_IPV4 && (uri + 1).utype == LTTNG_URI_DST && (uri + 1).stype == 0U && (uri + 1).port == 4242 && strcmp((uri + 0).subdir, "my/test/path") == 0 && strcmp((uri + 1).dst.ipv4, "127.0.0.1") == 0 can never be true because (uri + 0).subdir cannot be equal to two different values at the same time. Review of the test source file led to further fixes. Signed-off-by: Mathieu Desnoyers Signed-off-by: Jérémie Galarneau --- diff --git a/tests/unit/test_uri.c b/tests/unit/test_uri.c index cd9575396..61e89e120 100644 --- a/tests/unit/test_uri.c +++ b/tests/unit/test_uri.c @@ -75,7 +75,7 @@ void test_uri_parsing(void) uri[1].utype == LTTNG_URI_DST && uri[1].stype == 0 && uri[1].port == 4242 && - strcmp(uri[0].subdir, "my/test/path") == 0 && + strlen(uri[1].subdir) == 0 && strcmp(uri[1].dst.ipv4, "127.0.0.1") == 0, "URI set to net://localhost:8989:4242/my/test/path"); @@ -93,7 +93,7 @@ void test_uri_parsing(void) uri[0].utype == LTTNG_URI_DST && uri[0].stype == 0 && uri[0].port == 8989 && - strlen(uri[1].subdir) == 0 && + strlen(uri[0].subdir) == 0 && strcmp(uri[0].dst.ipv4, "127.0.0.1") == 0 && uri[1].dtype == LTTNG_DST_IPV4 && uri[1].utype == LTTNG_URI_DST && @@ -117,14 +117,14 @@ void test_uri_parsing(void) uri[0].utype == LTTNG_URI_DST && uri[0].stype == 0 && uri[0].port == 8989 && - strlen(uri[1].subdir) == 0 && + strlen(uri[0].subdir) == 0 && strcmp(uri[0].dst.ipv6, "::1") == 0 && uri[1].dtype == LTTNG_DST_IPV6 && uri[1].utype == LTTNG_URI_DST && uri[1].stype == 0 && uri[1].port == 0 && strlen(uri[1].subdir) == 0 && - strcmp(uri[0].dst.ipv6, "::1") == 0, + strcmp(uri[1].dst.ipv6, "::1") == 0, "URI set to net6://[::1]:8989"); if (uri) { @@ -230,7 +230,7 @@ void test_uri_cmp() assert(uri2[0].utype == LTTNG_URI_DST); assert(uri2[0].stype == 0); assert(uri2[0].port == 8989); - assert(strlen(uri2[1].subdir) == 0); + assert(strlen(uri2[0].subdir) == 0); assert(strcmp(uri2[0].dst.ipv4, "127.0.0.1") == 0); assert(uri2[1].dtype == LTTNG_DST_IPV4); assert(uri2[1].utype == LTTNG_URI_DST);