[asterisk-commits] kmoore: branch 10 r377656 - in /branches/10: ./ res/res_fax.c

[SVN commits to the Asterisk project]

Author: kmoore
Date: Mon Dec 10 10:53:16 2012
New Revision: 377656

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=377656
Log:
Ensure ReceiveFax provides a CED tone via T.38

When using res_fax_digium, the T.38 CED tone was not being provided
properly which would cause some incoming faxes to fail. This was not an
issue with res_fax_spandsp since it does not strictly honor the
send_ced flag and sends the CED tone whenever receiving a T.38 fax.

(closes issue FAX-343)
Reported-by: Benjamin Tietz
Patch-by: Kinsey Moore
........

Merged revisions 377655 from http://svn.asterisk.org/svn/asterisk/branches/1.8

Modified:
    branches/10/   (props changed)
    branches/10/res/res_fax.c

Propchange: branches/10/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: branches/10/res/res_fax.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/res/res_fax.c?view=diff&rev=377656&r1=377655&r2=377656
==============================================================================
--- branches/10/res/res_fax.c (original)
+++ branches/10/res/res_fax.c Mon Dec 10 10:53:16 2012
@@ -1837,6 +1837,7 @@
 	ast_verb(3, "Channel '%s' receiving FAX '%s'\n", chan->name, args.filename);
 
 	details->caps = AST_FAX_TECH_RECEIVE;
+	details->option.send_ced = AST_FAX_OPTFLAG_TRUE;
 
 	/* check for debug */
 	if (ast_test_flag(&opts, OPT_DEBUG) || global_fax_debug) {
@@ -1901,8 +1902,6 @@
 			ast_log(LOG_ERROR, "error initializing channel '%s' in T.38 mode\n", chan->name);
 			return -1;
 		}
-	} else {
-		details->option.send_ced = 1;
 	}
 
 	if ((channel_alive = generic_fax_exec(chan, details, s, token)) < 0) {