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

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
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:              05-06-2008 15:29 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)) {

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

---------------------------------------------------------------------- 
 qwell - 05-06-08 15:29  
---------------------------------------------------------------------- 
If you have a patch that you've tested, it would be useful for you to
upload it here, so we can review it and potentially commit. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
05-06-08 15:29  qwell          Note Added: 0086494                          
======================================================================