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

[Asterisk Bug Tracker]

The following issue has been RESOLVED. 
====================================================================== 
http://bugs.digium.com/view.php?id=13675 
====================================================================== 
Reported By:                alecdavis
Assigned To:                svnbot
====================================================================== 
Project:                    Asterisk
Issue ID:                   13675
Category:                   Applications/app_sms
Reproducibility:            always
Severity:                   trivial
Priority:                   normal
Status:                     resolved
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:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
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 */
        .......
 
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-10-16 17:33 svnbot         Status                   new => resolved     
2008-10-16 17:33 svnbot         Resolution               open => fixed       
2008-10-16 17:33 svnbot         Assigned To               => svnbot          
======================================================================