[asterisk-commits] mmichelson: branch 11 r423360 - /branches/11/res/res_fax_spandsp.c

[SVN commits to the Asterisk project]

Author: mmichelson
Date: Thu Sep 18 11:19:51 2014
New Revision: 423360

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=423360
Log:
res_fax_spandsp: Properly handle cleanup before starting FAXes.

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.

ASTERISK-24301 #close
Reported by Matt Jordan
Patches:
	ASTERISK-24301-fax.diff Uploaded by Mark Michelson (License #5049)



Modified:
    branches/11/res/res_fax_spandsp.c

Modified: branches/11/res/res_fax_spandsp.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/res/res_fax_spandsp.c?view=diff&rev=423360&r1=423359&r2=423360
==============================================================================
--- branches/11/res/res_fax_spandsp.c (original)
+++ branches/11/res/res_fax_spandsp.c Thu Sep 18 11:19:51 2014
@@ -176,8 +176,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);