Re: [FFmpeg-devel] [PATCH] png: split header state and data state in two separate variables.

2017-04-03 Thread Michael Niedermayer
On Mon, Apr 03, 2017 at 10:09:30AM -0400, Ronald S. Bultje wrote: > Fixes a reported (but false) race condition in tsan for fate-apng: > > WARNING: ThreadSanitizer: data race (pid=6274) > Read of size 4 at 0x7d680001ec78 by main thread (mutexes: write M1338): > #0 update_thread_context src/l

[FFmpeg-devel] [PATCH] png: split header state and data state in two separate variables.

2017-04-03 Thread Ronald S. Bultje
Fixes a reported (but false) race condition in tsan for fate-apng: WARNING: ThreadSanitizer: data race (pid=6274) Read of size 4 at 0x7d680001ec78 by main thread (mutexes: write M1338): #0 update_thread_context src/libavcodec/pngdec.c:1456 (ffmpeg+0x00dacf0c) [..] Previous write of si

Re: [FFmpeg-devel] [PATCH] png: split header state and data state in two separate variables.

2017-03-31 Thread Michael Niedermayer
On Fri, Mar 31, 2017 at 09:49:52AM -0400, Ronald S. Bultje wrote: > Fixes a reported (but false) race condition in tsan for fate-apng. > --- > libavcodec/png.h| 5 > libavcodec/pngdec.c | 68 > +++-- > 2 files changed, 40 insertions(+), 33

[FFmpeg-devel] [PATCH] png: split header state and data state in two separate variables.

2017-03-31 Thread Ronald S. Bultje
Fixes a reported (but false) race condition in tsan for fate-apng. --- libavcodec/png.h| 5 libavcodec/pngdec.c | 68 +++-- 2 files changed, 40 insertions(+), 33 deletions(-) diff --git a/libavcodec/png.h b/libavcodec/png.h index 948c2f7..