On 07/26/2016 11:02 AM, Lukas Slebodnik wrote:
On (26/07/16 10:40), Petr Cech wrote:
Hello list,
there is simple patch attached.
Please, see subject for more info.
Regards
--
Petr^4 Čech
From da1f3c3835e47f46e9c1fba377c223b78aea9939 Mon Sep 17 00:00:00 2001
From: Petr Cech <[email protected]>
Date: Tue, 26 Jul 2016 10:37:19 +0200
Subject: [PATCH] TESTS: Fixing assignment discards 'const' qualifier
---
src/tests/cmocka/test_utils.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/tests/cmocka/test_utils.c b/src/tests/cmocka/test_utils.c
index
4c72a59437105e683cec2d39c36951aeff63767b..41344b5bbb6dcf4dcb19267bc3478ebbc836b8c8
100644
--- a/src/tests/cmocka/test_utils.c
+++ b/src/tests/cmocka/test_utils.c
@@ -1806,13 +1806,13 @@ static void test_sss_get_domain_mappings_content(void
**state)
assert_non_null(c);
c->forest_root = find_domain_by_name(dom, "subdom1.dom", true);
assert_non_null(c->forest_root);
- c->forest = "subdom1.dom";
+ c->forest = discard_const("subdom1.dom");
c = find_domain_by_name(dom, "subdom3.dom", true);
assert_non_null(c);
c->forest_root = find_domain_by_name(dom, "subdom1.dom", true);
assert_non_null(c->forest_root);
- c->forest = "subdom1.dom";
+ c->forest = discard_const("subdom1.dom");
Could you use discard_const_p ?
LS
Thanks, Lukas.
Fix attached.
Regards
--
Petr^4 Čech
>From 4891cfd63e773e86c1fccd2decee0f6f49433954 Mon Sep 17 00:00:00 2001
From: Petr Cech <[email protected]>
Date: Tue, 26 Jul 2016 10:37:19 +0200
Subject: [PATCH] TESTS: Fixing assignment discards 'const' qualifier
---
src/tests/cmocka/test_utils.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/tests/cmocka/test_utils.c b/src/tests/cmocka/test_utils.c
index 4c72a59437105e683cec2d39c36951aeff63767b..5349accc5c6b55f7d725bfdeaf318b90289880dd 100644
--- a/src/tests/cmocka/test_utils.c
+++ b/src/tests/cmocka/test_utils.c
@@ -1806,13 +1806,13 @@ static void test_sss_get_domain_mappings_content(void **state)
assert_non_null(c);
c->forest_root = find_domain_by_name(dom, "subdom1.dom", true);
assert_non_null(c->forest_root);
- c->forest = "subdom1.dom";
+ c->forest = discard_const_p(char, "subdom1.dom");
c = find_domain_by_name(dom, "subdom3.dom", true);
assert_non_null(c);
c->forest_root = find_domain_by_name(dom, "subdom1.dom", true);
assert_non_null(c->forest_root);
- c->forest = "subdom1.dom";
+ c->forest = discard_const_p(char, "subdom1.dom");
ret = sss_get_domain_mappings_content(test_ctx, dom, &content);
assert_int_equal(ret, EOK);
--
2.7.4
_______________________________________________
sssd-devel mailing list
[email protected]
https://lists.fedorahosted.org/admin/lists/[email protected]
