[svn-commits] bebuild: tag certified-11.6-cert6 r423447 - in /certified/tags/11.6-cert6: ./...

[SVN commits to the Digium repositories]

Author: bebuild
Date: Thu Sep 18 12:34:31 2014
New Revision: 423447

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=423447
Log:
Merge changes for AST-2014-010

Modified:
    certified/tags/11.6-cert6/   (props changed)
    certified/tags/11.6-cert6/ChangeLog
    certified/tags/11.6-cert6/res/res_fax_spandsp.c

Propchange: certified/tags/11.6-cert6/
------------------------------------------------------------------------------
Binary property 'branch-11-merged' - no diff available.

Propchange: certified/tags/11.6-cert6/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Thu Sep 18 12:34:31 2014
@@ -1,2 +1,3 @@
 /branches/11:399513,401167,401179,401182,415825
 /certified/branches/1.8.15:382389
+/certified/branches/11.6:423426

Modified: certified/tags/11.6-cert6/ChangeLog
URL: http://svnview.digium.com/svn/asterisk/certified/tags/11.6-cert6/ChangeLog?view=diff&rev=423447&r1=423446&r2=423447
==============================================================================
--- certified/tags/11.6-cert6/ChangeLog (original)
+++ certified/tags/11.6-cert6/ChangeLog Thu Sep 18 12:34:31 2014
@@ -1,3 +1,24 @@
+2014-09-18  Asterisk Development Team <asteriskteam at digium.com>
+
+	* Certified Asterisk 11.6-cert6 Released.
+
+	* AST-2014-010: Resolve crash when the Message channel technology
+	  enters into the ReceiveFax application using res_fax_spandsp
+
+	  If faxing fails at a very early stage, then it is possible for
+	  us to pass a NULL t30 state pointer to spandsp, which spandsp
+	  is none too pleased with.
+
+	  This patch ensures that we pass the correct pointer to spandsp
+	  in the situation where we have not yet set our local t30 state
+	  pointer.
+
+	  An advisory was made for this issue due to the likelihood of
+	  it occurring in some Asterisk configurations.
+
+	  ASTERISK-24301 #close
+	  Reported by Matt Jordan, Philippe Lindheimer
+
 2014-09-05  Asterisk Development Team <asteriskteam at digium.com>
 
 	* Certified Asterisk 11.6-cert5 Released.

Modified: certified/tags/11.6-cert6/res/res_fax_spandsp.c
URL: http://svnview.digium.com/svn/asterisk/certified/tags/11.6-cert6/res/res_fax_spandsp.c?view=diff&rev=423447&r1=423446&r2=423447
==============================================================================
--- certified/tags/11.6-cert6/res/res_fax_spandsp.c (original)
+++ certified/tags/11.6-cert6/res/res_fax_spandsp.c Thu Sep 18 12:34:31 2014
@@ -177,8 +177,25 @@
 static void session_destroy(struct spandsp_pvt *p)
 {
 	struct ast_frame *f;
-
-	t30_terminate(p->t30_state);
+	t30_state_t *t30_to_terminate;
+
+	if (p->t30_state) {
+		t30_to_terminate = p->t30_state;
+	} else if (p->ist38) {
+#if SPANDSP_RELEASE_DATE >= 20080725
+		t30_to_terminate = &p->t38_state.t30;
+#else
+		t30_to_terminate = &p->t38_state.t30_state;
+#endif
+	} else {
+#if SPANDSP_RELEASE_DATE >= 20080725
+		t30_to_terminate = &p->fax_state.t30;
+#else
+		t30_to_terminate = &p->fax_state.t30_state;
+#endif
+	}
+
+	t30_terminate(t30_to_terminate);
 	p->isdone = 1;
 
 	ast_timer_close(p->timer);