[asterisk-dev] [Code Review] Changes from Mantis ID 13495 in trunk.

[rmudgett]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1676/#review6994
-----------------------------------------------------------


I also committed the SS7-27_libss7_trunk2_v2.diff for libss7 that fixes the GRS message loop.

The trunk libss7 has become chatty with all the unconditional timer start/stop messages.  There are many other new ss7_message() calls that are not conditional on debug output being turned on.  I expect a subsequent patch will quiet these messages.

We're getting closer. :-)


trunk/channels/sig_ss7.c
<https://reviewboard.asterisk.org/r/1676/#comment13599>

    This assertion fails on the next call on this channel if the previous call hangs up before getting any response.



trunk/configs/ss7.timers.sample
<https://reviewboard.asterisk.org/r/1676/#comment13598>

    I cannot find the definition for this timer in ITU.  It is really the overlap digit timeout and must always be defined (Even for ANSI).
    
    If your dialplan has extension 100 and extension 1000 and an incoming call calls 100, then the call will not go through and the channel will get stuck in the allocated state. (p->call_level = SIG_SS7_CALL_LEVEL_ALLOCATED)
    
    Also if the incoming call hangs up before this timer expires, the channel will be stuck because the call was never passed to Asterisk core.
    
    Another incoming call on this circuit will clear this stuck condition if it gets passed to the Asterisk core.  However, it will have an assertion failure in the IAM processing about the channel not being idle.
    ast_assert(!p->owner && p->call_level == SIG_SS7_CALL_LEVEL_IDLE)


- rmudgett


On Aug. 31, 2012, 10:36 a.m., KNK wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1676/
> -----------------------------------------------------------
> 
> (Updated Aug. 31, 2012, 10:36 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> chan_dahdi / sig_ss7 part of changes
> 
> 
> This addresses bug SS7-27.
>     https://issues.asterisk.org/jira/browse/SS7-27
> 
> 
> Diffs
> -----
> 
>   trunk/channels/chan_dahdi.c 372115 
>   trunk/channels/sig_ss7.h 372115 
>   trunk/channels/sig_ss7.c 372115 
>   trunk/configs/chan_dahdi.conf.sample 372115 
>   trunk/configs/ss7.timers.sample PRE-CREATION 
>   trunk/configure.ac 372115 
> 
> Diff: https://reviewboard.asterisk.org/r/1676/diff
> 
> 
> Testing
> -------
> 
> compiles, link setup, cli commands, bassic calls, connected line and redirection.
> 
> 
> Thanks,
> 
> KNK
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120831/2c6e4245/attachment.htm>