[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:31 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 */
        .......
 
====================================================================== 

---------------------------------------------------------------------- 
 (0093858) putnopvut (administrator) - 2008-10-16 17:31
 http://bugs.digium.com/view.php?id=13675#c93858 
---------------------------------------------------------------------- 
Makes sense to me. Thanks for the patch! 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-10-16 17:31 putnopvut      Note Added: 0093858                          
======================================================================