Merge pull request #1053 from CEED/natalie/fix-magma-jit-mem

Fix small memory leaks in JIT source code management
CEED · Aug 30, 2022 · 01005ea · 01005ea
2 parents 11a69ce + 9af6094
commit 01005ea
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/backends/magma/ceed-magma-basis.c b/backends/magma/ceed-magma-basis.c
@@ -757,6 +757,11 @@ int CeedBasisCreateTensorH1_Magma(CeedInt dim, CeedInt P1d, CeedInt Q1d,
                   data->queue);
 
   ierr = CeedBasisSetData(basis, impl); CeedChkBackend(ierr);
+  ierr = CeedFree(&magma_common_path); CeedChkBackend(ierr);
+  ierr = CeedFree(&interp_path); CeedChkBackend(ierr);
+  ierr = CeedFree(&grad_path); CeedChkBackend(ierr);
+  ierr = CeedFree(&weight_path); CeedChkBackend(ierr);
+  ierr = CeedFree(&basis_kernel_source); CeedChkBackend(ierr);
 
   return CEED_ERROR_SUCCESS;
 }

diff --git a/backends/magma/ceed-magma-restriction.c b/backends/magma/ceed-magma-restriction.c
@@ -320,6 +320,9 @@ int CeedElemRestrictionCreate_Magma(CeedMemType mtype, CeedCopyMode cmode,
   CeedChkBackend(ierr);
   ierr = CeedSetBackendFunction(ceed, "ElemRestriction", r, "Destroy",
                                 CeedElemRestrictionDestroy_Magma); CeedChkBackend(ierr);
+  ierr = CeedFree(&restriction_kernel_path); CeedChkBackend(ierr);
+  ierr = CeedFree(&restriction_kernel_source); CeedChkBackend(ierr);
+
   return CEED_ERROR_SUCCESS;
 }
 

diff --git a/interface/ceed-jit-tools.c b/interface/ceed-jit-tools.c
@@ -171,6 +171,7 @@ int CeedLoadSourceToInitializedBuffer(Ceed ceed,
           memcpy(ceed_relative_path, &next_left_chevron[1], ceed_relative_path_length);
           ierr = CeedGetJitAbsolutePath(ceed, ceed_relative_path, &include_source_path);
           CeedChk(ierr);
+          ierr = CeedFree(&ceed_relative_path); CeedChk(ierr);
         }
         // ---- Recursive call to load source to buffer
         CeedDebug256(ceed, 2, "JiT Including: %s\n", include_source_path);