[asterisk-bugs] [Asterisk 0013493]: In some cases parking extension isn't removed after parked channel is gone (continuation of problem reported in #13425)

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13493 
====================================================================== 
Reported By:                mdu113
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13493
Category:                   Resources/res_features
Reproducibility:            sometimes
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 142474 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-09-16 11:32 CDT
Last Modified:              2008-09-16 16:34 CDT
====================================================================== 
Summary:                    In some cases parking extension isn't removed after
parked channel is gone (continuation of problem reported in
http://bugs.digium.com/view.php?id=13425)
Description: 
The fix commited for issue 13425 is not completely solved the problem I
reported in http://bugs.digium.com/view.php?id=13425#92536
In some cases, specifically when parked channel hangs up during parking
announcement is still being played to the parkee, the parking extension
isn't properly removed from parking context. There's some interesting
detail to it though. Parked channel must hangup close to the end of
announcement for it to happen.
Scenario: A calls B (PARINGEXTEN=701 set), B answers and parks A. A is
parked at extension 701 at parkedcalls. Now, if I hangup parked channel during
asterisk playing "one" then extension 701 at parkedcalls isn't removed.
====================================================================== 

---------------------------------------------------------------------- 
 (0092593) svnbot (reporter) - 2008-09-16 16:34
 http://bugs.digium.com/view.php?id=13493#c92593 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 143270

U   branches/1.4/res/res_features.c

------------------------------------------------------------------------
r143270 | jpeeler | 2008-09-16 16:34:35 -0500 (Tue, 16 Sep 2008) | 9 lines

(closes issue http://bugs.digium.com/view.php?id=13493)
Reported by: mdu113
Patches:
      bug13493.diff uploaded by jpeeler (license 325)
Tested by: mdu113

Changed park_call_full to hold the parkinglot lock a little longer, which
protects the parkeduser struct from being freed out from underneath. Made
sure that the parking extension is added to the parking context while
holding the lock thereby ensuring that there are no spurious warnings from
removal attempts when a hangup occurs while the parking lot is being
announced.


------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=143270 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-09-16 16:34 svnbot         Checkin                                      
2008-09-16 16:34 svnbot         Note Added: 0092593                          
======================================================================