Summary:ASTERISK-10320: MobileStatus crash
Reporter:tootai (tootai)Labels:
Date Opened:2007-09-17 10:50:11Date Closed:2007-09-18 01:13:33
Versions:Frequency of
Environment:Attachments:( 0) chan_mobile-standard_app_args.diff
Description:This is the dialplan:

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.

Comments:By: Jason Parker (jparker) 2007-09-17 16:48:59

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

By: Jason Parker (jparker) 2007-09-17 17:23:31

I went ahead and wrote a patch for what I think the problem might be.  I think this should probably be committed anyways - even if it isn't the fix for this.

By: Dave Bowerman (dbowerman) 2007-09-18 01:13:33

patch applied in rev 449 - thanks Qwell :)