Summary:	ASTERISK-06556: Zap channel may use wrong cidsignalling in ss_thread
Reporter:	Gareth Palmer (gareth)	Labels:
Date Opened:	2006-03-16 02:28:23.000-0600	Date Closed:	2008-01-15 17:33:17.000-0600
Priority:	Minor	Regression?	No
Status:	Closed/Complete	Components:	Channels/chan_zap
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) chan_zap.c.patch
Description:	When cidstart is set to ring ss_thread will use the last definition of cid_signalling instead of the value from the channels private structure.
Comments:	By: Russell Bryant (russell) 2006-03-16 11:47:25.000-0600 This has been fixed in the 1.2 branch and the trunk in revisions 13237 and 13238.  In the 1.2 branch, I actually found a second place where this bug existed that had already been fixed in the trunk. Thanks! By: Digium Subversion (svnbot) 2008-01-15 17:33:17.000-0600 Repository: asterisk Revision: 13237 U   branches/1.2/channels/chan_zap.c ------------------------------------------------------------------------ r13237 | russell | 2008-01-15 17:33:16 -0600 (Tue, 15 Jan 2008) | 3 lines always use the callerid signalling method set in the zt_pvt strucutre as opposed to the last one read from the config file (issue ASTERISK-6556, with mods) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=13237 By: Digium Subversion (svnbot) 2008-01-15 17:33:17.000-0600 Repository: asterisk Revision: 13238 _U  trunk/ U   trunk/channels/chan_zap.c ------------------------------------------------------------------------ r13238 | russell | 2008-01-15 17:33:17 -0600 (Tue, 15 Jan 2008) | 11 lines Merged revisions 13237 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.2 ........ r13237 | russell | 2006-03-16 12:42:46 -0500 (Thu, 16 Mar 2006) | 3 lines always use the callerid signalling method set in the zt_pvt strucutre as opposed to the last one read from the config file (issue ASTERISK-6556, with mods) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=13238