[asterisk-bugs] [LibSS7 0015563]: program path that'll never be executed!

[Asterisk Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15563 
====================================================================== 
Reported By:                Tian00di00
Assigned To:                mattf
====================================================================== 
Project:                    LibSS7
Issue ID:                   15563
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-23 22:52 CDT
Last Modified:              2009-07-23 22:52 CDT
====================================================================== 
Summary:                    program path that'll never be executed!
Description: 
Version SVN
SVN Trunk
SVN Revision 269

Source file: mtp3.c

Lines: 470-476

if (LINKSET_UP_DELAY > -1) { // #define LINKSET_UP_DELAY 500
    if (ss7->linkset_up_timer > -1)
        ss7_schedule_del(ss7, &ss7->linkset_up_timer);
    ss7->linkset_up_timer = ss7_schedule_event(ss7, LINKSET_UP_DELAY, 
&linkset_up_expired, ss7);
    ss7_message(ss7, "LINKSET UP DELAYING RESETTING\n");
} else // **** this clause will NEVER be excecuted!
    ss7_linkset_up_event(ss7);


Description of the bug: since LINKSET_UP_DELAY is #define'd to be 500, the

if condition is always true so the else clause will never be executed!
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-07-23 22:52 Tian00di00     New Issue                                    
2009-07-23 22:52 Tian00di00     Status                   new => assigned     
2009-07-23 22:52 Tian00di00     Assigned To               => mattf           
2009-07-23 22:52 Tian00di00     Asterisk Version          => 1.6.0.10        
2009-07-23 22:52 Tian00di00     SVN Branch (only for SVN checkouts, not tarball
releases) =>  trunk          
2009-07-23 22:52 Tian00di00     SVN Revision (number only!) => 269             
======================================================================