[asterisk-bugs] [Asterisk 0013675]: [patch] app_sms doesn't answer the call, currently requires Answer() before hand

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13675 
====================================================================== 
Reported By:                alecdavis
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13675
Category:                   Applications/app_sms
Reproducibility:            always
Severity:                   trivial
Priority:                   normal
Status:                     new
Asterisk Version:           1.6.0 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-10-12 15:16 CDT
Last Modified:              2008-10-16 17:33 CDT
====================================================================== 
Summary:                    [patch] app_sms doesn't answer the call, currently
requires Answer() before hand
Description: 
In app_sms.c sms_exec there is code that answers the channel if it's not
already answered. But when sms is invoked with option 'a', the channel may
not have been answered with Answer() app.

the code I believe is answering too late:
      if (chan->_state != AST_STATE_UP)
              ast_answer(chan);

needs to be before:
        if (ast_test_flag(&flags, OPTION_ANSWER)) {
                h.framenumber = 1;           /* Proto 2 */
        .......
 
====================================================================== 

---------------------------------------------------------------------- 
 (0093859) svnbot (reporter) - 2008-10-16 17:33
 http://bugs.digium.com/view.php?id=13675#c93859 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 150257

U   trunk/apps/app_sms.c

------------------------------------------------------------------------
r150257 | mmichelson | 2008-10-16 17:33:38 -0500 (Thu, 16 Oct 2008) | 9
lines

Answer the channel prior to checking for the 'a'
option in app_sms.

(closes issue http://bugs.digium.com/view.php?id=13675)
Reported by: alecdavis
Patches:
      app_sms.bug13675.148985.diff.txt uploaded by alecdavis (license 585)


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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-10-16 17:33 svnbot         Checkin                                      
2008-10-16 17:33 svnbot         Note Added: 0093859                          
======================================================================