Fix small memory leak in storage of names of boundary conditions.

Thanks to Frank Loeffler for spotting this.


git-svn-id: http://svn.cactuscode.org/arrangements/CactusBase/Boundary/trunk@233 6a38eb6e-646e-4a02-a296-d141613ad6c4

1 files changed, 6 insertions, 3 deletions

diff --git a/src/Boundary.c b/src/Boundary.c
index 91362e0..a94b6d2 100644
--- a/src/Boundary.c
+++ b/src/Boundary.c
@@ -480,8 +480,13 @@ CCTK_INT Bdry_Boundary_SelectVarForBCI(CCTK_POINTER GH,
 		   "Unable to allocate memory for internal 'bcdata' list");
 	retval = -4;
       }
+      /* new_bcdata->next is set below, when inserting into bcdata_list */
+      new_bcdata->var_list = NULL;
+      new_bcdata->bc_name = Util_Strdup(bc_name);
+      /* new_bcdata->num is set further below, when adding first entry to 
+	 var_list */
       
-      /* Place new entry into bcdata list, maintaining case independent sort. */
+      /* Place new entry into bcdata list, maintaining case independent sort */
       for (current_bcdata = bcdata_list, previous_bcdata = NULL; 
 	   current_bcdata; 
 	   previous_bcdata=current_bcdata, 
@@ -541,11 +546,9 @@ CCTK_INT Bdry_Boundary_SelectVarForBCI(CCTK_POINTER GH,
 #endif
       current_bcdata->var_list = new_entry;
       new_entry->next = NULL;
-      current_bcdata->bc_name = Util_Strdup(bc_name);
       current_bcdata->num = 1;
     }
     
-    
     /* Enter new_entry into correct location in linked list.
      * Note that this loop is skipped if new_entry was already inserted as
      * first element of a new var list above (since in that case current will