[asterisk-commits] seanbright: trunk r204749 - in /trunk: ./ channels/ configs/ funcs/
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Thu Jul 2 12:46:19 CDT 2009
Author: seanbright
Date: Thu Jul 2 12:46:14 2009
New Revision: 204749
URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=204749
Log:
Support setting and receiving Reverse Charging Indication over ISDN PRI.
This is a continuation of revision 885 to LibPRI (Capture and expose the Reverse
Charging Indication IE on ISDN PRI) which added the ability to get/set Reverse
Charging Indication in LibPRI. This patch adds the ability to specify RCI on
the outbound leg of a PRI call from within Asterisk, by prefixing the dialed
number with a capital 'C' like:
...,Dial(DAHDI/g1/C4445556666)
And to read it off an inbound channel:
exten => s,1,Set(RCI=${CHANNEL(reversecharge)})
Thanks again to rmudgett for the thorough review.
(closes issue #13760)
Reported by: mrgabu
Review: https://reviewboard.asterisk.org/r/303/
Modified:
trunk/CHANGES
trunk/channels/chan_dahdi.c
trunk/channels/sig_pri.c
trunk/channels/sig_pri.h
trunk/configs/chan_dahdi.conf.sample
trunk/funcs/func_channel.c
Modified: trunk/CHANGES
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/CHANGES?view=diff&rev=204749&r1=204748&r2=204749
==============================================================================
--- trunk/CHANGES (original)
+++ trunk/CHANGES Thu Jul 2 12:46:14 2009
@@ -46,6 +46,11 @@
-----------
* Added rtsavesysname option into iax.conf to allow the systname to be saved
on realtime updates.
+
+DAHDI Changes
+-------------
+ * Added Reverse Charging Indication receipt & transmission (requires latest
+ LibPRI).
Applications
------------
Modified: trunk/channels/chan_dahdi.c
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/channels/chan_dahdi.c?view=diff&rev=204749&r1=204748&r2=204749
==============================================================================
--- trunk/channels/chan_dahdi.c (original)
+++ trunk/channels/chan_dahdi.c Thu Jul 2 12:46:14 2009
@@ -5403,7 +5403,7 @@
{
struct dahdi_pvt *p = chan->tech_pvt;
int res = 0;
-
+
if (!strcasecmp(data, "rxgain")) {
ast_mutex_lock(&p->lock);
snprintf(buf, len, "%f", p->rxgain);
@@ -5412,8 +5412,19 @@
ast_mutex_lock(&p->lock);
snprintf(buf, len, "%f", p->txgain);
ast_mutex_unlock(&p->lock);
+#ifdef HAVE_PRI
+ } else if (!strcasecmp(data, "reversecharge")) {
+ ast_mutex_lock(&p->lock);
+ if (p->sig == SIG_PRI || p->sig == SIG_BRI || p->sig == SIG_BRI_PTMP) {
+ snprintf(buf, len, "%d", ((struct sig_pri_chan *) p->sig_pvt)->reverse_charging_indication);
+ } else {
+ *buf = '\0';
+ res = -1;
+ }
+ ast_mutex_unlock(&p->lock);
+#endif
} else {
- ast_copy_string(buf, "", len);
+ *buf = '\0';
res = -1;
}
Modified: trunk/channels/sig_pri.c
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/channels/sig_pri.c?view=diff&rev=204749&r1=204748&r2=204749
==============================================================================
--- trunk/channels/sig_pri.c (original)
+++ trunk/channels/sig_pri.c Thu Jul 2 12:46:14 2009
@@ -1163,7 +1163,8 @@
pbx_builtin_setvar_helper(c, "CALLEDTON", calledtonstr);
if (e->ring.redirectingreason >= 0)
pbx_builtin_setvar_helper(c, "PRIREDIRECTREASON", redirectingreason2str(e->ring.redirectingreason));
-
+ pri->pvts[chanpos]->reverse_charging_indication = e->ring.reversecharge;
+
sig_pri_lock_private(pri->pvts[chanpos]);
ast_mutex_lock(&pri->lock);
@@ -1208,7 +1209,8 @@
if (e->ring.redirectingreason >= 0)
pbx_builtin_setvar_helper(c, "PRIREDIRECTREASON", redirectingreason2str(e->ring.redirectingreason));
-
+ pri->pvts[chanpos]->reverse_charging_indication = e->ring.reversecharge;
+
snprintf(calledtonstr, sizeof(calledtonstr)-1, "%d", e->ring.calledplan);
pbx_builtin_setvar_helper(c, "CALLEDTON", calledtonstr);
@@ -1888,6 +1890,11 @@
case 'r':
pridialplan = PRI_NPI_RESERVED | (pridialplan & 0xf0);
break;
+#if defined(PRI_REVERSECHARGE_REQUESTED)
+ case 'C':
+ pri_sr_set_reversecharge(sr, PRI_REVERSECHARGE_REQUESTED);
+ break;
+#endif
default:
if (isalpha(c[p->stripmsd])) {
ast_log(LOG_WARNING, "Unrecognized pridialplan %s modifier: %c\n",
Modified: trunk/channels/sig_pri.h
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/channels/sig_pri.h?view=diff&rev=204749&r1=204748&r2=204749
==============================================================================
--- trunk/channels/sig_pri.h (original)
+++ trunk/channels/sig_pri.h Thu Jul 2 12:46:14 2009
@@ -162,6 +162,7 @@
struct sig_pri_callback *calls;
void *chan_pvt;
ast_mutex_t service_lock; /*!< Mutex for service messages */
+ int reverse_charging_indication;
};
struct sig_pri_pri {
Modified: trunk/configs/chan_dahdi.conf.sample
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/configs/chan_dahdi.conf.sample?view=diff&rev=204749&r1=204748&r2=204749
==============================================================================
--- trunk/configs/chan_dahdi.conf.sample (original)
+++ trunk/configs/chan_dahdi.conf.sample Thu Jul 2 12:46:14 2009
@@ -76,6 +76,11 @@
;
;service_message_support=yes
; Enable service message support for channel. Must be set after switchtype.
+;
+; PRI Reverse Charging Indication: Indicate to the called party that the
+; call will be reverse charged. To enable, prefix the dialed number with one
+; of the following letters:
+; C - Reverse Charge Indication Requested
;
; PRI Dialplan: The ISDN-level Type Of Number (TON) or numbering plan, used for
; the dialed number. For most installations, leaving this as 'unknown' (the
Modified: trunk/funcs/func_channel.c
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/funcs/func_channel.c?view=diff&rev=204749&r1=204748&r2=204749
==============================================================================
--- trunk/funcs/func_channel.c (original)
+++ trunk/funcs/func_channel.c Thu Jul 2 12:46:14 2009
@@ -205,6 +205,16 @@
<para>R/O Get the peer's username.</para>
</enum>
</enumlist>
+ <para><emphasis>chan_dahdi</emphasis> provides the following additional options:</para>
+ <enumlist>
+ <enum name="reversecharge">
+ <para>R/O Reverse Charging Indication, one of:</para>
+ <enumlist>
+ <enum name="-1 - None" />
+ <enum name="1 - Reverse Charging Requested" />
+ </enumlist>
+ </enum>
+ </enumlist>
</parameter>
</syntax>
<description>
More information about the asterisk-commits
mailing list