Răsfoiți Sursa

atomic: Fix infinite recursion in SDL_CompilerBarrier() fallback

On some platforms, SDL_MemoryBarrierRelease() is defined to
SDL_CompilerBarrier(). If SDL_CompilerBarrier() is also defined to
the fallback spinlock acquire/release, then we will infinitely
recurse in SDL_UnlockSpinlock(). Avoid this by not unlocking the
temporary spinlock we create.

(cherry picked from commit 66e98b5598d840b3e4d767ace145b197bc0a9962)
Cameron Gutman 2 săptămâni în urmă
părinte
comite
22b5ceb0c7
1 a modificat fișierele cu 2 adăugiri și 1 ștergeri
  1. 2 1
      include/SDL3/SDL_atomic.h

+ 2 - 1
include/SDL3/SDL_atomic.h

@@ -169,8 +169,9 @@ void _ReadWriteBarrier(void);
 extern __inline void SDL_CompilerBarrier(void);
 #pragma aux SDL_CompilerBarrier = "" parm [] modify exact [];
 #else
+/* We don't unlock here to avoid possible infinite recursion */
 #define SDL_CompilerBarrier()   \
-{ SDL_SpinLock _tmp = 0; SDL_LockSpinlock(&_tmp); SDL_UnlockSpinlock(&_tmp); }
+{ SDL_SpinLock _tmp = 0; SDL_LockSpinlock(&_tmp); }
 #endif
 
 /**