Fix memory leak in SUNMemoryHelpers

CHANGELOG.md

@@ -1,5 +1,10 @@
 # SUNDIALS Changelog
 
+## Changes to SUNDIALS in release x.x.x
+
+Fixed a memory leak when destroying a CUDA, HIP, SYCL, or system SUNMemoryHelper
+object.
+
 ## Changes to SUNDIALS in release 6.6.0
 
 A new time-stepping module, `SPRKStep`, was added to ARKODE. This time-stepper

doc/arkode/guide/source/Introduction.rst

@@ -130,6 +130,12 @@ provided with SUNDIALS, or again may utilize a user-supplied module.
 Changes from previous versions
 ==============================
 
+Changes in vX.X.X
+-----------------
+
+Fixed a memory leak when destroying a CUDA, HIP, SYCL, or system SUNMemoryHelper
+object.
+
 Changes in v5.6.0
 -----------------
 

doc/cvode/guide/source/Introduction.rst

@@ -111,6 +111,12 @@ implementations.
 Changes from previous versions
 ==============================
 
+Changes in vX.X.X
+-----------------
+
+Fixed a memory leak when destroying a CUDA, HIP, SYCL, or system SUNMemoryHelper
+object.
+
 Changes in v6.6.0
 -----------------
 

doc/cvodes/guide/source/Introduction.rst

@@ -111,6 +111,12 @@ Fortran.
 Changes from previous versions
 ==============================
 
+Changes in vX.X.X
+-----------------
+
+Fixed a memory leak when destroying a CUDA, HIP, SYCL, or system SUNMemoryHelper
+object.
+
 Changes in v6.6.0
 -----------------
 

doc/ida/guide/source/Introduction.rst

@@ -72,6 +72,12 @@ systems.
 Changes from previous versions
 ==============================
 
+Changes in vX.X.X
+-----------------
+
+Fixed a memory leak when destroying a CUDA, HIP, SYCL, or system SUNMemoryHelper
+object.
+
 Changes in v6.6.0
 -----------------
 

doc/idas/guide/source/Introduction.rst

@@ -86,6 +86,12 @@ integrate any final-condition ODE dependent on the solution of the original IVP
 Changes from previous versions
 ==============================
 
+Changes in vX.X.X
+-----------------
+
+Fixed a memory leak when destroying a CUDA, HIP, SYCL, or system SUNMemoryHelper
+object.
+
 Changes in v5.6.0
 -----------------
 

doc/kinsol/guide/source/Introduction.rst

@@ -88,6 +88,12 @@ applications written in Fortran.
 Changes from previous versions
 ==============================
 
+Changes in vX.X.X
+-----------------
+
+Fixed a memory leak when destroying a CUDA, HIP, SYCL, or system SUNMemoryHelper
+object.
+
 Changes in v6.6.0
 -----------------
 

src/sunmemory/cuda/sundials_cuda_memory.cu

@@ -341,7 +341,8 @@ int SUNMemoryHelper_Destroy_Cuda(SUNMemoryHelper helper)
 {
   if (helper)
   {
-    free(helper->content);
+    if (helper->content) { free(helper->content); }
+    if (helper->ops) { free(helper->ops); }
     free(helper);
   }
   return 0;

src/sunmemory/hip/sundials_hip_memory.hip.cpp

@@ -340,7 +340,8 @@ int SUNMemoryHelper_Destroy_Hip(SUNMemoryHelper helper)
 {
   if (helper)
   {
-    free(helper->content);
+    if (helper->content) { free(helper->content); }
+    if (helper->ops) { free(helper->ops); }
     free(helper);
   }
   return 0;

src/sunmemory/sycl/sundials_sycl_memory.cpp

@@ -304,7 +304,8 @@ int SUNMemoryHelper_Destroy_Sycl(SUNMemoryHelper helper)
 {
   if (helper)
   {
-    free(helper->content);
+    if (helper->content) { free(helper->content); }
+    if (helper->ops) { free(helper->ops); }
     free(helper);
   }
   return 0;

src/sunmemory/system/sundials_system_memory.c

@@ -160,7 +160,8 @@ int SUNMemoryHelper_Destroy_Sys(SUNMemoryHelper helper)
 {
   if (helper)
   {
-    free(helper->content);
+    if (helper->content) { free(helper->content); }
+    if (helper->ops) { free(helper->ops); }
     free(helper);
   }
   return 0;