[Home]

Summary:ASTERISK-17152: Empty Redirection Number in outgoing SETUP when calling from IAX
Reporter:Birger "WIMPy" Harzenetter (wimpy)Labels:
Date Opened:2010-12-23 00:33:01.000-0600Date Closed:2011-12-20 17:09:39.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Channels/chan_dahdi
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) chan_iax2.c.patch.txt
( 1) emptyredir
Description:When a call coming in via IAX calls out via dahdi, chan_dahdi inserts an empty Redirection Number in to the outgoing SETUP message.


****** ADDITIONAL INFORMATION ******

It does not happen when using channel originate or from dahdi to dahdi.
No other channeltypes in use on that box ATM.

The presence of that IE does no harm but it gets a cause 63 IE/parameter non existent or not implemented.
Comments:By: Richard Mudgett (rmudgett) 2010-12-23 14:47:44.000-0600

wimpy:
In the future, could you not provide intense debug output for problems at the Q.931 level.  The Q.921 output really clutters the trace.
By: Birger "WIMPy" Harzenetter (wimpy) 2010-12-23 15:14:24.000-0600

Oh, sorry. I need intense because of the full hex dumps.
I feed them in to another decoder to make them more readable.
By: Birger "WIMPy" Harzenetter (wimpy) 2010-12-25 00:32:42.000-0600

Ok, I did a little patch to 'pri set debug' that should give easier reading in the future.
See ASTERISK-17159 PRI-119
By: yarique (yarique) 2011-12-13 03:41:00.020-0600

Some telco carriers appear to really dislike that empty redirection number and reject the call.

The bug is in chan_iax2.c where rdnis is set on the call unconditionally.

Please review the attached patch and consider committing it.  It worked for me.

Thanks!
By: yarique (yarique) 2011-12-13 04:03:36.576-0600

I should have mentioned that my patch was against 1.8.8.0-rc4.  It would be great if it could make its way into 1.8.8.0 as this problem severity isn't as minor as believed at first.
By: Richard Mudgett (rmudgett) 2011-12-13 11:53:25.767-0600

You need to sign a contributor license agreement to post a patch.
Please resubmit after signing a license agreement.

Thanks

It is too late for the fix to make it into v1.8.8.
By: yarique (yarique) 2011-12-13 16:01:43.543-0600

Richard,

I'm afraid I need to speak with my legal advisor first before I can sign your license agreement.  The patch is as simple as wrapping 2 existing lines of code in an if{} block very similar to the one a few lines above, so please feel free to reimplement the fix without using my patch.

Thanks.