OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-4075-gd8f75d3
Gerrit Code Review
gerrit@openafs.org
Wed, 18 Dec 2013 14:44:45 -0800 (PST)
The following commit has been merged in the master branch:
commit d8f75d3206eaa56b3a819a5bc13a4bf3a9130512
Author: Andrew Deason <adeason@sinenomine.net>
Date: Tue Dec 10 17:02:34 2013 -0600
cellconfig: Do not use 'long' for dbserver IPs
A few places in this file assume that our dbserver IP addresses are
"long"s. A long int can be 8 bytes on some platforms, but we know
these IP addresses are all 4-byte integers. In the rare instances
where we have the maximum number of dbservers, this can overwrite a
bit of extra memory. This can also result in a misaligned access on
platforms such as SPARC v9, since the elements of he->h_addr_list are
not guaranteed to be 8-byte aligned.
So instead, treat these as 4-byte integers. For copying out of
he->h_addr_list, also use a memcpy anyway to be safe, since we are not
guaranteed alignment.
Change-Id: I1afd6e49df32693f86392cb39ce8d7477422aa94
Reviewed-on: http://gerrit.openafs.org/10599
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
src/auth/cellconfig.c | 9 ++++++---
1 files changed, 6 insertions(+), 3 deletions(-)
--
OpenAFS Master Repository