[Openmp-commits] [openmp] r292348 - Fix small memory leak regarding __kmp_nested_proc_bind

Jonathan Peyton via Openmp-commits openmp-commits at lists.llvm.org
Tue Jan 17 22:40:19 PST 2017


Author: jlpeyton
Date: Wed Jan 18 00:40:19 2017
New Revision: 292348

URL: http://llvm.org/viewvc/llvm-project?rev=292348&view=rev
Log:
Fix small memory leak regarding __kmp_nested_proc_bind

There is no corresponding free() for this expandable array.  The logic is
added in __kmp_cleanup() next to the freeing of __kmp_nested_nth.

Modified:
    openmp/trunk/runtime/src/kmp_runtime.cpp

Modified: openmp/trunk/runtime/src/kmp_runtime.cpp
URL: http://llvm.org/viewvc/llvm-project/openmp/trunk/runtime/src/kmp_runtime.cpp?rev=292348&r1=292347&r2=292348&view=diff
==============================================================================
--- openmp/trunk/runtime/src/kmp_runtime.cpp (original)
+++ openmp/trunk/runtime/src/kmp_runtime.cpp Wed Jan 18 00:40:19 2017
@@ -7293,6 +7293,10 @@ __kmp_cleanup( void )
     __kmp_nested_nth.nth = NULL;
     __kmp_nested_nth.size = 0;
     __kmp_nested_nth.used = 0;
+    KMP_INTERNAL_FREE( __kmp_nested_proc_bind.bind_types );
+    __kmp_nested_proc_bind.bind_types = NULL;
+    __kmp_nested_proc_bind.size = 0;
+    __kmp_nested_proc_bind.used = 0;
 
     __kmp_i18n_catclose();
 




More information about the Openmp-commits mailing list