summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStef Walter <stefw@redhat.com>2014-01-10 08:08:58 +0100
committerJakub Hrozek <jhrozek@redhat.com>2014-01-10 15:19:42 +0100
commit4b8021779e4db2a212a8214c17e778e843ae2b3a (patch)
tree444d1d8ef3cf7d3049696ba5a200881f6c71356a
parentef4d56c1ace330f46f408a7f7d203f5364fefcfc (diff)
downloadsssd-4b8021779e4db2a212a8214c17e778e843ae2b3a.tar.gz
sssd-4b8021779e4db2a212a8214c17e778e843ae2b3a.tar.xz
sssd-4b8021779e4db2a212a8214c17e778e843ae2b3a.zip
util: Fix const cast failures when building with -Werror
When building with -Werror 'make check' fails with many errors like: ../src/tests/cmocka/test_authtok.c: In function ‘test_sss_authtok_password’: ../src/tests/cmocka/test_authtok.c:98:48: error: cast discards ‘__attribute__((const))’ qualifier from pointer target type [-Werror=cast-qual] Make sss_authtok_set() @data argument const, and fix its documentation
-rw-r--r--src/tests/cmocka/test_authtok.c12
-rw-r--r--src/util/authtok.c2
-rw-r--r--src/util/authtok.h4
3 files changed, 9 insertions, 9 deletions
diff --git a/src/tests/cmocka/test_authtok.c b/src/tests/cmocka/test_authtok.c
index 7a497937f..c4e394cf8 100644
--- a/src/tests/cmocka/test_authtok.c
+++ b/src/tests/cmocka/test_authtok.c
@@ -95,7 +95,7 @@ static void test_sss_authtok_password(void **state)
len = strlen(data) + 1;
type = SSS_AUTHTOK_TYPE_PASSWORD;
- ret = sss_authtok_set(ts->authtoken, type, (uint8_t *)data, len);
+ ret = sss_authtok_set(ts->authtoken, type, (const uint8_t *)data, len);
assert_int_equal(ret, EOK);
assert_int_equal(type, sss_authtok_get_type(ts->authtoken));
@@ -134,7 +134,7 @@ static void test_sss_authtok_ccfile(void **state)
len = strlen(data) + 1;
type = SSS_AUTHTOK_TYPE_CCFILE;
- ret = sss_authtok_set(ts->authtoken, type, (uint8_t *)data, len);
+ ret = sss_authtok_set(ts->authtoken, type, (const uint8_t *)data, len);
assert_int_equal(ret, EOK);
assert_int_equal(type, sss_authtok_get_type(ts->authtoken));
@@ -158,7 +158,7 @@ static void test_sss_authtok_ccfile(void **state)
assert_int_equal(len - 1, ret_len);
- ret = sss_authtok_set(ts->authtoken, type, (uint8_t *) data, 0);
+ ret = sss_authtok_set(ts->authtoken, type, (const uint8_t *) data, 0);
assert_int_equal(ret, EOK);
assert_int_equal(type, sss_authtok_get_type(ts->authtoken));
@@ -236,7 +236,7 @@ static void test_sss_authtok_wipe_password(void **state)
len = strlen(data) + 1;
type = SSS_AUTHTOK_TYPE_PASSWORD;
- ret = sss_authtok_set(ts->authtoken, type, (uint8_t *)data, len);
+ ret = sss_authtok_set(ts->authtoken, type, (const uint8_t *)data, len);
assert_int_equal(ret, EOK);
@@ -268,14 +268,14 @@ static void test_sss_authtok_copy(void **state)
len = strlen(data) + 1;
type = SSS_AUTHTOK_TYPE_EMPTY;
- ret = sss_authtok_set(ts->authtoken, type, (uint8_t *)data, len);
+ ret = sss_authtok_set(ts->authtoken, type, (const uint8_t *)data, len);
assert_int_equal(ret, EOK);
assert_int_equal(EOK, sss_authtok_copy(ts->authtoken, dest_authtoken));
assert_int_equal(type, sss_authtok_get_type(dest_authtoken));
type = SSS_AUTHTOK_TYPE_PASSWORD;
- ret = sss_authtok_set(ts->authtoken, type, (uint8_t *)data, len);
+ ret = sss_authtok_set(ts->authtoken, type, (const uint8_t *)data, len);
assert_int_equal(ret, EOK);
diff --git a/src/util/authtok.c b/src/util/authtok.c
index 73a4ccf1a..e9c2881bf 100644
--- a/src/util/authtok.c
+++ b/src/util/authtok.c
@@ -171,7 +171,7 @@ errno_t sss_authtok_set_ccfile(struct sss_auth_token *tok,
errno_t sss_authtok_set(struct sss_auth_token *tok,
enum sss_authtok_type type,
- uint8_t *data, size_t len)
+ const uint8_t *data, size_t len)
{
switch (type) {
case SSS_AUTHTOK_TYPE_PASSWORD:
diff --git a/src/util/authtok.h b/src/util/authtok.h
index 8f327d4c0..1f6def4c3 100644
--- a/src/util/authtok.h
+++ b/src/util/authtok.h
@@ -131,7 +131,7 @@ void sss_authtok_set_empty(struct sss_auth_token *tok);
* @param tok A pointer to a sss_auth_token structure to change, also
* used as a memory context to allocate the internal data.
* @param type A valid authtok type
- * @param ccfile A data pointer
+ * @param data A data pointer
* @param len The length of the data
*
* @return EOK on success
@@ -139,7 +139,7 @@ void sss_authtok_set_empty(struct sss_auth_token *tok);
*/
errno_t sss_authtok_set(struct sss_auth_token *tok,
enum sss_authtok_type type,
- uint8_t *data, size_t len);
+ const uint8_t *data, size_t len);
/**
* @brief Copy an auth token from source to destination