fixed bug with potentially changing global variable in thread

[rocksndiamonds.git] / src / files.c

diff --git a/src/files.c b/src/files.c
index 93d38e7c44e2d238e1cc7bb9b64168c489e30b2b..8c37e1c01fde9d33f1583e3e8f25f21f96936dab 100644 (file)
--- a/src/files.c
+++ b/src/files.c
@@ -9002,14 +9002,10 @@ void SaveScore(int nr)
   SaveScoreToFilename(filename);
 }
 
-static void ExecuteAsThread(SDL_ThreadFunction function, char *name, int data,
+static void ExecuteAsThread(SDL_ThreadFunction function, char *name, void *data,
                            char *error)
 {
-  static int data_static;
-
-  data_static = data;
-
-  SDL_Thread *thread = SDL_CreateThread(function, name, &data_static);
+  SDL_Thread *thread = SDL_CreateThread(function, name, data);
 
   if (thread != NULL)
     SDL_DetachThread(thread);
@@ -9098,17 +9094,28 @@ static void DownloadServerScoreToCache(int nr)
   checked_free(response);
 }
 
-static int DownloadServerScoreToCacheThread(void *data)
+struct DownloadServerScoreToCacheThreadData
 {
-  DownloadServerScoreToCache(*(int *)data);
+  int level_nr;
+};
+
+static int DownloadServerScoreToCacheThread(void *data_raw)
+{
+  struct DownloadServerScoreToCacheThreadData *data = data_raw;
+
+  DownloadServerScoreToCache(data->level_nr);
 
   return 0;
 }
 
 static void DownloadServerScoreToCacheAsThread(int nr)
 {
+  static struct DownloadServerScoreToCacheThreadData data;
+
+  data.level_nr = nr;
+
   ExecuteAsThread(DownloadServerScoreToCacheThread,
-                 "DownloadServerScoreToCache", nr,
+                 "DownloadServerScoreToCache", &data,
                  "download scores from server");
 }
 
@@ -9304,7 +9311,7 @@ static void UploadScoreToServerExt(struct HttpRequest *request,
           levelset_author,
           leveldir_current->levels,
           leveldir_current->first_level,
-          level_nr,
+          nr,
           level_name,
           level_author,
           level.rate_time_over_score,
@@ -9353,17 +9360,28 @@ static void UploadScoreToServer(int nr)
   checked_free(response);
 }
 
-static int UploadScoreToServerThread(void *data)
+struct UploadScoreToServerThreadData
+{
+  int level_nr;
+};
+
+static int UploadScoreToServerThread(void *data_raw)
 {
-  UploadScoreToServer(*(int *)data);
+  struct UploadScoreToServerThreadData *data = data_raw;
+
+  UploadScoreToServer(data->level_nr);
 
   return 0;
 }
 
 static void UploadScoreToServerAsThread(int nr)
 {
+  static struct UploadScoreToServerThreadData data;
+
+  data.level_nr = nr;
+
   ExecuteAsThread(UploadScoreToServerThread,
-                 "UploadScoreToServer", nr,
+                 "UploadScoreToServer", &data,
                  "upload score to server");
 }