diff options
author | knarf <knarf@0f49ee68-0e4f-0410-9b9c-b2c123ded7ef> | 2011-09-20 17:32:07 +0000 |
---|---|---|
committer | knarf <knarf@0f49ee68-0e4f-0410-9b9c-b2c123ded7ef> | 2011-09-20 17:32:07 +0000 |
commit | 18bc4665359045d9ef115c7817b2c6b76ab0cfc5 (patch) | |
tree | 36f043b810a322adf9346a28f7c45dabc2ab7293 | |
parent | 3192ba91b2ff13f2e494282f4b09ed7740497716 (diff) |
Steven Brandt:
In AEIThorns/AEILocalInterp/src/InterpLocalUniform.c
1370 if (value_not_set)
1371 then {
1372 /* default operand will use each input exactly once, */
1373 /* but this only makes since if N_input_arrays == N_output_arrays
*/
1374 if (N_input_arrays != N_input_arrays)
1375 then {
1376 free(input_array_offsets);
1377 free(operand_indices);
1378 CCTK_VWarn(ERROR_MSG_SEVERITY_LEVEL,
The test is always going to fail and data is not going to be freed up.
Based on the comment, I think that the test should compare N_input_arrays
to N_output_arrays.
git-svn-id: http://svn.aei.mpg.de/numrel/AEIThorns/AEILocalInterp/trunk@52 0f49ee68-0e4f-0410-9b9c-b2c123ded7ef
-rw-r--r-- | src/InterpLocalUniform.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/InterpLocalUniform.c b/src/InterpLocalUniform.c index 7cc6ff0..3d35b2c 100644 --- a/src/InterpLocalUniform.c +++ b/src/InterpLocalUniform.c @@ -1371,7 +1371,7 @@ if (value_not_set) then { /* default operand will use each input exactly once, */ /* but this only makes since if N_input_arrays == N_output_arrays */ - if (N_input_arrays != N_input_arrays) + if (N_input_arrays != N_output_arrays) then { free(input_array_offsets); free(operand_indices); |