[Asterisk-code-review] Fix crash in codec lpc10 when MALLOC DEBUG is enabled. (asterisk[master])

Matt Jordan asteriskteam at digium.com
Fri May 8 23:33:36 CDT 2015


Matt Jordan has submitted this change and it was merged.

Change subject: Fix crash in codec_lpc10 when MALLOC_DEBUG is enabled.
......................................................................


Fix crash in codec_lpc10 when MALLOC_DEBUG is enabled.

This switches codecs/lpc10/lpcini.c back to including "asterisk.h"
instead of <stdlib.h>.  lpcini.c allocates memory that is freed by
codec_lpc10.c, so it is important to use MALLOC_DEBUG allocator.
Added #define WRAP_LIBC_MALLOC to the start of the source to prevent
runtime symbol link error's.

Change-Id: I74f63fd09fdeb673ee7753122c3bb4722ab6e1ac
---
M codecs/lpc10/lpcini.c
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Matt Jordan: Looks good to me, approved; Verified



diff --git a/codecs/lpc10/lpcini.c b/codecs/lpc10/lpcini.c
index ebe229a..8efb640 100644
--- a/codecs/lpc10/lpcini.c
+++ b/codecs/lpc10/lpcini.c
@@ -34,7 +34,8 @@
 	-lf2c -lm   (in that order)
 */
 
-#include <stdlib.h>
+#define WRAP_LIBC_MALLOC
+#include "asterisk.h"
 #include "f2c.h"
 
 #ifdef P_R_O_T_O_T_Y_P_E_S

-- 
To view, visit https://gerrit.asterisk.org/409
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I74f63fd09fdeb673ee7753122c3bb4722ab6e1ac
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list