diff options
author | Stef Walter <stefw@redhat.com> | 2014-01-10 08:08:58 +0100 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-01-10 15:19:42 +0100 |
commit | 4b8021779e4db2a212a8214c17e778e843ae2b3a (patch) | |
tree | 444d1d8ef3cf7d3049696ba5a200881f6c71356a /src | |
parent | ef4d56c1ace330f46f408a7f7d203f5364fefcfc (diff) | |
download | sssd-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
Diffstat (limited to 'src')
-rw-r--r-- | src/tests/cmocka/test_authtok.c | 12 | ||||
-rw-r--r-- | src/util/authtok.c | 2 | ||||
-rw-r--r-- | src/util/authtok.h | 4 |
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 |