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-image-mask-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-image-mask-compositor.c')
-rw-r--r-- | src/cairo-image-mask-compositor.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/cairo-image-mask-compositor.c b/src/cairo-image-mask-compositor.c index bb990dd65..feb7c6222 100644 --- a/src/cairo-image-mask-compositor.c +++ b/src/cairo-image-mask-compositor.c @@ -387,9 +387,10 @@ composite_boxes (void *_dst, const cairo_compositor_t * _cairo_image_mask_compositor_get (void) { + static cairo_atomic_once_t once = CAIRO_ATOMIC_ONCE_INIT; static cairo_mask_compositor_t compositor; - if (compositor.base.delegate == NULL) { + if (_cairo_atomic_init_once_enter(&once)) { _cairo_mask_compositor_init (&compositor, _cairo_image_traps_compositor_get ()); compositor.acquire = acquire; @@ -405,6 +406,8 @@ _cairo_image_mask_compositor_get (void) compositor.composite = composite; //compositor.check_composite_boxes = check_composite_boxes; compositor.composite_boxes = composite_boxes; + + _cairo_atomic_init_once_leave(&once); } return &compositor.base; |