[asterisk-bugs] [LibSS7 0015566]: Statement after 'return' (link->adj_sp->state will not be set correctly)

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15566 
====================================================================== 
Reported By:                Tian00di00
Assigned To:                mattf
====================================================================== 
Project:                    LibSS7
Issue ID:                   15566
Category:                   General
Reproducibility:            have not tried
Severity:                   major
Priority:                   normal
Status:                     assigned
Asterisk Version:           1.6.0.10 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 269 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2009-07-24 03:34 CDT
Last Modified:              2009-08-03 02:59 CDT
====================================================================== 
Summary:                    Statement after 'return' (link->adj_sp->state will
not be set correctly)
Description: 
Version SVN
SVN Trunk
SVN Revision 269

Source file: mtp3.c

Lines: 707-708

static int mtp3_init(struct mtp2 *link)
{
    int res;
    struct adjacent_sp *adj_sp = link->adj_sp;

    if (adj_sp->state == MTP3_DOWN) {
        AUTORL(rl, link);
        if (link->inhibit & INHIBITED_LOCALLY) {
            return net_mng_send(link, NET_MNG_LIN, rl, 0);
            link->adj_sp->state = MTP3_ALIGN;  // ***This line will NEVER
be executed?****
        } else if (!(adj_sp->tra & SENT)) {
            res = net_mng_send(link, NET_MNG_TRA, rl, 0);
            mtp3_check(adj_sp);
            return res;
        }
    }
    return 0;
}

Bug description: the line after 'return net_mng_send(link, NET_MNG_LIN,
rl, 0);
' will NEVER be excecuted!
====================================================================== 

---------------------------------------------------------------------- 
 (0108525) adomjan (reporter) - 2009-08-03 02:59
 https://issues.asterisk.org/view.php?id=15566#c108525 
---------------------------------------------------------------------- 
yes, you are right!
I will turn on the test envinroment when I will have time, and I will test
it 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-08-03 02:59 adomjan        Note Added: 0108525                          
======================================================================