[asterisk-commits] mnicholson: branch irroot/t38gateway-trunk r324465 - /team/irroot/t38gateway-...

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Wed Jun 22 08:15:11 CDT 2011


Author: mnicholson
Date: Wed Jun 22 08:15:08 2011
New Revision: 324465

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=324465
Log:
don't free frames passed to the framehook.  The bridge will free these later.

Modified:
    team/irroot/t38gateway-trunk/res/res_fax.c

Modified: team/irroot/t38gateway-trunk/res/res_fax.c
URL: http://svnview.digium.com/svn/asterisk/team/irroot/t38gateway-trunk/res/res_fax.c?view=diff&rev=324465&r1=324464&r2=324465
==============================================================================
--- team/irroot/t38gateway-trunk/res/res_fax.c (original)
+++ team/irroot/t38gateway-trunk/res/res_fax.c Wed Jun 22 08:15:08 2011
@@ -2416,7 +2416,6 @@
 				if ((activestate == T38_STATE_NEGOTIATING) || (activestate == T38_STATE_UNKNOWN)) {
 					ast_queue_control_data(inactive, AST_CONTROL_T38_PARAMETERS, active_param, sizeof(struct ast_control_t38_parameters));
 				}
-				ast_frfree(f);
 				f = &ast_null_frame;
 			} else {
 				/*im capable of T.38 make the paramaters mine*/
@@ -2574,7 +2573,6 @@
 	if (f->frametype == AST_FRAME_CONTROL && f->subclass.integer == AST_CONTROL_T38_PARAMETERS) {
 		if (s || (f->datalen != sizeof(struct ast_control_t38_parameters))) {
 			/* invalaid frame AST_CONTROL_T38_PARAMETERS frame */
-			ast_frfree(f);
 			return &ast_null_frame;
 		}
 
@@ -2647,7 +2645,6 @@
 			/* XXX perhaps we should check the return value of
 			 * tech->write() */
 			s->tech->write(s, f);
-			ast_frfree(f);
 			f = &ast_null_frame;
 			break;
 		default:




More information about the asterisk-commits mailing list