diff options
author | Mark Dickinson <dickinsm@gmail.com> | 2010-06-13 10:52:38 (GMT) |
---|---|---|
committer | Mark Dickinson <dickinsm@gmail.com> | 2010-06-13 10:52:38 (GMT) |
commit | bcdf9da26581266a35a2c7adc835c4580177c1ec (patch) | |
tree | 8223c5a8f1faa149c69305f68e97c463c1369a19 /Modules/mathmodule.c | |
parent | c6f1396be0a09d44de4ea8415c72c947521229d7 (diff) | |
download | cpython-bcdf9da26581266a35a2c7adc835c4580177c1ec.zip cpython-bcdf9da26581266a35a2c7adc835c4580177c1ec.tar.gz cpython-bcdf9da26581266a35a2c7adc835c4580177c1ec.tar.bz2 |
Merged revisions 81967 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk
........
r81967 | mark.dickinson | 2010-06-13 11:50:29 +0100 (Sun, 13 Jun 2010) | 4 lines
Issue #8986: erfc was raising OverflowError on Linux for arguments in
the (approximate) range (-27.3, 30.0), as a result of an escaped errno
value.
........
Diffstat (limited to 'Modules/mathmodule.c')
-rw-r--r-- | Modules/mathmodule.c | 22 |
1 files changed, 16 insertions, 6 deletions
diff --git a/Modules/mathmodule.c b/Modules/mathmodule.c index d57ad90..bc3f5dd 100644 --- a/Modules/mathmodule.c +++ b/Modules/mathmodule.c @@ -428,8 +428,8 @@ m_lgamma(double x) static double m_erf_series(double x) { - double x2, acc, fk; - int i; + double x2, acc, fk, result; + int i, saved_errno; x2 = x * x; acc = 0.0; @@ -438,7 +438,12 @@ m_erf_series(double x) acc = 2.0 + x2 * acc / fk; fk -= 1.0; } - return acc * x * exp(-x2) / sqrtpi; + /* Make sure the exp call doesn't affect errno; + see m_erfc_contfrac for more. */ + saved_errno = errno; + result = acc * x * exp(-x2) / sqrtpi; + errno = saved_errno; + return result; } /* @@ -453,8 +458,8 @@ m_erf_series(double x) static double m_erfc_contfrac(double x) { - double x2, a, da, p, p_last, q, q_last, b; - int i; + double x2, a, da, p, p_last, q, q_last, b, result; + int i, saved_errno; if (x >= ERFC_CONTFRAC_CUTOFF) return 0.0; @@ -472,7 +477,12 @@ m_erfc_contfrac(double x) temp = p; p = b*p - a*p_last; p_last = temp; temp = q; q = b*q - a*q_last; q_last = temp; } - return p / q * x * exp(-x2) / sqrtpi; + /* Issue #8986: On some platforms, exp sets errno on underflow to zero; + save the current errno value so that we can restore it later. */ + saved_errno = errno; + result = p / q * x * exp(-x2) / sqrtpi; + errno = saved_errno; + return result; } /* Error function erf(x), for general x */ |