From c524f905a6c7d6f4a0f5c3c97a8a761f1a166959 Mon Sep 17 00:00:00 2001 From: Tilghman Lesher Date: Wed, 13 May 2009 00:13:43 +0000 Subject: [PATCH] Two fixes found while debugging with ast_backtrace(): 1) If MALLOC_DEBUG is used when concurrently using ast_backtrace, the free() used in that routine will trigger an error, because the memory was allocated internally to libc, where we could not intercept that call to wrap it. Therefore, it's not memory we knew about, and the free is reported as an error. 2) Now that channels are objects, the old hack of initializing a channel to all zeroes no longer works, since we may try to call something like ast_channel_lock() within a function on that reference. In that case, it's reported as an error, because the pointer isn't an object reference. git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@194101 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- main/logger.c | 3 +++ main/pbx.c | 8 +++----- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/main/logger.c b/main/logger.c index bceaf092f0..91ae572f90 100644 --- a/main/logger.c +++ b/main/logger.c @@ -1159,6 +1159,9 @@ void ast_backtrace(void) for (i = 0; i < bt->num_frames; i++) { ast_log(LOG_DEBUG, "#%d: [%p] %s\n", i, bt->addresses[i], strings[i]); } + + /* MALLOC_DEBUG will erroneously report an error here, unless we undef the macro. */ +#undef free free(strings); } else { ast_debug(1, "Could not allocate memory for backtrace\n"); diff --git a/main/pbx.c b/main/pbx.c index d5f4ead90a..dbde400b9a 100644 --- a/main/pbx.c +++ b/main/pbx.c @@ -7908,12 +7908,10 @@ static int ast_add_extension2_lockopt(struct ast_context *con, /* If we are adding a hint evalulate in variables and global variables */ if (priority == PRIORITY_HINT && strstr(application, "${") && !strstr(extension, "_")) { - struct ast_channel c = {0, }; - - ast_copy_string(c.exten, extension, sizeof(c.exten)); - ast_copy_string(c.context, con->name, sizeof(c.context)); - pbx_substitute_variables_helper(&c, application, expand_buf, sizeof(expand_buf)); + struct ast_channel *c = ast_channel_alloc(0, AST_STATE_DOWN, 0, 0, "", extension, con->name, 0, "Bogus/%s", __PRETTY_FUNCTION__); + pbx_substitute_variables_helper(c, application, expand_buf, sizeof(expand_buf)); application = expand_buf; + ast_channel_release(c); } length = sizeof(struct ast_exten);