Summary: | ASTERISK-11695: Deprecated SIP-Client session crashes Asterisk in startup | ||
Reporter: | Michael_qq12345 (qq12345) | Labels: | |
Date Opened: | 2008-03-21 10:48:53 | Date Closed: | 2008-03-25 13:07:22 |
Priority: | Critical | Regression? | No |
Status: | Closed/Complete | Components: | Channels/chan_sip/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) backtrace_SIP_deprecated_client.txt ( 1) core.24516.zip ( 2) core.26154.zip ( 3) core.27031.zip ( 4) core.6241.zip | |
Description: | SIP Phone 102 is disrupted. RE-Starting Asterisk (trunk, 21.03.2008) crashes Asterisk: (SIP-Phone 101 is running and tuned in, but out of scope). [...] ad: Parsing /etc/asterisk/cli.conf == Found *CLI> [Mar 21 16:25:55] DEBUG[8959]: sched.c:270 ast_sched_del: ast_sched_del() [Mar 21 16:25:55] DEBUG[8959]: chan_sip.c:3017 __sip_ack: Stopping retransmission on '73d8b1892ccbd840740906f24458c34c@217.236.227.63' of Request 102: Match Found [Mar 21 16:25:55] NOTICE[8959]: chan_sip.c:15097 handle_response_peerpoke: Peer '101' is now Reachable. (239ms / 2000ms) [Mar 21 16:25:55] DEBUG[8959]: sched.c:270 ast_sched_del: ast_sched_del() [Mar 21 16:25:55] DEBUG[8959]: sched.c:219 ast_sched_add_variable: ast_sched_add() [Mar 21 16:25:55] DEBUG[8959]: sched.c:338 ast_sched_runq: ast_sched_runq() [Mar 21 16:25:55] DEBUG[8959]: sched.c:270 ast_sched_del: ast_sched_del() [Mar 21 16:25:55] DEBUG[8959]: sched.c:291 ast_sched_del: Attempted to delete nonexistent schedule entry -1! Segmentation fault ****** ADDITIONAL INFORMATION ****** asterisk trunk 21.03.2008 mISDN 1.1.7 or mISDN 1.2 SIP-Client XLite or Eyebeam | ||
Comments: | By: Jason Parker (jparker) 2008-03-21 10:52:15 Please upload a backtrace, per doc/backtrace.txt By: Jason Parker (jparker) 2008-03-21 12:16:58 No, the core files are completely useless. Please read doc/backtrace.txt By: Mark Michelson (mmichelson) 2008-03-24 13:56:16 Do you have DO_CRASH enabled in menuselect? By: Michael_qq12345 (qq12345) 2008-03-24 14:03:20 Yes, I try to get another issue nailed down, therefore I enabled DO_CRASH. But I cannot execute till my working place due to the fact, that asterisk crashes much before. How can I overcome this issue? Michael By: Mark Michelson (mmichelson) 2008-03-24 14:11:01 Actually, I can see from the line number in the backtrace that you do have DO_CRASH enabled. This crash is due to a commit I made as part of what will become a mass fix for scheduler behavior, but I see now that this was probably premature. Instead changes such as this should be handled in controlled environments so that they don't cause troubles such as this. I apologize and will revert the commit that caused this. By: Digium Subversion (svnbot) 2008-03-24 14:13:17 Repository: asterisk Revision: 110618 U branches/1.4/channels/chan_sip.c ------------------------------------------------------------------------ r110618 | mmichelson | 2008-03-24 14:13:15 -0500 (Mon, 24 Mar 2008) | 15 lines This is a revert for revision 108288. The reason is that that revision was not for an actual bug fix per se, and so it really should not have been in 1.4 in the first place. Plus, people who compile with DO_CRASH are more likely to encounter a crash due to this change. While I think the usage of DO_CRASH in ast_sched_del is a bit absurd, this sort of change is beyond the scope of 1.4 and should be done instead in a developer branch based on trunk so that all scheduler functions are fixed at once. I also am reverting the change to trunk and 1.6 since they also suffer from the DO_CRASH potential. (closes issue ASTERISK-11695) Reported by: qq12345 ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=110618 By: Digium Subversion (svnbot) 2008-03-24 14:15:18 Repository: asterisk Revision: 110619 U trunk/channels/chan_sip.c ------------------------------------------------------------------------ r110619 | mmichelson | 2008-03-24 14:15:17 -0500 (Mon, 24 Mar 2008) | 23 lines Merged revisions 110618 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110618 | mmichelson | 2008-03-24 14:17:41 -0500 (Mon, 24 Mar 2008) | 15 lines This is a revert for revision 108288. The reason is that that revision was not for an actual bug fix per se, and so it really should not have been in 1.4 in the first place. Plus, people who compile with DO_CRASH are more likely to encounter a crash due to this change. While I think the usage of DO_CRASH in ast_sched_del is a bit absurd, this sort of change is beyond the scope of 1.4 and should be done instead in a developer branch based on trunk so that all scheduler functions are fixed at once. I also am reverting the change to trunk and 1.6 since they also suffer from the DO_CRASH potential. (closes issue ASTERISK-11695) Reported by: qq12345 ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=110619 By: Digium Subversion (svnbot) 2008-03-24 14:16:12 Repository: asterisk Revision: 110620 _U branches/1.6.0/ U branches/1.6.0/channels/chan_sip.c ------------------------------------------------------------------------ r110620 | mmichelson | 2008-03-24 14:16:11 -0500 (Mon, 24 Mar 2008) | 31 lines Merged revisions 110619 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r110619 | mmichelson | 2008-03-24 14:19:37 -0500 (Mon, 24 Mar 2008) | 23 lines Merged revisions 110618 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110618 | mmichelson | 2008-03-24 14:17:41 -0500 (Mon, 24 Mar 2008) | 15 lines This is a revert for revision 108288. The reason is that that revision was not for an actual bug fix per se, and so it really should not have been in 1.4 in the first place. Plus, people who compile with DO_CRASH are more likely to encounter a crash due to this change. While I think the usage of DO_CRASH in ast_sched_del is a bit absurd, this sort of change is beyond the scope of 1.4 and should be done instead in a developer branch based on trunk so that all scheduler functions are fixed at once. I also am reverting the change to trunk and 1.6 since they also suffer from the DO_CRASH potential. (closes issue ASTERISK-11695) Reported by: qq12345 ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=110620 By: Digium Subversion (svnbot) 2008-03-24 16:20:37 Repository: asterisk Revision: 110623 _U team/qwell/codec_consistency/ U team/qwell/codec_consistency/channels/chan_iax2.c U team/qwell/codec_consistency/channels/chan_sip.c ------------------------------------------------------------------------ r110623 | qwell | 2008-03-24 16:20:36 -0500 (Mon, 24 Mar 2008) | 58 lines Merged revisions 110615,110619,110621 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r110615 | russell | 2008-03-24 12:36:04 -0500 (Mon, 24 Mar 2008) | 10 lines Merged revisions 110614 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110614 | russell | 2008-03-24 12:34:56 -0500 (Mon, 24 Mar 2008) | 2 lines Turn a NOTICE into a DEBUG message. ........ ................ r110619 | mmichelson | 2008-03-24 14:19:37 -0500 (Mon, 24 Mar 2008) | 23 lines Merged revisions 110618 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110618 | mmichelson | 2008-03-24 14:17:41 -0500 (Mon, 24 Mar 2008) | 15 lines This is a revert for revision 108288. The reason is that that revision was not for an actual bug fix per se, and so it really should not have been in 1.4 in the first place. Plus, people who compile with DO_CRASH are more likely to encounter a crash due to this change. While I think the usage of DO_CRASH in ast_sched_del is a bit absurd, this sort of change is beyond the scope of 1.4 and should be done instead in a developer branch based on trunk so that all scheduler functions are fixed at once. I also am reverting the change to trunk and 1.6 since they also suffer from the DO_CRASH potential. (closes issue ASTERISK-11695) Reported by: qq12345 ........ ................ r110621 | mmichelson | 2008-03-24 15:14:07 -0500 (Mon, 24 Mar 2008) | 11 lines Remove the "Event: registration" header from Asterisk-generated SIP REGISTER requests. rjain points out that RFC 3265 specifies that the Event: header is not a valid header for REGISTER requests and that the "registration" value is not defined at IANA. (closes issue ASTERISK-11702) Reported by: rjain Patches: chan_sip.c.diff uploaded by rjain (license 226) ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=110623 By: Digium Subversion (svnbot) 2008-03-25 12:59:04 Repository: asterisk Revision: 110693 _U team/group/cdr_backend_ast_str/ U team/group/cdr_backend_ast_str/CHANGES U team/group/cdr_backend_ast_str/Makefile U team/group/cdr_backend_ast_str/channels/chan_iax2.c U team/group/cdr_backend_ast_str/channels/chan_sip.c U team/group/cdr_backend_ast_str/configs/extensions.conf.sample U team/group/cdr_backend_ast_str/configs/sip.conf.sample U team/group/cdr_backend_ast_str/configs/voicemail.conf.sample U team/group/cdr_backend_ast_str/include/asterisk/options.h U team/group/cdr_backend_ast_str/main/app.c U team/group/cdr_backend_ast_str/main/asterisk.c U team/group/cdr_backend_ast_str/main/channel.c ------------------------------------------------------------------------ r110693 | tilghman | 2008-03-25 12:58:50 -0500 (Tue, 25 Mar 2008) | 140 lines Merged revisions 110610,110615,110619,110621,110625,110629,110631,110636,110639,110689,110691 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r110610 | file | 2008-03-24 10:28:25 -0500 (Mon, 24 Mar 2008) | 6 lines Only print out the set_address_from_contact host verbose message if debugging is enabled on the dialog. (closes issue ASTERISK-11703) Reported by: rjain Patches: chan_sip.c.diff uploaded by rjain (license 226) ................ r110615 | russell | 2008-03-24 12:36:04 -0500 (Mon, 24 Mar 2008) | 10 lines Merged revisions 110614 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110614 | russell | 2008-03-24 12:34:56 -0500 (Mon, 24 Mar 2008) | 2 lines Turn a NOTICE into a DEBUG message. ........ ................ r110619 | mmichelson | 2008-03-24 14:19:37 -0500 (Mon, 24 Mar 2008) | 23 lines Merged revisions 110618 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110618 | mmichelson | 2008-03-24 14:17:41 -0500 (Mon, 24 Mar 2008) | 15 lines This is a revert for revision 108288. The reason is that that revision was not for an actual bug fix per se, and so it really should not have been in 1.4 in the first place. Plus, people who compile with DO_CRASH are more likely to encounter a crash due to this change. While I think the usage of DO_CRASH in ast_sched_del is a bit absurd, this sort of change is beyond the scope of 1.4 and should be done instead in a developer branch based on trunk so that all scheduler functions are fixed at once. I also am reverting the change to trunk and 1.6 since they also suffer from the DO_CRASH potential. (closes issue ASTERISK-11695) Reported by: qq12345 ........ ................ r110621 | mmichelson | 2008-03-24 15:14:07 -0500 (Mon, 24 Mar 2008) | 11 lines Remove the "Event: registration" header from Asterisk-generated SIP REGISTER requests. rjain points out that RFC 3265 specifies that the Event: header is not a valid header for REGISTER requests and that the "registration" value is not defined at IANA. (closes issue ASTERISK-11702) Reported by: rjain Patches: chan_sip.c.diff uploaded by rjain (license 226) ................ r110625 | oej | 2008-03-25 05:54:07 -0500 (Tue, 25 Mar 2008) | 6 lines Use the "Server" header when responding to SIP requests. (closes issue ASTERISK-11701) Reported by: rjain Patches: chan_sip.c.diff uploaded by rjain (license 226) ................ r110629 | file | 2008-03-25 09:39:45 -0500 (Tue, 25 Mar 2008) | 12 lines Merged revisions 110628 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110628 | file | 2008-03-25 11:37:35 -0300 (Tue, 25 Mar 2008) | 4 lines Add an option (transmit_silence) which transmits silence during both Record() and DTMF generation. The reason this is an option is that in order to transmit silence we have to setup a translation path. This may not be needed/wanted in all cases. (closes issue ASTERISK-9755) Reported by: tracinet ........ ................ r110631 | file | 2008-03-25 10:18:41 -0500 (Tue, 25 Mar 2008) | 4 lines Add a special dialplan variable to chan_sip which will cause an audio file to be played upon completion of an attended transfer. (closes issue ASTERISK-8972) Reported by: sunder ................ r110636 | mmichelson | 2008-03-25 10:41:33 -0500 (Tue, 25 Mar 2008) | 15 lines Merged revisions 110635 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110635 | mmichelson | 2008-03-25 10:40:33 -0500 (Tue, 25 Mar 2008) | 7 lines When reverting a commit, I accidentally left in this bit which was an experiment to see what would happen. It passed the compile test, and I didn't notice I had left this change in too. So this is a revert of a revert...sort of. ........ ................ r110639 | mmichelson | 2008-03-25 10:44:01 -0500 (Tue, 25 Mar 2008) | 3 lines Oops here too. I need to stop coding for a while... ................ r110689 | tilghman | 2008-03-25 12:40:28 -0500 (Tue, 25 Mar 2008) | 6 lines Update the sample configuration, to use Macro less (since it's now deprecated). (closes issue ASTERISK-11716) Reported by: pprindeville Patches: bugid-0012293.1.6.patch uploaded by pprindeville (license 347) ................ r110691 | tilghman | 2008-03-25 12:46:34 -0500 (Tue, 25 Mar 2008) | 6 lines Update sample configurations to make virtual hosting more obvious. (closes issue ASTERISK-11414) Reported by: pprindeville Patches: acme-virtualpbx.1.6.patch uploaded by pprindeville (license 347) ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=110693 By: Digium Subversion (svnbot) 2008-03-25 13:07:22 Repository: asterisk Revision: 110694 _U team/murf/bug11210/ U team/murf/bug11210/CHANGES U team/murf/bug11210/Makefile U team/murf/bug11210/channels/chan_iax2.c U team/murf/bug11210/channels/chan_sip.c U team/murf/bug11210/configs/extensions.conf.sample U team/murf/bug11210/configs/sip.conf.sample U team/murf/bug11210/configs/voicemail.conf.sample U team/murf/bug11210/include/asterisk/options.h U team/murf/bug11210/main/app.c U team/murf/bug11210/main/asterisk.c U team/murf/bug11210/main/channel.c ------------------------------------------------------------------------ r110694 | murf | 2008-03-25 13:07:18 -0500 (Tue, 25 Mar 2008) | 140 lines Merged revisions 110610,110615,110619,110621,110625,110629,110631,110636,110639,110689,110691 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r110610 | file | 2008-03-24 09:28:25 -0600 (Mon, 24 Mar 2008) | 6 lines Only print out the set_address_from_contact host verbose message if debugging is enabled on the dialog. (closes issue ASTERISK-11703) Reported by: rjain Patches: chan_sip.c.diff uploaded by rjain (license 226) ................ r110615 | russell | 2008-03-24 11:36:04 -0600 (Mon, 24 Mar 2008) | 10 lines Merged revisions 110614 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110614 | russell | 2008-03-24 12:34:56 -0500 (Mon, 24 Mar 2008) | 2 lines Turn a NOTICE into a DEBUG message. ........ ................ r110619 | mmichelson | 2008-03-24 13:19:37 -0600 (Mon, 24 Mar 2008) | 23 lines Merged revisions 110618 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110618 | mmichelson | 2008-03-24 14:17:41 -0500 (Mon, 24 Mar 2008) | 15 lines This is a revert for revision 108288. The reason is that that revision was not for an actual bug fix per se, and so it really should not have been in 1.4 in the first place. Plus, people who compile with DO_CRASH are more likely to encounter a crash due to this change. While I think the usage of DO_CRASH in ast_sched_del is a bit absurd, this sort of change is beyond the scope of 1.4 and should be done instead in a developer branch based on trunk so that all scheduler functions are fixed at once. I also am reverting the change to trunk and 1.6 since they also suffer from the DO_CRASH potential. (closes issue ASTERISK-11695) Reported by: qq12345 ........ ................ r110621 | mmichelson | 2008-03-24 14:14:07 -0600 (Mon, 24 Mar 2008) | 11 lines Remove the "Event: registration" header from Asterisk-generated SIP REGISTER requests. rjain points out that RFC 3265 specifies that the Event: header is not a valid header for REGISTER requests and that the "registration" value is not defined at IANA. (closes issue ASTERISK-11702) Reported by: rjain Patches: chan_sip.c.diff uploaded by rjain (license 226) ................ r110625 | oej | 2008-03-25 04:54:07 -0600 (Tue, 25 Mar 2008) | 6 lines Use the "Server" header when responding to SIP requests. (closes issue ASTERISK-11701) Reported by: rjain Patches: chan_sip.c.diff uploaded by rjain (license 226) ................ r110629 | file | 2008-03-25 08:39:45 -0600 (Tue, 25 Mar 2008) | 12 lines Merged revisions 110628 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110628 | file | 2008-03-25 11:37:35 -0300 (Tue, 25 Mar 2008) | 4 lines Add an option (transmit_silence) which transmits silence during both Record() and DTMF generation. The reason this is an option is that in order to transmit silence we have to setup a translation path. This may not be needed/wanted in all cases. (closes issue ASTERISK-9755) Reported by: tracinet ........ ................ r110631 | file | 2008-03-25 09:18:41 -0600 (Tue, 25 Mar 2008) | 4 lines Add a special dialplan variable to chan_sip which will cause an audio file to be played upon completion of an attended transfer. (closes issue ASTERISK-8972) Reported by: sunder ................ r110636 | mmichelson | 2008-03-25 09:41:33 -0600 (Tue, 25 Mar 2008) | 15 lines Merged revisions 110635 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r110635 | mmichelson | 2008-03-25 10:40:33 -0500 (Tue, 25 Mar 2008) | 7 lines When reverting a commit, I accidentally left in this bit which was an experiment to see what would happen. It passed the compile test, and I didn't notice I had left this change in too. So this is a revert of a revert...sort of. ........ ................ r110639 | mmichelson | 2008-03-25 09:44:01 -0600 (Tue, 25 Mar 2008) | 3 lines Oops here too. I need to stop coding for a while... ................ r110689 | tilghman | 2008-03-25 11:40:28 -0600 (Tue, 25 Mar 2008) | 6 lines Update the sample configuration, to use Macro less (since it's now deprecated). (closes issue ASTERISK-11716) Reported by: pprindeville Patches: bugid-0012293.1.6.patch uploaded by pprindeville (license 347) ................ r110691 | tilghman | 2008-03-25 11:46:34 -0600 (Tue, 25 Mar 2008) | 6 lines Update sample configurations to make virtual hosting more obvious. (closes issue ASTERISK-11414) Reported by: pprindeville Patches: acme-virtualpbx.1.6.patch uploaded by pprindeville (license 347) ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=110694 |