From 3cba7058084175c2a9940fa33e4867c4c0a05314 Mon Sep 17 00:00:00 2001
From: Jeremy McNamara <jj@nufone.net>
Date: Thu, 15 Sep 2005 03:26:02 +0000
Subject: [PATCH] Lets try this - an attempt to properly deal with DTMF

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@6615 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---
 channels/chan_h323.c | 28 ++++++++++++++++++----------
 1 file changed, 18 insertions(+), 10 deletions(-)

diff --git a/channels/chan_h323.c b/channels/chan_h323.c
index c2f9571361..302540da89 100755
--- a/channels/chan_h323.c
+++ b/channels/chan_h323.c
@@ -347,22 +347,30 @@ static int oh323_digit(struct ast_channel *c, char digit)
 {
 	struct oh323_pvt *pvt = (struct oh323_pvt *) c->tech_pvt;
 	char *token;
-	if (h323debug)
-		ast_log(LOG_DEBUG, "Sending digit %c on %s\n", digit, c->name);
-	if (!pvt)
+
+	if (!pvt) {
+		ast_log(LOG_ERROR, "No private structure?! This is bad\n";
 		return -1;
+	}
 	ast_mutex_lock(&pvt->lock);
 	if (pvt->rtp && (pvt->options.dtmfmode & H323_DTMF_RFC2833)) {
+		/* out-of-band DTMF */
+		if (h323debug) {
+			ast_log(LOG_DEBUG, "Sending out-of-band digit %c on %s\n", digit, c->name);
+		}
 		ast_rtp_senddigit(pvt->rtp, digit);
+	} else {
+		/* in-band DTMF */
+		if (h323debug) {
+			ast_log(LOG_DEBUG, "Sending inband digit %c on %s\n", digit, c->name);
+		}
+		token = pvt->cd.call_token ? strdup(pvt->cd.call_token) : NULL;
+		h323_send_tone(token, digit);
+		if (token) {
+			free(token);
+		}
 	}
-	/* If in-band DTMF is desired, send that */
-	if (h323debug)
-		ast_log(LOG_DEBUG, "Sending digit  %c on %s\n", digit, c->name);
-	token = pvt->cd.call_token ? strdup(pvt->cd.call_token) : NULL;
 	ast_mutex_unlock(&pvt->lock);
-	h323_send_tone(token, digit);
-	if (token)
-		free(token);
 	oh323_update_info(c);
 	return 0;
 }