[asterisk-bugs] [Asterisk 0018126]: [patch] stuck channels if followme context doesnt exists

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18126 
====================================================================== 
Reported By:                junky
Assigned To:                tilghman
====================================================================== 
Project:                    Asterisk
Issue ID:                   18126
Category:                   Applications/app_followme
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Asterisk Version:           1.4.36 
JIRA:                       SWP-2366 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-10-12 20:46 CDT
Last Modified:              2010-12-06 18:21 CST
====================================================================== 
Summary:                    [patch] stuck channels if followme context doesnt
exists
Description: 
If followme tried to call a non-existent context, the 2 local channels will
be stucked.
You cant soft hangup those channels, you must restart *.
example without patch:
    -- Executing [11 at junky:1] FollowMe("SIP/10-00000001", "junky") in new
stack
    -- <SIP/10-00000001> Playing 'followme/pls-hold-while-try' (language
'en')
    -- Music class default requested but no musiconhold loaded.
    -- calling 10 at doesntexist
[Oct 12 20:49:34] NOTICE[15253]: chan_local.c:511 local_call: No such
extension/context 10 at doesntexist while calling Local channel
    -- couldn't reach at this number.
  == Auto fallthrough, channel 'SIP/10-00000001' status is 'UNKNOWN'


The logic is just to check if the context is valid or not. If it not
valid, it will return. This also prevents to create in memory, since we
already know that chan_local.c will refuse it.


example with patch:
    -- Executing [11 at junky:1] FollowMe("SIP/10-00000001", "junky") in new
stack
    -- <SIP/10-00000001> Playing 'followme/pls-hold-while-try' (language
'en')
    -- Music class default requested but no musiconhold loaded.
[Oct 12 21:43:36] WARNING[20162]: app_followme.c:832 findmeexec:
doesntexist doesnt exists, no need to continue...
    -- Executing [11 at junky:2] Hangup("SIP/10-00000001", "") in new stack
  == Spawn extension (junky, 11, 2) exited non-zero on 'SIP/10-00000001'

====================================================================== 

---------------------------------------------------------------------- 
 (0129376) svnbot (reporter) - 2010-12-06 18:21
 https://issues.asterisk.org/view.php?id=18126#c129376 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 297713

_U  branches/1.6.2/
U   branches/1.6.2/apps/app_followme.c

------------------------------------------------------------------------
r297713 | tilghman | 2010-12-06 18:21:51 -0600 (Mon, 06 Dec 2010) | 15
lines

Merged revisions 297689 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r297689 | tilghman | 2010-12-06 18:07:37 -0600 (Mon, 06 Dec 2010) | 8
lines
  
  Don't create a Local channel if the target extension does not exist.
  
  (closes issue https://issues.asterisk.org/view.php?id=18126)
   Reported by: junky
   Patches: 
         followme.diff uploaded by junky (license 177)
         (partially restructured by me to avoid a possible memory leak)
........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-12-06 18:21 svnbot         Checkin                                      
2010-12-06 18:21 svnbot         Note Added: 0129376                          
======================================================================