[PATCH 1/6] dlls/msvcrt*: ensure variable sse2_cw is set for all code paths in _control87 (GCC11)

[Zebediah Figura (she/her)]

On 9/28/21 11:49, Eric Pouech wrote:
> Signed-off-by: Eric Pouech <eric.pouech at 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()?