Re: [PATCH v2 0/1] MR4654: Draft: msvcrt: Protect setlocale against concurrent accesses.

[Piotr Caban (＠piotr)]

On Sun Jan  7 16:43:44 2024 +0000, Piotr Caban wrote:
>     /* Make sure that locinfo is not freed when thread locale is updated
> by e.g. stricmp(). */
>     grab_locinfo(locinfo);
>     newlocinfo = create_locinfo(category, locale, locinfo);
>     free_locinfo(locinfo);

After thinking about it more it would be even better to do something like:
```suggestion:-0+0
    /* Make sure that locinfo is not updated by e.g. stricmp function */
    locale_flags = data->locale_flags;
    data->locale_flags |= LOCALE_THREAD;
    newlocinfo = create_locinfo(category, locale, locinfo);
    data->locale_flags = locale_flags;
```
This way all the helper functions will be called with the same locale.