[asterisk-bugs] [Asterisk 0010744]: MobileStatus crash

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=10744 
====================================================================== 
Reported By:                tootai
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   10744
Category:                   Addons/chan_mobile
Reproducibility:            always
Severity:                   crash
Priority:                   normal
Status:                     new
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 81404 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             09-17-2007 10:50 CDT
Last Modified:              09-17-2007 16:48 CDT
====================================================================== 
Summary:                    MobileStatus crash
Description: 
This is the dialplan:

[macro-mymacro]
exten => s,1,MobileStatus(o2,O2CELL)
exten => s,n,NoOp(Call through GSM - o2 status is ${O2CELL})

and Asterisk crash before the NoOp. If we remove the MobileStatus,
everything is ok. chan_mobile updated today 17/09/07.

Daniel
====================================================================== 

---------------------------------------------------------------------- 
 qwell - 09-17-07 16:48  
---------------------------------------------------------------------- 
I think I know what the problem is.  Can you try using
MobileStatus(o2|O2CELL) instead of MobileStatus(o2,O2CELL)?  Basically,
replace the comma with a vertical pipe.


I think the problem is with not using the standard app arg parsing
functions.  If no | is found in the string, device will be NULL, which will
crash - probably in the strcmp.  If that fixes the problem, I'll work on a
patch for this and MobileSendSMS 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
09-17-07 16:48  qwell          Note Added: 0070705                          
======================================================================