Apply suggestions from code review

Co-authored-by: David Gardner <[email protected]>
LLNL · Nov 1, 2023 · 5e099bf · 5e099bf
1 parent a554117
commit 5e099bf
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/src/arkode/arkode.c b/src/arkode/arkode.c
@@ -1119,6 +1119,7 @@ void arkFree(void **arkode_mem)
   if (ark_mem->hadapt_mem != NULL) {
     if (ark_mem->hadapt_mem->owncontroller) {
       (void) SUNAdaptController_Destroy(ark_mem->hadapt_mem->hcontroller);
+      ark_mem->hadapt_mem->owncontroller = SUNFALSE;
     }
     free(ark_mem->hadapt_mem);
     ark_mem->hadapt_mem = NULL;

diff --git a/src/arkode/arkode_arkstep_io.c b/src/arkode/arkode_arkstep_io.c
@@ -502,6 +502,7 @@ int ARKStepSetOptimalParams(void *arkode_mem)
   }
   if (hadapt_mem->owncontroller) {
     retval = SUNAdaptController_Destroy(hadapt_mem->hcontroller);
+    ark_mem->hadapt_mem->owncontroller = SUNFALSE;
     if (retval != SUNADAPTCONTROLLER_SUCCESS) {
       arkProcessError(ark_mem, ARK_MEM_FAIL, "ARKODE", "ARKStepSetOptimalParams",
                       "SUNAdaptController_Destroy failure");

diff --git a/src/arkode/arkode_erkstep_io.c b/src/arkode/arkode_erkstep_io.c
@@ -294,6 +294,7 @@ int ERKStepSetDefaults(void* arkode_mem)
   }
   if (ark_mem->hadapt_mem->owncontroller) {
     retval = SUNAdaptController_Destroy(ark_mem->hadapt_mem->hcontroller);
+    ark_mem->hadapt_mem->owncontroller = SUNFALSE;
     if (retval != SUNADAPTCONTROLLER_SUCCESS) {
       arkProcessError(ark_mem, ARK_MEM_FAIL, "ARKODE", "ERKStepSetDefaults",
                       "SUNAdaptController_Destroy failure");