[Home]

Summary:ASTERISK-00954: [patch] data calls through * with EuroISDN E1
Reporter:thaeger (thaeger)Labels:
Date Opened:2004-01-30 11:31:04.000-0600Date Closed:2008-02-18 14:21:51.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) q931_digi.patch
( 1) q931_Digital_Data.patch
Description:Hi all,

if you get an incoming call with bearer capability 0x08 (64k data) and do a "Dial(Zap/g2d/${EXTEN}" to route the data call to another data endpoint, maybe (for us it happens with data calls through a Teles iSwitch) you get a RELEASE from the termination endpoint because following:

The incoming call (SETUP) comes with "Bearer Capability: 04 02 88 90"

The outgoing call (SETUP) from * is as following:

"Bearer Capability: 04 03 88 90 A3"

In this case you could get a RELEASE (cause: Incompatible Destination) immediately, because of the "A3" (User information layer 1) and the outgoing datacall fail. I don't know much about q931... but maybe the "A3" is wrong in this case....

Therefore i wrote a very little patch to except that this happens :-)

It would be nice if this patch goes into the main source branch...and feel free to correct me!

Regards,

Thomas.
Comments:By: Brian West (bkw918) 2004-02-02 00:33:11.000-0600

is this disclaimed yet?
By: thaeger (thaeger) 2004-02-02 03:20:59.000-0600

is what disclaimed ...?
By: Olle Johansson (oej) 2004-04-10 08:34:00

Thaeger: For patches to be included, you need to send a disclaimer to Digium. See http://bugs.digium.com for more information and forms. Fax it to Digium and then confirm here so we can move this patch forward quickly.
By: Olle Johansson (oej) 2004-04-12 04:28:17

Reminder...
By: Mark Spencer (markster) 2004-04-12 12:10:26

I need a "cvs diff -u" in order to evaluate this patch.
By: thaeger (thaeger) 2004-04-12 13:27:43

Hi Mark,

here the patch with -u.

Regards,

Thomas.
By: Olle Johansson (oej) 2004-04-12 14:02:53

And what about the disclaimer? Please update status - is this patch disclaimed (see above) ?
By: Mark Spencer (markster) 2004-04-12 14:43:23

I still don't see the diff -u.  I doubt a disclaimer is necessary for such a short patch.
By: thaeger (thaeger) 2004-04-12 15:02:23

I agree with Mark that no disclaimer is necessary.
But the patch is made with "cvs diff -u q931.c".... oh i see.
The upload failed... once again...
By: Mark Spencer (markster) 2004-04-16 14:13:50

Okay it's in CVS.  I hope nobody gets upset!
By: Digium Subversion (svnbot) 2008-02-18 14:21:51.000-0600

Repository: libpri
Revision: 521

U   branches/1.2/libpri.h
U   branches/1.2/q931.c

------------------------------------------------------------------------
r521 | mattf | 2008-02-18 14:21:50 -0600 (Mon, 18 Feb 2008) | 1 line

Fix for bug ASTERISK-954.  Better support for more exotic bearer capabilities
------------------------------------------------------------------------

http://svn.digium.com/view/libpri?view=rev&revision=521