[asterisk-bugs] [Asterisk 0014241]: [patch] h exten getting run at the wrong time

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14241 
====================================================================== 
Reported By:                jmls
Assigned To:                murf
====================================================================== 
Project:                    Asterisk
Issue ID:                   14241
Category:                   Resources/res_features
Reproducibility:            always
Severity:                   block
Priority:                   normal
Status:                     ready for testing
Target Version:             1.4.24
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 168602 
Request Review:              
====================================================================== 
Date Submitted:             2009-01-14 12:34 CST
Last Modified:              2009-01-20 14:17 CST
====================================================================== 
Summary:                    [patch] h exten getting run at the wrong time
Description: 
in 1.4 svn, If A calls B, and A is redirected via AMI to another extension
(meetme room for example), the h extension is being run on A before it is
redirected.

The h extension should be run on A when A hangs up.
====================================================================== 

---------------------------------------------------------------------- 
 (0098234) murf (administrator) - 2009-01-20 14:17
 http://bugs.digium.com/view.php?id=14241#c98234 
---------------------------------------------------------------------- 
OK, I took some time and ran around the code base and looked at just how
many different places this patch could affect, and it made me nervous. So,
I generated a new patch where the patch is localized to the Redirect
handler in the manager code. I tested it and it looks good. I made the same
mods to the app_channelredirect app, which is basically the same operation.
I also went after the one bad (obviously bad, that is) case where, with the
one-touch blind transfer (builtin_blindtransfer()) you get two h extens run
on the same channel (but at different times). Now, just one gets run. Since
changing the h-exten run time changes how the CDR is generated, I did a CDR
swap operation to give reasonable results (just as good/bad as before).

So, if people find h-exten probs with the fax extension in chan_dahdi, or
chan_misdn, or chan_vpb, then I have a quick fix. The same for the
console_transfer func in the chan_oss; the transfer command handler in
chan_iax2 and chan_sip-- then I have a possible quick fix.

So, jmls, if you have some time, take a spin with the
14241_redirect_no_bridgeCDR_or_h_exten_via_transfer file I attached above! 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-01-20 14:17 murf           Note Added: 0098234                          
======================================================================