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/nss_netgroup.c | |
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/nss_netgroup.c')
-rw-r--r-- | src/sss_client/nss_netgroup.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/src/sss_client/nss_netgroup.c b/src/sss_client/nss_netgroup.c index f72d547e8..8594fc460 100644 --- a/src/sss_client/nss_netgroup.c +++ b/src/sss_client/nss_netgroup.c @@ -160,6 +160,7 @@ enum nss_status _nss_sss_setnetgrent(const char *netgroup, { uint8_t *repbuf = NULL; size_t replen; + uint32_t num_results; enum nss_status nret; struct sss_cli_req_data rd; int errnop; @@ -198,8 +199,11 @@ enum nss_status _nss_sss_setnetgrent(const char *netgroup, goto out; } + /* Get number of results from repbuf */ + SAFEALIGN_COPY_UINT32(&num_results, repbuf, NULL); + /* no results if not found */ - if ((((uint32_t *)repbuf)[0] == 0) || (replen < NETGR_METADATA_COUNT)) { + if ((num_results == 0) || (replen < NETGR_METADATA_COUNT)) { free(repbuf); nret = NSS_STATUS_NOTFOUND; goto out; @@ -221,6 +225,7 @@ static enum nss_status internal_getnetgrent_r(struct __netgrent *result, struct sss_nss_netgr_rep netgrrep; uint8_t *repbuf; size_t replen; + uint32_t num_results; enum nss_status nret; uint32_t num_entries; int ret; @@ -266,8 +271,11 @@ static enum nss_status internal_getnetgrent_r(struct __netgrent *result, return nret; } + /* Get number of results from repbuf. */ + SAFEALIGN_COPY_UINT32(&num_results, repbuf, NULL); + /* no results if not found */ - if ((((uint32_t *)repbuf)[0] == 0) || (replen <= NETGR_METADATA_COUNT)) { + if ((num_results == 0) || (replen <= NETGR_METADATA_COUNT)) { free(repbuf); return NSS_STATUS_RETURN; } |