[asterisk-commits] coreyfarrell: trunk r426253 - in /trunk: ./ funcs/func_cdr.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Oct 28 06:12:06 CDT 2014


Author: coreyfarrell
Date: Tue Oct 28 06:12:03 2014
New Revision: 426253

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=426253
Log:
func_cdr: Fix CDR_PROP payload leak

Remove duplicate allocation of payload, preventing leak.

ASTERISK-24455 #close
Reported by: Corey Farrell
Review: https://reviewboard.asterisk.org/r/4113/
........

Merged revisions 426252 from http://svn.asterisk.org/svn/asterisk/branches/13

Modified:
    trunk/   (props changed)
    trunk/funcs/func_cdr.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-13-merged' - no diff available.

Modified: trunk/funcs/func_cdr.c
URL: http://svnview.digium.com/svn/asterisk/trunk/funcs/func_cdr.c?view=diff&rev=426253&r1=426252&r2=426253
==============================================================================
--- trunk/funcs/func_cdr.c (original)
+++ trunk/funcs/func_cdr.c Tue Oct 28 06:12:03 2014
@@ -540,8 +540,7 @@
 		     const char *value)
 {
 	RAII_VAR(struct stasis_message *, message, NULL, ao2_cleanup);
-	RAII_VAR(struct cdr_func_payload *, payload,
-		ao2_alloc(sizeof(*payload), NULL), ao2_cleanup);
+	RAII_VAR(struct cdr_func_payload *, payload, NULL, ao2_cleanup);
 	RAII_VAR(struct stasis_message_router *, router, ast_cdr_message_router(), ao2_cleanup);
 
 	if (!chan) {




More information about the asterisk-commits mailing list