ffmpeg/compat
Anton Khirnov d2096679d5 compat/w32pthreads: change pthread_t into pointer to malloced struct
pthread_t is currently defined as a struct, which gets placed into
caller's memory and filled by pthread_create() (which accepts a
pthread_t*).

The problem with this approach is that pthread_join() accepts pthread_t
itself rather than a pointer to it, so it gets a _copy_ of this
structure. This causes non-deterministic failures of pthread_join() to
produce the correct return value - depending on whether the thread
already finished before pthread_join() is called (and thus the copy
contains the correct value), or not (then it contains 0).

Change the definition of pthread_t into a pointer to a struct, that gets
malloced by pthread_create() and freed by pthread_join().

Fixes random failures of fate-ffmpeg-error-rate-fail on Windows after
433cf391f5.

See also [1] for an alternative approach that does not require dynamic
allocation, but relies on an assumption that the pthread_t value
remains in a fixed memory location.

[1] 23829dd2b2

Reviewed-By: Martin Storsjö <martin@martin.st>
2024-12-16 09:43:19 +01:00
..
aix
atomics
cuda
dispatch_semaphore
djgpp
float
msvcrt
solaris
stdbit compat: Fix the fallback definition of stdc_trailing_zeros 2024-09-24 14:00:27 +03:00
windows
getopt.c
os2threads.h
strtod.c
va_copy.h
w32dlfcn.h
w32pthreads.h compat/w32pthreads: change pthread_t into pointer to malloced struct 2024-12-16 09:43:19 +01:00