diff options
author | Michal Zidek <mzidek@redhat.com> | 2013-08-28 12:46:58 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-02-05 14:41:38 +0100 |
commit | d8057ec487e452038f0106042021fa612bbb8555 (patch) | |
tree | 18a64c53087b928ef0bbb03820077be40117ab6e /src/responder/common | |
parent | 8263ecef6f5c6dab802fdabaf57c25fd2fcf692e (diff) | |
download | sssd-d8057ec487e452038f0106042021fa612bbb8555.tar.gz sssd-d8057ec487e452038f0106042021fa612bbb8555.tar.xz sssd-d8057ec487e452038f0106042021fa612bbb8555.zip |
responder: Use SAFEALIGN macros where appropriate.
https://fedorahosted.org/sssd/ticket/1359
Diffstat (limited to 'src/responder/common')
-rw-r--r-- | src/responder/common/responder_cmd.c | 18 |
1 files changed, 13 insertions, 5 deletions
diff --git a/src/responder/common/responder_cmd.c b/src/responder/common/responder_cmd.c index 3a3fca9b0..b7483d640 100644 --- a/src/responder/common/responder_cmd.c +++ b/src/responder/common/responder_cmd.c @@ -51,8 +51,12 @@ int sss_cmd_empty_packet(struct sss_packet *packet) if (ret != EOK) return ret; sss_packet_get_body(packet, &body, &blen); - ((uint32_t *)body)[0] = 0; /* num results */ - ((uint32_t *)body)[1] = 0; /* reserved */ + + /* num results */ + SAFEALIGN_SETMEM_UINT32(body, 0, NULL); + + /* reserved */ + SAFEALIGN_SETMEM_UINT32(body + sizeof(uint32_t), 0, NULL); return EOK; } @@ -97,6 +101,7 @@ int sss_cmd_get_version(struct cli_ctx *cctx) size_t blen; int ret; uint32_t client_version; + uint32_t protocol_version; int i; static struct cli_protocol_version *cli_protocol_version = NULL; @@ -133,9 +138,12 @@ int sss_cmd_get_version(struct cli_ctx *cctx) return ret; } sss_packet_get_body(cctx->creq->out, &body, &blen); - ((uint32_t *)body)[0] = cctx->cli_protocol_version!=NULL ? - cctx->cli_protocol_version->version : 0; - DEBUG(5, ("Offered version [%d].\n", ((uint32_t *)body)[0])); + + protocol_version = (cctx->cli_protocol_version != NULL) + ? cctx->cli_protocol_version->version : 0; + + SAFEALIGN_COPY_UINT32(body, &protocol_version, NULL); + DEBUG(SSSDBG_FUNC_DATA, ("Offered version [%d].\n", protocol_version)); sss_cmd_done(cctx, NULL); return EOK; |