TeamHepta/SDL
1
0
Fork 0
mirror of https://github.com/libsdl-org/SDL.git synced 2026-03-31 14:21:06 +02:00
Code Issues Packages Projects Releases Wiki Activity

thread: SDL_CreateThread() shouldn't return before the new thread is set up.

Browse Source 
Fixes #15290.
This commit is contained in:
Ryan C. Gordon
2026-03-30 10:08:06 -04:00
parent e66f1b5162
commit 0756603e6d
2 changed files with 15 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/thread/SDL_thread.c
View File

@@ -333,6 +333,8 @@ void SDL_RunThread(SDL_Thread *thread)
// Get the thread id
thread->threadid = SDL_GetCurrentThreadID();
SDL_SignalSemaphore(thread->ready_sem); // the thread is officially ready to run!
// Run the function
*statusloc = userfunc(userdata);
@@ -389,6 +391,13 @@ SDL_Thread *SDL_CreateThreadWithPropertiesRuntime(SDL_PropertiesID props,
}
}
thread->ready_sem = SDL_CreateSemaphore(0);
if (!thread->ready_sem) {
SDL_free(thread->name);
SDL_free(thread);
return NULL;
}
thread->userfunc = fn;
thread->userdata = userdata;
thread->stacksize = stacksize;
@@ -399,11 +408,16 @@ SDL_Thread *SDL_CreateThreadWithPropertiesRuntime(SDL_PropertiesID props,
if (!SDL_SYS_CreateThread(thread, pfnBeginThread, pfnEndThread)) {
// Oops, failed. Gotta free everything
SDL_SetObjectValid(thread, SDL_OBJECT_TYPE_THREAD, false);
SDL_DestroySemaphore(thread->ready_sem);
SDL_free(thread->name);
SDL_free(thread);
thread = NULL;
}
SDL_WaitSemaphore(thread->ready_sem);
SDL_DestroySemaphore(thread->ready_sem);
thread->ready_sem = NULL;
// Everything is running now
return thread;
}

1
src/thread/SDL_thread_c.h
View File

@@ -54,6 +54,7 @@ struct SDL_Thread
SDL_error errbuf;
char *name;
size_t stacksize; // 0 for default, >0 for user-specified stack size.
SDL_Semaphore *ready_sem; // signals when the thread is set up and about to start running.
int(SDLCALL *userfunc)(void *);
void *userdata;
void *data;