[asterisk-commits] kmoore: branch 1.8 r377655 - /branches/1.8/res/res_fax.c

[SVN commits to the Asterisk project]

Author: kmoore
Date: Mon Dec 10 10:51:35 2012
New Revision: 377655

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=377655
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

Modified:
    branches/1.8/res/res_fax.c

Modified: branches/1.8/res/res_fax.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/res/res_fax.c?view=diff&rev=377655&r1=377654&r2=377655
==============================================================================
--- branches/1.8/res/res_fax.c (original)
+++ branches/1.8/res/res_fax.c Mon Dec 10 10:51:35 2012
@@ -1702,6 +1702,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) {
@@ -1764,8 +1765,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) {