Skip to content

Commit fb4fb86

Browse files
author
Darth Vader
committed
Squashed 'src/' changes from c035d9c3..8f6cb45a
8f6cb45a Pad needed a free. Ran all test cases. 56ba6801 Freed result from strexpr git-subtree-dir: src git-subtree-split: 8f6cb45a04ca40ea253c0f3ca90f52427d99592d
1 parent 819297a commit fb4fb86

File tree

1 file changed

+11
-3
lines changed

1 file changed

+11
-3
lines changed

PBasic.cpp

+11-3
Original file line numberDiff line numberDiff line change
@@ -3211,13 +3211,15 @@ factor(struct LOC_exec * LINK)
32113211
case tokpad_:
32123212
case tokpad:
32133213
{
3214+
char* str;
32143215
n.stringval = true;
32153216
require(toklp, LINK);
3216-
string1 = strexpr(LINK);
3217+
str = strexpr(LINK);
32173218
require(tokcomma, LINK);
32183219
i = intexpr(LINK);
32193220
require(tokrp, LINK);
3220-
n.UU.sval = PhreeqcPtr->string_pad(string1, i);
3221+
n.UU.sval = PhreeqcPtr->string_pad(str, i);
3222+
PhreeqcPtr->PHRQ_free(str);
32213223
}
32223224
break;
32233225

@@ -3249,6 +3251,7 @@ factor(struct LOC_exec * LINK)
32493251
break;
32503252
}
32513253
std::string min_string = min_name;
3254+
PhreeqcPtr->PHRQ_free(min_name);
32523255
Utilities::str_tolower(min_string);
32533256
std::map<std::string, std::vector<double> >::const_iterator it = PhreeqcPtr->rate_parameters_pk.find(min_string);
32543257
if (it == PhreeqcPtr->rate_parameters_pk.end())
@@ -3420,6 +3423,7 @@ factor(struct LOC_exec * LINK)
34203423
break;
34213424
}
34223425
std::string min_string = min_name;
3426+
PhreeqcPtr->PHRQ_free(min_name);
34233427
Utilities::str_tolower(min_string);
34243428
std::map<std::string, std::vector<double> >::const_iterator it = PhreeqcPtr->rate_parameters_svd.find(min_string);
34253429
if (it == PhreeqcPtr->rate_parameters_svd.end())
@@ -3574,6 +3578,7 @@ factor(struct LOC_exec * LINK)
35743578
break;
35753579
}
35763580
std::string min_string = min_name;
3581+
PhreeqcPtr->PHRQ_free(min_name);
35773582
Utilities::str_tolower(min_string);
35783583
std::map<std::string, std::vector<double> >::const_iterator it = PhreeqcPtr->rate_parameters_hermanska.find(min_string);
35793584
if (it == PhreeqcPtr->rate_parameters_hermanska.end())
@@ -3667,6 +3672,7 @@ factor(struct LOC_exec * LINK)
36673672
break;
36683673
}
36693674
std::string min_string = min_name;
3675+
PhreeqcPtr->PHRQ_free(min_name);
36703676
Utilities::str_tolower(min_string);
36713677
std::map<std::string, cxxNameDouble>::const_iterator it = PhreeqcPtr->mean_gammas.find(min_string);
36723678
if (it == PhreeqcPtr->mean_gammas.end() || it->second.size() == 0)
@@ -5403,7 +5409,9 @@ cmdput_(struct LOC_exec* LINK)
54035409
require(toklp, LINK);
54045410

54055411
/* get first argumen */
5406-
std::string s_value = strexpr(LINK);
5412+
char* str = strexpr(LINK);
5413+
std::string s_value = str;
5414+
PhreeqcPtr->PHRQ_free(str);
54075415

54085416
for (;;)
54095417
{

0 commit comments

Comments
 (0)