[Openmp-commits] [PATCH] D26860: Fix for D25504 - segfault because of double free()-ing in task deps shutdown code.

[Jonathan Peyton via Openmp-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL287551: Fix for D25504 - segfault because of double free()-ing in shutdown code. (authored by jlpeyton).

Changed prior to commit:
  https://reviews.llvm.org/D26860?vs=78538&id=78726#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D26860

Files:
  openmp/trunk/runtime/src/kmp_runtime.c


Index: openmp/trunk/runtime/src/kmp_runtime.c
===================================================================
--- openmp/trunk/runtime/src/kmp_runtime.c
+++ openmp/trunk/runtime/src/kmp_runtime.c
@@ -5707,6 +5707,8 @@
         --__kmp_thread_pool_nth;
     }; // if
 
+    __kmp_free_implicit_task(thread);
+
     // Free the fast memory for tasking
     #if USE_FAST_MEMORY
         __kmp_free_fast_memory( thread );
@@ -5762,7 +5764,6 @@
     }; // if
 #endif /* KMP_AFFINITY_SUPPORTED */
 
-    __kmp_free_implicit_task(thread);
     __kmp_reap_team( thread->th.th_serial_team );
     thread->th.th_serial_team = NULL;
     __kmp_free( thread );


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26860.78726.patch
Type: text/x-patch
Size: 653 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20161121/ebab16a8/attachment.bin>