diff options
author | Michal Zidek <mzidek@redhat.com> | 2013-08-21 17:17:06 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-12-03 14:41:15 +0100 |
commit | a171d77f40aa92e240e91aa4bafe5a392a98b5a2 (patch) | |
tree | 67af7f48c89fbc1e7daa56c1d0c423cc0e19af74 /src/sss_client/idmap | |
parent | cebdc563a094d305b91da5b5af4d95d8e3a1bf27 (diff) | |
download | sssd-a171d77f40aa92e240e91aa4bafe5a392a98b5a2.tar.gz sssd-a171d77f40aa92e240e91aa4bafe5a392a98b5a2.tar.xz sssd-a171d77f40aa92e240e91aa4bafe5a392a98b5a2.zip |
sss_client: Use SAFEALIGN_COPY_<type> macros where appropriate.
resolves:
https://fedorahosted.org/sssd/ticket/1359
Diffstat (limited to 'src/sss_client/idmap')
-rw-r--r-- | src/sss_client/idmap/sss_nss_idmap.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/sss_client/idmap/sss_nss_idmap.c b/src/sss_client/idmap/sss_nss_idmap.c index e0faf6e78..adb588975 100644 --- a/src/sss_client/idmap/sss_nss_idmap.c +++ b/src/sss_client/idmap/sss_nss_idmap.c @@ -108,7 +108,7 @@ static int sss_nss_getyyybyxxx(union input inp, enum sss_cli_command cmd , goto done; } - num_results = ((uint32_t *)repbuf)[0]; + SAFEALIGN_COPY_UINT32(&num_results, repbuf, NULL); if (num_results == 0) { ret = ENOENT; goto done; @@ -117,7 +117,9 @@ static int sss_nss_getyyybyxxx(union input inp, enum sss_cli_command cmd , goto done; } - out->type = ((uint32_t *)repbuf)[2]; + /* Skip first two 32 bit values (number of results and + * reserved padding) */ + SAFEALIGN_COPY_UINT32(&out->type, repbuf + 2 * sizeof(uint32_t), NULL); data_len = replen - DATA_START; |