[asterisk-bugs] [Asterisk 0012276]: hanguponpolarityswitch ignored because I am not using answeronpolarityswitch

[noreply at bugs.digium.com]

The following issue has been SUBMITTED. 
====================================================================== 
http://bugs.digium.com/view.php?id=12276 
====================================================================== 
Reported By:                benbrown
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   12276
Category:                   Channels/chan_zap
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           1.4.18 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             03-22-2008 07:55 CDT
Last Modified:              03-22-2008 07:55 CDT
====================================================================== 
Summary:                    hanguponpolarityswitch ignored because I am not
using answeronpolarityswitch
Description: 
As you can see from the debug below I am using hanguponpolarityswitch but
not answeronpolarityswitch.

However, the hangup polarity detection is being ignored, I believe,
because p->polaritydelaytv is not being initialised and calculating the
large negative delay.

Perhaps line 4478 should read:

 if ((p->answeronpolarityswitch) && (ast_tvdiff_ms(ast_tvnow(),
p->polaritydelaytv) > p->polarityonanswerdelay)) {

====================================================================== 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
03-22-08 07:55  benbrown       Asterisk Version          => 1.4.18          
03-22-08 07:55  benbrown       SVN Branch (only for SVN checkou => N/A          
  
======================================================================