[asterisk-bugs] [Asterisk 0011342]: Impliment call park

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=11342 
====================================================================== 
Reported By:                DEA
Assigned To:                qwell
====================================================================== 
Project:                    Asterisk
Issue ID:                   11342
Category:                   Channels/chan_skinny
Reproducibility:            always
Severity:                   feature
Priority:                   normal
Status:                     assigned
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 86150 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             11-21-2007 15:54 CST
Last Modified:              01-22-2008 16:26 CST
====================================================================== 
Summary:                    Impliment call park
Description: 
Add support for the Park Softkey on skinny phones.

Depends on a working Call Park setup (features.conf)
====================================================================== 

---------------------------------------------------------------------- 
 DEA - 01-22-08 16:26  
---------------------------------------------------------------------- 
OK, it was not related to the use of macros.  It seems that using a macro
would alter the timing just enough that SOMETIMES it would work.

The real issue appears to be the relationship between:
ast_masq_park_call -> ast_park_call -> park_call_full

ast_masq_park_call sets up the masquerade and calls ast_park_call which
explicitly calls park_call_full with orig_chan_name set to NULL. Once
inside park_call_full, we use orig_chan_name to detect if this is a
masqueraded channel.

I've done a cursory review of apps/channels, and it appears the only
way into park_call_full is through ast_park_call or ast_masq_park_call,
so the logic used to test for masquerading could not possibly work. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
01-22-08 16:26  DEA            Note Added: 0081050                          
======================================================================