[asterisk-commits] rmudgett: trunk r397570 - in /trunk: ./ channels/ include/asterisk/ main/
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Fri Aug 23 13:07:48 CDT 2013
Author: rmudgett
Date: Fri Aug 23 13:07:40 2013
New Revision: 397570
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=397570
Log:
Fix memory corruption when trying to get "core show locks".
Review https://reviewboard.asterisk.org/r/2580/ tried to fix the mismatch
in memory pools but had a math error determining the buffer size and
didn't address other similar memory pool mismatches.
* Effectively reverted the previous patch to go in the same direction as
trunk for the returned memory pool of ast_bt_get_symbols().
* Fixed memory leak in ast_bt_get_symbols() when BETTER_BACKTRACES is
defined.
* Fixed some formatting in ast_bt_get_symbols().
* Fixed sig_pri.c freeing memory allocated by libpri when MALLOC_DEBUG is
enabled.
* Fixed __dump_backtrace() freeing memory from ast_bt_get_symbols() when
MALLOC_DEBUG is enabled.
* Moved __dump_backtrace() because of compile issues with the utils
directory.
(closes issue ASTERISK-22221)
Reported by: Matt Jordan
Review: https://reviewboard.asterisk.org/r/2778/
........
Merged revisions 397525 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........
Merged revisions 397528 from http://svn.asterisk.org/svn/asterisk/branches/11
Modified:
trunk/ (props changed)
trunk/channels/sig_pri.c
trunk/include/asterisk/astmm.h
trunk/include/asterisk/backtrace.h
trunk/include/asterisk/lock.h
trunk/include/asterisk/utils.h
trunk/main/astmm.c
trunk/main/astobj2.c
trunk/main/backtrace.c
trunk/main/lock.c
trunk/main/logger.c
trunk/main/utils.c
Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-11-merged' - no diff available.
Modified: trunk/channels/sig_pri.c
URL: http://svnview.digium.com/svn/asterisk/trunk/channels/sig_pri.c?view=diff&rev=397570&r1=397569&r2=397570
==============================================================================
--- trunk/channels/sig_pri.c (original)
+++ trunk/channels/sig_pri.c Fri Aug 23 13:07:40 2013
@@ -9430,7 +9430,7 @@
info_str = pri_dump_info_str(pri->pri);
if (info_str) {
ast_cli(fd, "%s", info_str);
- free(info_str);
+ ast_std_free(info_str);
}
#else
pri_dump_info(pri->pri);
Modified: trunk/include/asterisk/astmm.h
URL: http://svnview.digium.com/svn/asterisk/trunk/include/asterisk/astmm.h?view=diff&rev=397570&r1=397569&r2=397570
==============================================================================
--- trunk/include/asterisk/astmm.h (original)
+++ trunk/include/asterisk/astmm.h Fri Aug 23 13:07:40 2013
@@ -53,6 +53,12 @@
#undef asprintf
#undef vasprintf
#undef free
+
+void *ast_std_malloc(size_t size);
+void *ast_std_calloc(size_t nmemb, size_t size);
+void *ast_std_realloc(void *ptr, size_t size);
+void ast_std_free(void *ptr);
+void ast_free_ptr(void *ptr);
void *__ast_calloc(size_t nmemb, size_t size, const char *file, int lineno, const char *func);
void *__ast_calloc_cache(size_t nmemb, size_t size, const char *file, int lineno, const char *func);
Modified: trunk/include/asterisk/backtrace.h
URL: http://svnview.digium.com/svn/asterisk/trunk/include/asterisk/backtrace.h?view=diff&rev=397570&r1=397569&r2=397570
==============================================================================
--- trunk/include/asterisk/backtrace.h (original)
+++ trunk/include/asterisk/backtrace.h Fri Aug 23 13:07:40 2013
@@ -87,7 +87,7 @@
* \param addresses A list of addresses, such as the ->addresses structure element of struct ast_bt.
* \param num_frames Number of addresses in the addresses list
* \retval NULL Unable to allocate memory
- * \return List of strings. This should be freed with a single call to free.
+ * \return List of strings. Free the entire list with a single ast_std_free call.
* \since 1.6.2.16
*/
char **__ast_bt_get_symbols(void **addresses, size_t num_frames);
Modified: trunk/include/asterisk/lock.h
URL: http://svnview.digium.com/svn/asterisk/trunk/include/asterisk/lock.h?view=diff&rev=397570&r1=397569&r2=397570
==============================================================================
--- trunk/include/asterisk/lock.h (original)
+++ trunk/include/asterisk/lock.h Fri Aug 23 13:07:40 2013
@@ -289,22 +289,6 @@
#define ast_remove_lock_info(ignore)
#endif /* HAVE_BKTR */
#endif /* !defined(LOW_MEMORY) */
-
-#ifdef HAVE_BKTR
-static inline void __dump_backtrace(struct ast_bt *bt, int canlog)
-{
- char **strings;
-
- ssize_t i;
-
- strings = backtrace_symbols(bt->addresses, bt->num_frames);
-
- for (i = 0; i < bt->num_frames; i++)
- __ast_mutex_logger("%s\n", strings[i]);
-
- free(strings);
-}
-#endif
/*!
* \brief log info for the current lock with ast_log().
Modified: trunk/include/asterisk/utils.h
URL: http://svnview.digium.com/svn/asterisk/trunk/include/asterisk/utils.h?view=diff&rev=397570&r1=397569&r2=397570
==============================================================================
--- trunk/include/asterisk/utils.h (original)
+++ trunk/include/asterisk/utils.h Fri Aug 23 13:07:40 2013
@@ -465,24 +465,20 @@
*/
#define ast_random_double() (((double)ast_random()) / RAND_MAX)
+#ifndef __AST_DEBUG_MALLOC
+#define ast_std_malloc malloc
+#define ast_std_calloc calloc
+#define ast_std_realloc realloc
+#define ast_std_free free
+
/*!
* \brief free() wrapper
*
* ast_free_ptr should be used when a function pointer for free() needs to be passed
* as the argument to a function. Otherwise, astmm will cause seg faults.
*/
-#ifdef __AST_DEBUG_MALLOC
-static void ast_free_ptr(void *ptr) attribute_unused;
-static void ast_free_ptr(void *ptr)
-{
- ast_free(ptr);
-}
-#else
#define ast_free free
#define ast_free_ptr ast_free
-#endif
-
-#ifndef __AST_DEBUG_MALLOC
/*
* This buffer is in static memory. We never intend to read it,
Modified: trunk/main/astmm.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/astmm.c?view=diff&rev=397570&r1=397569&r2=397570
==============================================================================
--- trunk/main/astmm.c (original)
+++ trunk/main/astmm.c Fri Aug 23 13:07:40 2013
@@ -158,6 +158,31 @@
} \
} while (0)
+void *ast_std_malloc(size_t size)
+{
+ return malloc(size);
+}
+
+void *ast_std_calloc(size_t nmemb, size_t size)
+{
+ return calloc(nmemb, size);
+}
+
+void *ast_std_realloc(void *ptr, size_t size)
+{
+ return realloc(ptr, size);
+}
+
+void ast_std_free(void *ptr)
+{
+ free(ptr);
+}
+
+void ast_free_ptr(void *ptr)
+{
+ ast_free(ptr);
+}
+
static void print_backtrace(struct ast_bt *bt)
{
int i = 0;
@@ -172,10 +197,9 @@
for (i = 3; i < bt->num_frames - 2; i++) {
astmm_log("#%d: [%p] %s\n", i - 3, bt->addresses[i], strings[i]);
}
- free(strings);
- }
-}
-
+ ast_std_free(strings);
+ }
+}
/*!
* \internal
@@ -343,9 +367,7 @@
if (old) {
region_data_check(old);
- if (old->bt) {
- old->bt = ast_bt_destroy(old->bt);
- }
+ old->bt = ast_bt_destroy(old->bt);
free(old);
}
}
Modified: trunk/main/astobj2.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/astobj2.c?view=diff&rev=397570&r1=397569&r2=397570
==============================================================================
--- trunk/main/astobj2.c (original)
+++ trunk/main/astobj2.c Fri Aug 23 13:07:40 2013
@@ -132,7 +132,7 @@
for(i = 0; i < c; i++) {
ast_verbose("%d: %p %s\n", i, addresses[i], strings[i]);
}
- free(strings);
+ ast_std_free(strings);
}
#endif
Modified: trunk/main/backtrace.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/backtrace.c?view=diff&rev=397570&r1=397569&r2=397570
==============================================================================
--- trunk/main/backtrace.c (original)
+++ trunk/main/backtrace.c Fri Aug 23 13:07:40 2013
@@ -40,17 +40,10 @@
#include <bfd.h>
#endif
-/* Undefine the overrides for memory allocation. astmm.c uses these functions
- * as well.
- */
-#undef calloc
-#undef malloc
-#undef free
-#undef realloc
-
struct ast_bt *__ast_bt_create(void)
{
- struct ast_bt *bt = calloc(1, sizeof(*bt));
+ struct ast_bt *bt = ast_std_calloc(1, sizeof(*bt));
+
if (!bt) {
return NULL;
}
@@ -69,15 +62,15 @@
void *__ast_bt_destroy(struct ast_bt *bt)
{
- if (bt->alloced) {
- free(bt);
+ if (bt && bt->alloced) {
+ ast_std_free(bt);
}
return NULL;
}
char **__ast_bt_get_symbols(void **addresses, size_t num_frames)
{
- char **strings = NULL;
+ char **strings;
#if defined(BETTER_BACKTRACES)
int stackfr;
bfd *bfdobj; /* bfd.h */
@@ -97,9 +90,12 @@
#if defined(BETTER_BACKTRACES)
strings_size = num_frames * sizeof(*strings);
- eachlen = calloc(num_frames, sizeof(*eachlen));
-
- if (!(strings = calloc(num_frames, sizeof(*strings)))) {
+
+ eachlen = ast_std_calloc(num_frames, sizeof(*eachlen));
+ strings = ast_std_calloc(num_frames, sizeof(*strings));
+ if (!eachlen || !strings) {
+ ast_std_free(eachlen);
+ ast_std_free(strings);
return NULL;
}
@@ -114,6 +110,7 @@
if (strcmp(dli.dli_fname, "asterisk") == 0) {
char asteriskpath[256];
+
if (!(dli.dli_fname = ast_utils_which("asterisk", asteriskpath, sizeof(asteriskpath)))) {
/* This will fail to find symbols */
dli.dli_fname = "asterisk";
@@ -121,22 +118,22 @@
}
lastslash = strrchr(dli.dli_fname, '/');
- if ( (bfdobj = bfd_openr(dli.dli_fname, NULL)) &&
- bfd_check_format(bfdobj, bfd_object) &&
- (allocsize = bfd_get_symtab_upper_bound(bfdobj)) > 0 &&
- (syms = malloc(allocsize)) &&
- (symbolcount = bfd_canonicalize_symtab(bfdobj, syms))) {
+ if ((bfdobj = bfd_openr(dli.dli_fname, NULL)) &&
+ bfd_check_format(bfdobj, bfd_object) &&
+ (allocsize = bfd_get_symtab_upper_bound(bfdobj)) > 0 &&
+ (syms = ast_std_malloc(allocsize)) &&
+ (symbolcount = bfd_canonicalize_symtab(bfdobj, syms))) {
if (bfdobj->flags & DYNAMIC) {
offset = addresses[stackfr] - dli.dli_fbase;
} else {
- offset = addresses[stackfr] - (void *)0;
+ offset = addresses[stackfr] - (void *) 0;
}
for (section = bfdobj->sections; section; section = section->next) {
- if ( !bfd_get_section_flags(bfdobj, section) & SEC_ALLOC ||
- section->vma > offset ||
- section->size + section->vma < offset) {
+ if (!bfd_get_section_flags(bfdobj, section) & SEC_ALLOC ||
+ section->vma > offset ||
+ section->size + section->vma < offset) {
continue;
}
@@ -151,7 +148,9 @@
found++;
if ((lastslash = strrchr(file, '/'))) {
const char *prevslash;
- for (prevslash = lastslash - 1; *prevslash != '/' && prevslash >= file; prevslash--);
+
+ for (prevslash = lastslash - 1; *prevslash != '/' && prevslash >= file; prevslash--) {
+ }
if (prevslash >= file) {
lastslash = prevslash;
}
@@ -173,7 +172,7 @@
}
if (bfdobj) {
bfd_close(bfdobj);
- free(syms);
+ ast_std_free(syms);
}
/* Default output, if we cannot find the information within BFD */
@@ -193,27 +192,31 @@
if (!ast_strlen_zero(msg)) {
char **tmp;
- eachlen[stackfr] = strlen(msg);
- if (!(tmp = realloc(strings, strings_size + eachlen[stackfr] + 1))) {
- free(strings);
+
+ eachlen[stackfr] = strlen(msg) + 1;
+ if (!(tmp = ast_std_realloc(strings, strings_size + eachlen[stackfr]))) {
+ ast_std_free(strings);
strings = NULL;
break; /* out of stack frame iteration */
}
strings = tmp;
strings[stackfr] = (char *) strings + strings_size;
- ast_copy_string(strings[stackfr], msg, eachlen[stackfr] + 1);
- strings_size += eachlen[stackfr] + 1;
+ strcpy(strings[stackfr], msg);/* Safe since we just allocated the room. */
+ strings_size += eachlen[stackfr];
}
}
if (strings) {
- /* Recalculate the offset pointers */
+ /* Recalculate the offset pointers because of the reallocs. */
strings[0] = (char *) strings + num_frames * sizeof(*strings);
for (stackfr = 1; stackfr < num_frames; stackfr++) {
- strings[stackfr] = strings[stackfr - 1] + eachlen[stackfr - 1] + 1;
- }
- }
+ strings[stackfr] = strings[stackfr - 1] + eachlen[stackfr - 1];
+ }
+ }
+ ast_std_free(eachlen);
+
#else /* !defined(BETTER_BACKTRACES) */
+
strings = backtrace_symbols(addresses, num_frames);
#endif /* defined(BETTER_BACKTRACES) */
return strings;
Modified: trunk/main/lock.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/lock.c?view=diff&rev=397570&r1=397569&r2=397570
==============================================================================
--- trunk/main/lock.c (original)
+++ trunk/main/lock.c Fri Aug 23 13:07:40 2013
@@ -29,6 +29,7 @@
ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
+#include "asterisk/utils.h"
#include "asterisk/lock.h"
/* Allow direct use of pthread_mutex_* / pthread_cond_* */
@@ -44,6 +45,22 @@
#undef pthread_cond_destroy
#undef pthread_cond_wait
#undef pthread_cond_timedwait
+
+#if defined(DEBUG_THREADS) && defined(HAVE_BKTR)
+static void __dump_backtrace(struct ast_bt *bt, int canlog)
+{
+ char **strings;
+ ssize_t i;
+
+ strings = backtrace_symbols(bt->addresses, bt->num_frames);
+
+ for (i = 0; i < bt->num_frames; i++) {
+ __ast_mutex_logger("%s\n", strings[i]);
+ }
+
+ ast_std_free(strings);
+}
+#endif /* defined(DEBUG_THREADS) && defined(HAVE_BKTR) */
int __ast_pthread_mutex_init(int tracking, const char *filename, int lineno, const char *func,
const char *mutex_name, ast_mutex_t *t)
Modified: trunk/main/logger.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/logger.c?view=diff&rev=397570&r1=397569&r2=397570
==============================================================================
--- trunk/main/logger.c (original)
+++ trunk/main/logger.c Fri Aug 23 13:07:40 2013
@@ -1587,9 +1587,7 @@
ast_debug(1, "#%d: [%p] %s\n", i - 3, bt->addresses[i], strings[i]);
}
- /* MALLOC_DEBUG will erroneously report an error here, unless we undef the macro. */
-#undef free
- free(strings);
+ ast_std_free(strings);
} else {
ast_debug(1, "Could not allocate memory for backtrace\n");
}
Modified: trunk/main/utils.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/utils.c?view=diff&rev=397570&r1=397569&r2=397570
==============================================================================
--- trunk/main/utils.c (original)
+++ trunk/main/utils.c Fri Aug 23 13:07:40 2013
@@ -862,9 +862,8 @@
for (frame_iterator = 0; frame_iterator < num_frames; ++frame_iterator) {
ast_str_append(str, 0, "\t%s\n", symbols[frame_iterator]);
}
-/* Prevent MALLOC_DEBUG from complaining */
-#undef free
- free(symbols);
+
+ ast_std_free(symbols);
} else {
ast_str_append(str, 0, "\tCouldn't retrieve backtrace symbols\n");
}
More information about the asterisk-commits
mailing list