diff options
| author | Julian Brown <julian@codesourcery.com> | 2020-05-22 15:16:04 -0700 |
|---|---|---|
| committer | Thomas Schwinge <thomas@codesourcery.com> | 2020-06-04 18:56:36 +0200 |
| commit | dc9541545d9e5705a97c41713c557f55522b54dc (patch) | |
| tree | 861b408db11b729222b2ac3021fb1219185382ac /libgomp/oacc-mem.c | |
| parent | 1e378edd8f038c52914bc6cac8f6dd5a5c4f8a98 (diff) | |
[OpenACC] Missing unlocking on error paths in attach/detach code
libgomp/
* oacc-mem.c (acc_attach_async): Add missing gomp_mutex_unlock on
error path.
(goacc_detach_internal): Likewise.
Reviewed-by: Thomas Schwinge <thomas@codesourcery.com>
Diffstat (limited to 'libgomp/oacc-mem.c')
| -rw-r--r-- | libgomp/oacc-mem.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/libgomp/oacc-mem.c b/libgomp/oacc-mem.c index 2d4bba78efd..c06b7341cbb 100644 --- a/libgomp/oacc-mem.c +++ b/libgomp/oacc-mem.c @@ -887,7 +887,10 @@ acc_attach_async (void **hostaddr, int async) n = splay_tree_lookup (&acc_dev->mem_map, &cur_node); if (n == NULL) - gomp_fatal ("struct not mapped for acc_attach"); + { + gomp_mutex_unlock (&acc_dev->lock); + gomp_fatal ("struct not mapped for acc_attach"); + } gomp_attach_pointer (acc_dev, aq, &acc_dev->mem_map, n, (uintptr_t) hostaddr, 0, NULL); @@ -920,7 +923,10 @@ goacc_detach_internal (void **hostaddr, int async, bool finalize) n = splay_tree_lookup (&acc_dev->mem_map, &cur_node); if (n == NULL) - gomp_fatal ("struct not mapped for acc_detach"); + { + gomp_mutex_unlock (&acc_dev->lock); + gomp_fatal ("struct not mapped for acc_detach"); + } gomp_detach_pointer (acc_dev, aq, n, (uintptr_t) hostaddr, finalize, NULL); |