Fix: tests: incorrect uri index
authorMathieu Desnoyers <mathieu.desnoyers@efficios.com>
Tue, 17 May 2016 13:20:17 +0000 (09:20 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Wed, 18 May 2016 16:15:27 +0000 (12:15 -0400)
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 <mathieu.desnoyers@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
tests/unit/test_uri.c

index cd95753960006eb301ce440c88a0a042d557d607..61e89e120182833d0cfb038a71d9ccade6d16ae6 100644 (file)
@@ -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);
This page took 0.035317 seconds and 4 git commands to generate.