diff options
author | Brandon Bergren <bdragon@FreeBSD.org> | 2021-03-01 03:06:59 +0000 |
---|---|---|
committer | Brandon Bergren <bdragon@FreeBSD.org> | 2021-03-01 03:11:29 +0000 |
commit | 384ee7cc6e9e4ddc91a6e9e623fcbbe5826bce38 (patch) | |
tree | 1d64354b479d0e4348720f29147378c48fe38986 /lib/libc/powerpc | |
parent | dd95b39235dd81c890aa3cce02a5bb7f91f23803 (diff) | |
download | src-384ee7cc6e9e4ddc91a6e9e623fcbbe5826bce38.tar.gz src-384ee7cc6e9e4ddc91a6e9e623fcbbe5826bce38.zip |
[PowerPC] [PowerPCSPE] Fix multiple issues in fpsetmask().
Building R on powerpc64 exposed a problem in fpsetmask() whereby we
were not properly clamping the provided mask to the valid range.
This same issue affects powerpc and powerpcspe.
Properly limit the range of bits that can be set via fpsetmask().
While here, use the correct fp_except_t type instead of fp_rnd_t.
Reported by: pkubaj, jhibbits (in IRC)
Sponsored by: Tag1 Consulting, Inc.
MFC after: 1 week
Diffstat (limited to 'lib/libc/powerpc')
-rw-r--r-- | lib/libc/powerpc/gen/fpsetmask.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/libc/powerpc/gen/fpsetmask.c b/lib/libc/powerpc/gen/fpsetmask.c index 4d63552470be..f5d52eec5482 100644 --- a/lib/libc/powerpc/gen/fpsetmask.c +++ b/lib/libc/powerpc/gen/fpsetmask.c @@ -43,11 +43,11 @@ fp_except_t fpsetmask(fp_except_t mask) { u_int64_t fpscr; - fp_rnd_t old; + fp_except_t old; __asm__("mffs %0" : "=f"(fpscr)); - old = (fp_rnd_t)((fpscr >> 3) & 0x1f); - fpscr = (fpscr & 0xffffff07) | (mask << 3); + old = (fp_except_t)((fpscr >> 3) & 0x1f); + fpscr = (fpscr & 0xffffff07) | ((mask & 0x1f) << 3); __asm__ __volatile("mtfsf 0xff,%0" :: "f"(fpscr)); return (old); } |