| Summary: | ASTERISK-14427: [regression] parking c-e-p is missing, so timed-out parked call never returns to parker | ||
| Reporter: | David Rodman (djrodman) | Labels: | |
| Date Opened: | 2009-07-07 13:06:29 | Date Closed: | 2010-01-13 13:52:38.000-0600 |
| Priority: | Minor | Regression? | Yes |
| Status: | Closed/Complete | Components: | Resources/res_features |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ( 0) patch_15459.txt | |
| Description: | 1.4.25.1 res_features.c 476 ff This code is saving the parker's info so that if the park times out the call can be sent back to who parked it. The call comes in to context 'redroad', exten 501. From there it goes to macro stdexten. So the context is 'macro-stdexten', and the macrocontext is 'redroad'. Howveer, in park_call_full, the channel cata is: chan->context: macro-stdexten chan->exten: s chan->priority: 44 chan->macrocontext: blank chan->macroexten: blank chan->priority: 0 In version 1.4.17 this informatino was correct, i.e.: chan->context: macro-stdexten chan->exten: s chan->priority: 44 chan->macrocontext: redroad chan->macroexten: 501 chan->priority: 1 In 1.4.25.1 the verbose output shows "will timeout back to macro-stdexten, s, 44 in 90 seconds" (and that's what it does. In 1.4.17 it shows "will timeout back to redroad, 501, 1 in 90 seconds" | ||
| Comments: | By: Leif Madsen (lmadsen) 2009-08-31 09:08:20 Assigned to Tilghman for review. I can't tell if this was part of the call parking changes that Terry was working on, and if so, if this behaviour change happened to fix another bug or not. If you can't really review this, please set status back to New and unassign yourself. Thanks! By: Matthias Nick (mnick) 2009-11-02 14:28:50.000-0600 posted a patch for latest 1.4 By: Leif Madsen (lmadsen) 2010-01-05 13:12:20.000-0600 Pinging the reporter to see if they are still interested in testing this patch? By: Digium Subversion (svnbot) 2010-01-13 13:43:34.000-0600 Repository: asterisk Revision: 239838 U branches/1.4/res/res_features.c ------------------------------------------------------------------------ r239838 | jpeeler | 2010-01-13 13:43:33 -0600 (Wed, 13 Jan 2010) | 11 lines Fix regression for timed out parked call returning to caller This issue seems to have been exposed by the fix in 160390 whereby using a masquerade prevented a crash. The new channel used in the masquerade was not copying the macro information from the old channel. (closes issue ASTERISK-14427) Reported by: djrodman Patches: patch_15459.txt uploaded by mnick (license ) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=239838 By: Digium Subversion (svnbot) 2010-01-13 13:48:17.000-0600 Repository: asterisk Revision: 239839 _U trunk/ U trunk/main/features.c ------------------------------------------------------------------------ r239839 | jpeeler | 2010-01-13 13:48:17 -0600 (Wed, 13 Jan 2010) | 18 lines Merged revisions 239838 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r239838 | jpeeler | 2010-01-13 13:43:33 -0600 (Wed, 13 Jan 2010) | 11 lines Fix regression for timed out parked call returning to caller This issue seems to have been exposed by the fix in 160390 whereby using a masquerade prevented a crash. The new channel used in the masquerade was not copying the macro information from the old channel. (closes issue ASTERISK-14427) Reported by: djrodman Patches: patch_15459.txt uploaded by mnick (license ) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=239839 By: Digium Subversion (svnbot) 2010-01-13 13:51:41.000-0600 Repository: asterisk Revision: 239840 _U branches/1.6.2/ U branches/1.6.2/main/features.c ------------------------------------------------------------------------ r239840 | jpeeler | 2010-01-13 13:51:41 -0600 (Wed, 13 Jan 2010) | 25 lines Merged revisions 239839 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r239839 | jpeeler | 2010-01-13 13:48:16 -0600 (Wed, 13 Jan 2010) | 18 lines Merged revisions 239838 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r239838 | jpeeler | 2010-01-13 13:43:33 -0600 (Wed, 13 Jan 2010) | 11 lines Fix regression for timed out parked call returning to caller This issue seems to have been exposed by the fix in 160390 whereby using a masquerade prevented a crash. The new channel used in the masquerade was not copying the macro information from the old channel. (closes issue ASTERISK-14427) Reported by: djrodman Patches: patch_15459.txt uploaded by mnick (license ) ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=239840 By: Digium Subversion (svnbot) 2010-01-13 13:52:08.000-0600 Repository: asterisk Revision: 239841 _U branches/1.6.1/ U branches/1.6.1/main/features.c ------------------------------------------------------------------------ r239841 | jpeeler | 2010-01-13 13:52:08 -0600 (Wed, 13 Jan 2010) | 25 lines Merged revisions 239839 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r239839 | jpeeler | 2010-01-13 13:48:16 -0600 (Wed, 13 Jan 2010) | 18 lines Merged revisions 239838 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r239838 | jpeeler | 2010-01-13 13:43:33 -0600 (Wed, 13 Jan 2010) | 11 lines Fix regression for timed out parked call returning to caller This issue seems to have been exposed by the fix in 160390 whereby using a masquerade prevented a crash. The new channel used in the masquerade was not copying the macro information from the old channel. (closes issue ASTERISK-14427) Reported by: djrodman Patches: patch_15459.txt uploaded by mnick (license ) ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=239841 By: Digium Subversion (svnbot) 2010-01-13 13:52:37.000-0600 Repository: asterisk Revision: 239842 _U branches/1.6.0/ U branches/1.6.0/main/features.c ------------------------------------------------------------------------ r239842 | jpeeler | 2010-01-13 13:52:36 -0600 (Wed, 13 Jan 2010) | 25 lines Merged revisions 239839 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r239839 | jpeeler | 2010-01-13 13:48:16 -0600 (Wed, 13 Jan 2010) | 18 lines Merged revisions 239838 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r239838 | jpeeler | 2010-01-13 13:43:33 -0600 (Wed, 13 Jan 2010) | 11 lines Fix regression for timed out parked call returning to caller This issue seems to have been exposed by the fix in 160390 whereby using a masquerade prevented a crash. The new channel used in the masquerade was not copying the macro information from the old channel. (closes issue ASTERISK-14427) Reported by: djrodman Patches: patch_15459.txt uploaded by mnick (license ) ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=239842 | ||