From 6fa8e9ba378c8037676bbdab7d51d984fb00e674 Mon Sep 17 00:00:00 2001 From: rideout Date: Wed, 14 May 2003 13:13:41 +0000 Subject: 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 --- src/Boundary.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'src') 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 -- cgit v1.2.3