List Info

Thread: isup sls bug (isup.c)




isup sls bug (isup.c)
user name
2007-05-15 03:38:19
hi.

There is a mistake in isup_send_message function of isup.c

rl.sls is assigned sls_next(ss7), assuming there are 16
signaling
links amd sls in incrementsd one after other.

There will be a problem if the number of signaling links are
less than 16.

sls_next may be modified so that 16 is replaced by
numlinks.

Rgds
sai







-- 
A.K.V.Sai jayram
Bangalore
_______________________________________________
--Bandwidth and Colocation provided by Easynews.com --

asterisk-ss7 mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-ss7

Re: isup sls bug (isup.c)
country flaguser name
United States
2007-05-15 14:15:20
On May 15, 2007, at 3:38 AM, sai jayram AKV wrote:

> hi.
>
> There is a mistake in isup_send_message function of
isup.c
>
> rl.sls is assigned sls_next(ss7), assuming there are 16
signaling
> links amd sls in incrementsd one after other.
>
> There will be a problem if the number of signaling
links are less than 
> 16.
>
> sls_next may be modified so that 16 is replaced by
numlinks.

I have seen many different possible ways of doing this.  I
have not 
seen a good answer yet for how this works.  It Q.763, it
says in the 
spec under section 1.1 (routing label) that the "SLS
bits are set to 
the four least significant bits of the CIC".  Can you
point to a 
specification or document that can verify your
recommendation?

It is not exactly correct the way it is written right now. 
For ANSI 
networks, it doesn't do SLS balancing properly across all
possible 
values, and for ITU, it maybe that it need be changed to
follow what 
the ITU recommendation says, instead of the cross ANSI/ITU
mix I have 
currently.

Matthew Fredrickson

_______________________________________________
--Bandwidth and Colocation provided by Easynews.com --

asterisk-ss7 mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-ss7

Re: isup sls bug (isup.c)
country flaguser name
United States
2007-05-15 15:02:56
On May 15, 2007, at 2:15 PM, Matthew Fredrickson wrote:

>
> On May 15, 2007, at 3:38 AM, sai jayram AKV wrote:
>
>> hi.
>>
>> There is a mistake in isup_send_message function of
isup.c
>>
>> rl.sls is assigned sls_next(ss7), assuming there
are 16 signaling
>> links amd sls in incrementsd one after other.
>>
>> There will be a problem if the number of signaling
links are less 
>> than 16.
>>
>> sls_next may be modified so that 16 is replaced by
numlinks.
>
> I have seen many different possible ways of doing this.
 I have not 
> seen a good answer yet for how this works.  It Q.763,
it says in the 
> spec under section 1.1 (routing label) that the
"SLS bits are set to 
> the four least significant bits of the CIC".  Can
you point to a 
> specification or document that can verify your
recommendation?
>
> It is not exactly correct the way it is written right
now.  For ANSI 
> networks, it doesn't do SLS balancing properly across
all possible 
> values, and for ITU, it maybe that it need be changed
to follow what 
> the ITU recommendation says, instead of the cross
ANSI/ITU mix I have 
> currently.

For what it is worth, after doing a little bit of research,
I updated 
it (for ITU SS7) to be in accordance with the specifications
(using the 
least four significant bits of the CIC) until I hear
something else.

Matthew Fredrickson

_______________________________________________
--Bandwidth and Colocation provided by Easynews.com --

asterisk-ss7 mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-ss7

[1-3]

about | contact  Other archives ( Real Estate discussion Medical topics )