aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorfile <file@f38db490-d61c-443f-a65b-d21fe96a405b>2009-05-28 14:54:25 +0000
committerfile <file@f38db490-d61c-443f-a65b-d21fe96a405b>2009-05-28 14:54:25 +0000
commit2bdd3cbcf15c45ed19b8bf5615c8750c6fd2b52b (patch)
treeeb937f27fd873a146cbff3b82b7a6639f5c30aaf
parent77a617d7842f30b6b7ba5b4323f4b410eae7b2c9 (diff)
Merged revisions 197538 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk ........ r197538 | file | 2009-05-28 11:51:43 -0300 (Thu, 28 May 2009) | 5 lines Fix a bug in stringfields where it did not actually free the pools of memory. (closes issue #15074) Reported by: pj ........ git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.1@197540 f38db490-d61c-443f-a65b-d21fe96a405b
-rw-r--r--main/utils.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/main/utils.c b/main/utils.c
index 76223498e..c0418dafc 100644
--- a/main/utils.c
+++ b/main/utils.c
@@ -1530,7 +1530,11 @@ int __ast_string_field_init(struct ast_string_field_mgr *mgr, struct ast_string_
return add_string_pool(mgr, pool_head, needed, file, lineno, func);
}
if (needed < 0) { /* reset all pools */
- /* nothing to do */
+ if (*pool_head == NULL) {
+ ast_log(LOG_WARNING, "trying to reset empty pool\n");
+ return -1;
+ }
+ cur = *pool_head;
} else { /* preserve the last pool */
if (*pool_head == NULL) {
ast_log(LOG_WARNING, "trying to reset empty pool\n");