diff options
| author | Tom Lane | 2017-03-26 21:35:35 +0000 |
|---|---|---|
| committer | Tom Lane | 2017-03-26 21:35:35 +0000 |
| commit | 55c642c92b974362ece5ddf100fa2fa90b3442c9 (patch) | |
| tree | 0b26714d17099658cf1eb330e2e63ca91cce3519 /doc/src | |
| parent | 6423ed7d4eaa5f84e508ad6f03afb8039895f562 (diff) | |
Fix unportable disregard of alignment requirements in RADIUS code.
The compiler is entitled to store a char[] local variable with no
particular alignment requirement. Our RADIUS code cavalierly took such
a local variable and cast its address to a struct type that does have
alignment requirements. On an alignment-picky machine this would lead
to bus errors. To fix, declare the local variable honestly, and then
cast its address to char * for use in the I/O calls.
Given the lack of field complaints, there must be very few if any
people affected; but nonetheless this is a clear portability issue,
so back-patch to all supported branches.
Noted while looking at a Coverity complaint in the same code.
Diffstat (limited to 'doc/src')
0 files changed, 0 insertions, 0 deletions
