[Home]

Summary:ASTERISK-04323: ZapRAS does not work when used from Call File to place a call over Zaptel PRI
Reporter:Josef Bichlmeier (josy)Labels:
Date Opened:2005-06-02 10:10:28Date Closed:2011-06-07 14:03:17
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:Using a call file and placing following lines into it:
Channel: Zap/r1d/03731301181
Callerid: 8137935112
MaxRetries: 0
Application: ZapRAS
Data: -detach|mp|debug|maxconnect|3600|name|caminst|ipparam|01

This should bring up ZapRAS and enable passing of digital data through the
Zap Channel. I am using a T100P PRI card with 30 channels (Euro E1).

The digital indication gets lost inside chan_zap.c.
The following code on the line 6940 in chan_zap.c fixes the problem:

6935       } else if (opt == 'd') {
6936                    /* If this is an ISDN call, make it digital */
6937                    p->digital = 1;
6938
6939 /* included to pass through digital state of call option d from app_dial(), because gets lost */
6940                    ast_set_flag(tmp,AST_FLAG_DIGITAL);
6941
6942       } else {

I was using this fix against CVS Stable and now I use it against v1-0-7 and both work.

Comments:By: Mark Spencer (markster) 2005-06-02 13:13:55

Fixed in CVS head, thanks!
By: Michael Jerris (mikej) 2005-06-17 19:24:51

Looking in current stable, there is:
/* If this is an ISDN call, make it digital */
p->digital = 1;
if (tmp)
ast_set_flag(tmp, AST_FLAG_DIGITAL);

So not sure how, but this fix seems to already be in stable.
By: Digium Subversion (svnbot) 2008-01-15 15:36:43.000-0600

Repository: asterisk
Revision: 5813

U   trunk/channels/chan_zap.c

------------------------------------------------------------------------
r5813 | markster | 2008-01-15 15:36:43 -0600 (Tue, 15 Jan 2008) | 2 lines

Setup transfer capability properly for outbound calls (bug ASTERISK-4323)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=5813