From: Kinsey Moore <kmoore@digium.com>
Date: Mon, 10 Dec 2012 16:56:37 +0000 (+0000)
Subject: Ensure ReceiveFax provides a CED tone via T.38
X-Git-Tag: 13.0.0-beta1~2295
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=34cbefe62fc9851a03aaa3ace803b6c21b68fc25;p=thirdparty%2Fasterisk.git

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
........

Merged revisions 377656 from http://svn.asterisk.org/svn/asterisk/branches/10
........

Merged revisions 377657 from http://svn.asterisk.org/svn/asterisk/branches/11


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@377658 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---

diff --git a/res/res_fax.c b/res/res_fax.c
index 5ed7107d2a..9afad4a19c 100644
--- a/res/res_fax.c
+++ b/res/res_fax.c
@@ -1871,6 +1871,7 @@ static int receivefax_exec(struct ast_channel *chan, const char *data)
 	ast_verb(3, "Channel '%s' receiving FAX '%s'\n", ast_channel_name(chan), 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) {
@@ -1935,8 +1936,6 @@ static int receivefax_exec(struct ast_channel *chan, const char *data)
 			ast_log(LOG_ERROR, "error initializing channel '%s' in T.38 mode\n", ast_channel_name(chan));
 			return -1;
 		}
-	} else {
-		details->option.send_ced = 1;
 	}
 
 	if ((channel_alive = generic_fax_exec(chan, details, s, token)) < 0) {