[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/#review6978
-----------------------------------------------------------


I have committed Diff 13 plus fixing the finding in handle_ss7_linkset_mng() to the ss7_27_knk branch.

However, even with the SS7-27_libss7_trunk2_v1.diff patch from SS7-27 applied to libss7 I am still getting the message loop.  You should to setup the local DAHDI spans to replicate.  I described how to do that when I initially reported the problem.  Local DAHDI spans do not need any hardware.

I am concerned that the libss7 trunk code is now including non-call related status information in the struct isup_call.  The struct seems to be more like it should be called isup_circuit and thus should never be released unless the circuit is no longer provisioned.  The got_sent_msg and timer[] members are the principle reasons that isup_call is no longer call related but circuit related.



trunk/channels/chan_dahdi.c
<https://reviewboard.asterisk.org/r/1676/#comment13582>

    If sig_ss7_reset_cic() doesn't fail it will then do a sig_ss7_cic_blocking() with do_block==-1.
    
    I have fixed this in the ss7_27_knk branch.


- rmudgett


On Aug. 20, 2012, 6:35 a.m., KNK wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1676/
> -----------------------------------------------------------
> 
> (Updated Aug. 20, 2012, 6:35 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 371534 
>   trunk/channels/sig_ss7.h 371534 
>   trunk/channels/sig_ss7.c 371534 
>   trunk/configs/chan_dahdi.conf.sample 371534 
>   trunk/configs/ss7.timers.sample PRE-CREATION 
>   trunk/configure.ac 371534 
> 
> 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/20120830/685bcac9/attachment.htm>