On 9/28/21 11:49, Eric Pouech wrote:

Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

---
  dlls/msvcrt/math.c |    2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/msvcrt/math.c b/dlls/msvcrt/math.c
index 7f59a4d20d4..ad632e70548 100644
--- a/dlls/msvcrt/math.c
+++ b/dlls/msvcrt/math.c
@@ -5643,7 +5643,7 @@ unsigned int CDECL _control87(unsigned int newval, unsigned int mask)
  {
      unsigned int flags = 0;
  #ifdef __i386__
-    unsigned int sse2_cw;
+    unsigned int sse2_cw = 0;
        __control87_2( newval, mask, &flags, &sse2_cw );
 

Wouldn't it be better to check for failure from __control87_2()?

gcc doesn't seem to be smart enough to infer that ss2_cw is always when _control87_2() returns 1

A+