summaryrefslogtreecommitdiffstats
path: root/lib
diff options
context:
space:
mode:
authorpfg <pfg@FreeBSD.org>2015-03-23 20:00:35 +0000
committerpfg <pfg@FreeBSD.org>2015-03-23 20:00:35 +0000
commitd201e39fd34bb07644b00c9902c809c1d06618d3 (patch)
tree1018995d7abea6fd42eceeb1450ad75a32d753c7 /lib
parentb56d80f032bbae09a3aa86d5b67d88cd6051f8a2 (diff)
downloadFreeBSD-src-d201e39fd34bb07644b00c9902c809c1d06618d3.zip
FreeBSD-src-d201e39fd34bb07644b00c9902c809c1d06618d3.tar.gz
MFC r277948:
msun: use previously ignored "in" value. This fixes evaluation of exceptional values in scalblnl(). While here, simplify the code as suggested by Bruce Evans. Reported by: clang static analyzer
Diffstat (limited to 'lib')
-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