diff options
author | Mikhail Fludkov <misha@pexip.com> | 2017-10-06 13:47:51 +0200 |
---|---|---|
committer | Adrian Johnson <ajohnson@redneon.com> | 2017-10-15 18:51:04 +1030 |
commit | 90104809b0e03d28ac1152034fd4f05fc8e97b9a (patch) | |
tree | 3b2cd2961a1d85c56e8edaaf3c3cdb9f435ac7d0 /src/cairo-xlib-core-compositor.c | |
parent | 79e0e25e441a74e3ec207d95bd83437457ba1885 (diff) | |
download | cairo-90104809b0e03d28ac1152034fd4f05fc8e97b9a.tar.gz |
Surround initialisations with atomic critical section
Fixes the race condition when one thread uses cairo_mask_compositor_t
pointer returned by _cairo_image_mask_compositor_get, while another one
started but has not finished it's initialisation yet
Usage:
static cairo_atomic_once_t once = CAIRO_ATOMIC_ONCE_INIT;
if (_cairo_atomic_init_once_enter(&once)) {
/* Initialization code */
_cairo_atomic_init_once_leave(&once);
}
https://bugs.freedesktop.org/show_bug.cgi?id=103037
Diffstat (limited to 'src/cairo-xlib-core-compositor.c')
-rw-r--r-- | src/cairo-xlib-core-compositor.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/cairo-xlib-core-compositor.c b/src/cairo-xlib-core-compositor.c index 5babcc81b..139299961 100644 --- a/src/cairo-xlib-core-compositor.c +++ b/src/cairo-xlib-core-compositor.c @@ -632,9 +632,10 @@ _cairo_xlib_core_compositor_fill (const cairo_compositor_t *compositor, const cairo_compositor_t * _cairo_xlib_core_compositor_get (void) { + static cairo_atomic_once_t once = CAIRO_ATOMIC_ONCE_INIT; static cairo_compositor_t compositor; - if (compositor.delegate == NULL) { + if (_cairo_atomic_init_once_enter(&once)) { compositor.delegate = _cairo_xlib_fallback_compositor_get (); compositor.paint = _cairo_xlib_core_compositor_paint; @@ -642,6 +643,8 @@ _cairo_xlib_core_compositor_get (void) compositor.fill = _cairo_xlib_core_compositor_fill; compositor.stroke = _cairo_xlib_core_compositor_stroke; compositor.glyphs = NULL; /* XXX PolyGlyph? */ + + _cairo_atomic_init_once_leave(&once); } return &compositor; |