summaryrefslogtreecommitdiffstats
path: root/lib/msun
diff options
context:
space:
mode:
authorpfg <pfg@FreeBSD.org>2015-01-30 19:19:03 +0000
committerpfg <pfg@FreeBSD.org>2015-01-30 19:19:03 +0000
commit5c68a1d8dbde6d5e483c55495e78aa159a72b14f (patch)
treecb4ec86c8397bab4038ce81850fc4948acdf9074 /lib/msun
parentff569dac966a5784535fe9a751ac89966c293581 (diff)
downloadFreeBSD-src-5c68a1d8dbde6d5e483c55495e78aa159a72b14f.zip
FreeBSD-src-5c68a1d8dbde6d5e483c55495e78aa159a72b14f.tar.gz
msun: use previously ignored value.
This fixes evaluation of exceptional values in scalblnl(). While here, simplify the code as suggested by Bruce Evans. Reported by: clang static analyzer MFC after: 1 week
Diffstat (limited to 'lib/msun')
-rw-r--r--lib/msun/src/s_scalbln.c26
1 files changed, 4 insertions, 22 deletions
diff --git a/lib/msun/src/s_scalbln.c b/lib/msun/src/s_scalbln.c
index d609d4e..2aa6900 100644
--- a/lib/msun/src/s_scalbln.c
+++ b/lib/msun/src/s_scalbln.c
@@ -35,13 +35,7 @@ scalbln (double x, long n)
{
int in;
- in = (int)n;
- if (in != n) {
- if (n > 0)
- in = INT_MAX;
- else
- in = INT_MIN;
- }
+ in = (n > INT_MAX) ? INT_MAX : (n < INT_MIN) ? INT_MIN : n;
return (scalbn(x, in));
}
@@ -50,13 +44,7 @@ scalblnf (float x, long n)
{
int in;
- in = (int)n;
- if (in != n) {
- if (n > 0)
- in = INT_MAX;
- else
- in = INT_MIN;
- }
+ in = (n > INT_MAX) ? INT_MAX : (n < INT_MIN) ? INT_MIN : n;
return (scalbnf(x, in));
}
@@ -65,12 +53,6 @@ scalblnl (long double x, long n)
{
int in;
- in = (int)n;
- if (in != n) {
- if (n > 0)
- in = INT_MAX;
- else
- in = INT_MIN;
- }
- return (scalbnl(x, (int)n));
+ in = (n > INT_MAX) ? INT_MAX : (n < INT_MIN) ? INT_MIN : n;
+ return (scalbnl(x, in));
}
OpenPOWER on IntegriCloud