diff options
author | rideout <rideout@6a38eb6e-646e-4a02-a296-d141613ad6c4> | 2003-05-14 13:13:41 +0000 |
---|---|---|
committer | rideout <rideout@6a38eb6e-646e-4a02-a296-d141613ad6c4> | 2003-05-14 13:13:41 +0000 |
commit | 6fa8e9ba378c8037676bbdab7d51d984fb00e674 (patch) | |
tree | 10c0b0b3214b74a2b67645368920e29da25dd575 | |
parent | 1ef1806f0950b044c02c9c32d7e583ec10d17178 (diff) |
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
-rw-r--r-- | src/Boundary.c | 9 |
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 |