Summary: | ASTERISK-02859: V23 CID hangs on polarity reversal that is not a RING | ||
Reporter: | ian willoughby (ian willoughby) | Labels: | |
Date Opened: | 2004-11-19 03:30:58.000-0600 | Date Closed: | 2008-01-15 15:14:24.000-0600 |
Priority: | Major | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ||
Description: | My telco daily drops the line for around 5 seconds around minight resulting in a polarity reversal and NO_BATTERY. When using polarity reversal CID and V23 signalling in the UK chan_zap enters CID resolution on this polarity reversal but never leaves. This renders the Zap channel unusable unless a true incomming call is received thus freeing up the loop in CID resolution. ****** ADDITIONAL INFORMATION ****** I have manually changed the code in the CID_SIG_V23 handing block to add the additional break below:- if (i & ZT_IOMUX_SIGEVENT) { res = zt_get_event(p->subs[index].zfd); ast_log(LOG_NOTICE, "Got event %d (%s)...\n", res, event2str(res)); res = 0; break; //I added this line } else if (i & ZT_IOMUX_READ) { This seems to cure the hanging loop in disconnect of the line and still enable callerid to work here in the UK. | ||
Comments: | By: Mark Spencer (markster) 2004-11-19 08:44:53.000-0600 I'm happy with that fix. In principle no event should occur during the callerid spill. Fixed in CVS head. By: Russell Bryant (russell) 2004-11-25 02:10:09.000-0600 If I remember correctly, polarity reversal was a post-1.0 feature. It is 3 AM, so if I'm wrong, please re-open this bug so I can fix it in 1.0 as well. Thanks! By: Digium Subversion (svnbot) 2008-01-15 15:14:24.000-0600 Repository: asterisk Revision: 4296 U trunk/channels/chan_zap.c ------------------------------------------------------------------------ r4296 | markster | 2008-01-15 15:14:24 -0600 (Tue, 15 Jan 2008) | 2 lines Fix V23 callerid handling if an event is received (bug ASTERISK-2859) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=4296 |