diff options
author | Arnold D. Robbins <arnold@skeeve.com> | 2013-09-08 12:46:20 +0200 |
---|---|---|
committer | Arnold D. Robbins <arnold@skeeve.com> | 2013-09-08 12:46:20 +0200 |
commit | c3e4d0cf3f1fd24164e0a58db23b86b56c6dc7c8 (patch) | |
tree | 047d8f1a255b377026cdb8717a22d253401e3540 /regex_internal.c | |
parent | 0b8ef2cbc3e72b40046cd07f56e1cffb27690827 (diff) | |
download | egawk-c3e4d0cf3f1fd24164e0a58db23b86b56c6dc7c8.tar.gz egawk-c3e4d0cf3f1fd24164e0a58db23b86b56c6dc7c8.tar.bz2 egawk-c3e4d0cf3f1fd24164e0a58db23b86b56c6dc7c8.zip |
Fixes based on problems from a static checker.
Diffstat (limited to 'regex_internal.c')
-rw-r--r-- | regex_internal.c | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/regex_internal.c b/regex_internal.c index 5f77bcb0..c7de18b6 100644 --- a/regex_internal.c +++ b/regex_internal.c @@ -1451,7 +1451,18 @@ re_dfa_add_node (re_dfa_t *dfa, re_token_t token) new_eclosures = re_realloc (dfa->eclosures, re_node_set, new_nodes_alloc); if (BE (new_nexts == NULL || new_indices == NULL || new_edests == NULL || new_eclosures == NULL, 0)) - return -1; + { + /* if any are not NULL, free them, avoid leaks */ + if (new_nexts != NULL) + re_free(new_nexts); + if (new_indices != NULL) + re_free(new_indices); + if (new_edests != NULL) + re_free(new_edests); + if (new_eclosures != NULL) + re_free(new_eclosures); + return -1; + } dfa->nexts = new_nexts; dfa->org_indices = new_indices; dfa->edests = new_edests; |