From c91e0825251b4f57e8ce1d9d4673bee6e279e0cd Mon Sep 17 00:00:00 2001 From: Joseph Myers Date: Thu, 20 Jun 2013 19:10:44 +0000 Subject: [PATCH] Avoid spurious failures from fallback functions (bug 15654). --- ChangeLog | 16 ++++++++++++++++ NEWS | 2 +- math/fedisblxcpt.c | 4 ++-- math/feenablxcpt.c | 7 +++++-- math/fegetenv.c | 4 ++-- math/fegetexcept.c | 4 ++-- math/fegetround.c | 4 ++++ math/feholdexcpt.c | 3 ++- math/fesetenv.c | 4 ++-- math/fesetround.c | 4 ++++ math/feupdateenv.c | 4 ++-- math/fgetexcptflg.c | 4 ++-- 12 files changed, 44 insertions(+), 16 deletions(-) diff --git a/ChangeLog b/ChangeLog index 5fa4885..67ccca5 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,19 @@ +2013-06-20 Joseph Myers + + [BZ #15654] + * math/fedisblxcpt.c (fedisableexcept): Return 0. + * math/feenablxcpt.c (feenableexcept): Return 0 for argument 0. + * math/fegetenv.c (__fegetenv): Return 0. + * math/fegetexcept.c (fegetexcept): Return 0. + * math/fegetround.c (fegetround) [FE_TONEAREST]: Return + FE_TONEAREST. + * math/feholdexcpt.c (feholdexcept): Return 0. + * math/fesetenv.c (__fesetenv): Return 0. + * math/fesetround.c (fesetround) [FE_TONEAREST]: Return 0 for + argument FE_TONEAREST. + * math/feupdateenv.c (__feupdateenv): Return 0. + * math/fgetexcptflg.c (__fegetexceptflag): Return 0. + 2013-06-18 Roland McGrath * elf/rtld-Rules (rtld-compile-command.S): New variable. diff --git a/NEWS b/NEWS index be21e29..f8aa622 100644 --- a/NEWS +++ b/NEWS @@ -20,7 +20,7 @@ Version 2.18 15361, 15366, 15380, 15381, 15394, 15395, 15405, 15406, 15409, 15416, 15418, 15419, 15423, 15424, 15426, 15429, 15431, 15432, 15441, 15442, 15448, 15465, 15480, 15485, 15488, 15490, 15492, 15493, 15497, 15506, - 15529, 15536, 15553, 15577, 15583, 15618, 15627, 15631. + 15529, 15536, 15553, 15577, 15583, 15618, 15627, 15631, 15654. * CVE-2013-0242 Buffer overrun in regexp matcher has been fixed (Bugzilla #15078). diff --git a/math/fedisblxcpt.c b/math/fedisblxcpt.c index 9b6ef27..9b8374a 100644 --- a/math/fedisblxcpt.c +++ b/math/fedisblxcpt.c @@ -22,7 +22,7 @@ int fedisableexcept (int excepts) { - /* Signal failure. */ - return -1; + /* All exception traps are disabled. */ + return 0; } stub_warning (fedisableexcept) diff --git a/math/feenablxcpt.c b/math/feenablxcpt.c index d8620a0..e57d4e3 100644 --- a/math/feenablxcpt.c +++ b/math/feenablxcpt.c @@ -22,7 +22,10 @@ int feenableexcept (int excepts) { - /* Signal failure. */ - return -1; + /* Signal failure if any exception traps are to be enabled. */ + if (excepts != 0) + return -1; + else + return 0; } stub_warning (feenableexcept) diff --git a/math/fegetenv.c b/math/fegetenv.c index 14bb801..29229dc 100644 --- a/math/fegetenv.c +++ b/math/fegetenv.c @@ -23,8 +23,8 @@ int __fegetenv (fenv_t *envp) { - /* This always fails. */ - return 1; + /* Nothing to do. */ + return 0; } #if SHLIB_COMPAT (libm, GLIBC_2_1, GLIBC_2_2) strong_alias (__fegetenv, __old_fegetenv) diff --git a/math/fegetexcept.c b/math/fegetexcept.c index 42aaf93..9075160 100644 --- a/math/fegetexcept.c +++ b/math/fegetexcept.c @@ -22,7 +22,7 @@ int fegetexcept (void) { - /* Signal failure. */ - return -1; + /* All exception traps are disabled. */ + return 0; } stub_warning (fegetexcept) diff --git a/math/fegetround.c b/math/fegetround.c index deb3c5d..24bbd16 100644 --- a/math/fegetround.c +++ b/math/fegetround.c @@ -22,6 +22,10 @@ int fegetround (void) { +#ifdef FE_TONEAREST + return FE_TONEAREST; +#else return 0; +#endif } stub_warning (fegetround) diff --git a/math/feholdexcpt.c b/math/feholdexcpt.c index b4e3260..c830afb 100644 --- a/math/feholdexcpt.c +++ b/math/feholdexcpt.c @@ -22,7 +22,8 @@ int feholdexcept (fenv_t *envp) { - return 1; /* Signal failure. */ + /* No exception traps to disable and no state to save. */ + return 0; } libm_hidden_def (feholdexcept) stub_warning (feholdexcept) diff --git a/math/fesetenv.c b/math/fesetenv.c index 058700f..7361958 100644 --- a/math/fesetenv.c +++ b/math/fesetenv.c @@ -23,8 +23,8 @@ int __fesetenv (const fenv_t *envp) { - /* This always fails. */ - return 1; + /* Nothing to do. */ + return 0; } #if SHLIB_COMPAT (libm, GLIBC_2_1, GLIBC_2_2) strong_alias (__fesetenv, __old_fesetenv) diff --git a/math/fesetround.c b/math/fesetround.c index 0e06217..1895a20 100644 --- a/math/fesetround.c +++ b/math/fesetround.c @@ -22,7 +22,11 @@ int fesetround (int round) { +#ifdef FE_TONEAREST + return (round == FE_TONEAREST) ? 0 : 1; +#else return 1; /* Signal we are unable to set the direction. */ +#endif } libm_hidden_def (fesetround) stub_warning (fesetround) diff --git a/math/feupdateenv.c b/math/feupdateenv.c index e0350b3..5a39521 100644 --- a/math/feupdateenv.c +++ b/math/feupdateenv.c @@ -23,8 +23,8 @@ int __feupdateenv (const fenv_t *envp) { - /* This always fails. */ - return 1; + /* Nothing to do. */ + return 0; } #if SHLIB_COMPAT (libm, GLIBC_2_1, GLIBC_2_2) strong_alias (__feupdateenv, __old_feupdateenv) diff --git a/math/fgetexcptflg.c b/math/fgetexcptflg.c index 1512864..764be42 100644 --- a/math/fgetexcptflg.c +++ b/math/fgetexcptflg.c @@ -23,8 +23,8 @@ int __fegetexceptflag (fexcept_t *flagp, int excepts) { - /* This always fails. */ - return 1; + /* Nothing to do. */ + return 0; } #if SHLIB_COMPAT (libm, GLIBC_2_1, GLIBC_2_2) strong_alias (__fegetexceptflag, __old_fegetexceptflag) -- 2.7.4