[asterisk-bugs] [Asterisk 0013020]: [patch] better interoperability for app_fax

[noreply at bugs.digium.com]

The following issue has been ASSIGNED. 
====================================================================== 
http://bugs.digium.com/view.php?id=13020 
====================================================================== 
Reported By:                dimas
Assigned To:                russell
====================================================================== 
Project:                    Asterisk
Issue ID:                   13020
Category:                   Applications/General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     assigned
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 128495 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             07-08-2008 08:12 CDT
Last Modified:              07-08-2008 09:10 CDT
====================================================================== 
Summary:                    [patch] better interoperability for app_fax
Description: 
The patch fixes two issues

1. SpanDSP 0.0.5 now requires a call to call t38_terminal_release which is
not in the original app_fax

2. do actually call XXX_terminate SpanDSP API calls when finish. This is
to make sur Phase E handler actually gets called with proper status. This
is because there are a lot of softphones which after _successful_ sending
of a fax either re-INVITE without T38 immediately or even hangup the call
causing transmission loop to abort with error although the fax was actually
transferred Ok
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
has duplicate       0013021 [patch] better interoperability for app...
====================================================================== 

---------------------------------------------------------------------- 
 svnbot - 07-08-08 09:10  
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 129006

U   trunk/apps/app_fax.c

------------------------------------------------------------------------
r129006 | russell | 2008-07-08 09:10:12 -0500 (Tue, 08 Jul 2008) | 9 lines

Update app_fax for better compatibility with spandsp 0.0.5.  Add a call to
t38_terminal_release, and make sure that the phase E handler gets called
with proper status.

(closes issue http://bugs.digium.com/view.php?id=13020)
Reported by: dimas
Patches:
      v1-appfax.patch uploaded by dimas (license 88)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=129006 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
07-08-08 09:10  svnbot         Note Added: 0089887                          
07-08-08 09:10  svnbot         Status                   new => assigned     
07-08-08 09:10  svnbot         Assigned To               => russell         
======================================================================