Bernhard Übelacker : msvcrt: Protect setlocale against concurrent accesses.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 3481d165d16587101857991a02ce169018d9e657
URL:    https://gitlab.winehq.org/wine/wine/-/commit/3481d165d16587101857991a02ce169...

Author: Bernhard Übelacker <bernhardu(a)mailbox.org>
Date:   Fri Dec  8 16:54:45 2023 +0100

msvcrt: Protect setlocale against concurrent accesses.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55467

---

 dlls/msvcrt/locale.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/dlls/msvcrt/locale.c b/dlls/msvcrt/locale.c
index f7be2a1e9bc..99612edfa3e 100644
--- a/dlls/msvcrt/locale.c
+++ b/dlls/msvcrt/locale.c
@@ -2036,6 +2036,7 @@ char* CDECL setlocale(int category, const char* locale)
 {
     thread_data_t *data = msvcrt_get_thread_data();
     pthreadlocinfo locinfo = get_locinfo(), newlocinfo;
+    int locale_flags;
 
     if(category<LC_MIN || category>LC_MAX)
         return NULL;
@@ -2047,7 +2048,11 @@ char* CDECL setlocale(int category, const char* locale)
         return locinfo->lc_category[category].locale;
     }
 
+    /* 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;
     if(!newlocinfo) {
         WARN("%d %s failed\n", category, locale);
         return NULL;