[asterisk-bugs] [JIRA] (SS7-39) Wrong program logic (LFU will NOT be sent)

[Rusty Newton (JIRA)]

     [ https://issues.asterisk.org/jira/browse/SS7-39?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Rusty Newton closed SS7-39.
---------------------------

    Resolution: Fixed

Fixed in [LibSS7 2.0.0|http://downloads.asterisk.org/pub/telephony/libss7/ChangeLog-2.0.0]

> Wrong program logic (LFU will NOT be sent)
> ------------------------------------------
>
>                 Key: SS7-39
>                 URL: https://issues.asterisk.org/jira/browse/SS7-39
>             Project: LibSS7
>          Issue Type: Bug
>          Components: General
>            Reporter: tian
>
> Version SVN
> SVN Trunk
> SVN Revision 269
> Source file: mtp3.c
> Lines: 452-464
> /* try force uninhibit */
> if (i == ss7->numlinks) {
>     for (i = 0; i < ss7->numlinks; i++) {
>         if (ss7->links[i]->inhibit & INHIBITED_REMOTELY) {
>             if (!(ss7->links[i]->got_sent_netmsg & SENT_LFU))
>                 break;  // **** This break should be deleted! ****
>             AUTORL(rl, ss7->links[i]);
>             net_mng_send(ss7->links[i], NET_MNG_LFU, rl, 0);
>             ss7_message(ss7, "Forced uninhibiting remotely inhibited link 
> (no more signalling links are in service) SLC: %i ADJPC: %i\n", 
> ss7->links[i]->slc, ss7->links[i]->dpc);
>             break;
>         }
>     }
> }
> Bug description: the first 'break' keyword is misused and should be deleted! 
> Apparently the logic of the program is to check whether a LFU has been sent 
> to the far end, and if it is not the case, send a LFU, but the 'break' 
> breaks the logic.
> ****** ADDITIONAL INFORMATION ******
> I am only reading the source code without a running Asterisk box.



--
This message was sent by Atlassian JIRA
(v6.2#6252)