Summary: | ASTERISK-10320: MobileStatus crash | ||
Reporter: | tootai (tootai) | Labels: | |
Date Opened: | 2007-09-17 10:50:11 | Date Closed: | 2007-09-18 01:13:33 |
Priority: | Critical | Regression? | No |
Status: | Closed/Complete | Components: | Addons/chan_mobile |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) chan_mobile-standard_app_args.diff | |
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 | ||
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 :) |