Summary:ASTERISK-07331: sms receiving does not work
Reporter:Fabian Mueller (fmueller)Labels:
Date Opened:2006-07-13 03:55:23Date Closed:2006-07-14 08:34:41
Versions:Frequency of
Environment:Attachments:( 0) app_sms.c.patch
( 1) log.asterisk
Description:The Problem:
- I can receive SMS with Asterisk 1.0.9 and Asterisk 1.2.7 and Asterisk
- I cannot receive SMS with Asterisk 1.2.8 and Asterisk

Background information
- I use an AVM Fritz! ISDN Card (BRI) with chan_capi-cm-0.6.5
- I use these extensions to receive the SMS:

 exten => _X.,1,SMS(${EXTEN}|a)
 exten => _X.,2,Hangup
- The attached file shows CLI output with capi debug enabled.
Comments:By: Serge Vecher (serge-v) 2006-07-13 08:42:10

chan_capi is not supported officially by Asterisk project. Can you reproduce this with chan_misdn?

By: jmb (jmb) 2006-07-13 10:57:54

I have a similar problem. I can't send sms with asterisk
As I did a couple of upgrades at the same time, I was not sure if it was related to chan_capi-cm, the eicon driver or asterisk.

Finally, I found that sending sms works with 1.2.7, but it does not with 1.2.8, 1.2.9 and I looked a the differences between 1.2.7 and 1.2.8 and I found (I think) an error. With the following patch against, sending sms works again:

--- apps/app_sms.c.old  2006-07-13 16:11:36.000000000 +0200
+++ apps/app_sms.c      2006-07-13 16:13:29.000000000 +0200
@@ -1205,7 +1205,8 @@
       f.datalen = samples * 2;
       f.offset = AST_FRIENDLY_OFFSET;
-       f.data = buf + AST_FRIENDLY_OFFSET;
+       f.mallocd = 0;
+       f.data = buf;
       f.samples = samples;
       f.src = "app_sms";
       /* create a buffer containing the digital sms pattern */

By: Serge Vecher (serge-v) 2006-07-13 11:08:22

jmb: please confirm your disclaimer status and upload the change as a patchfile, please. fmueller: please try the patch and give feedback. Thanks!

By: jmb (jmb) 2006-07-14 01:39:41

Patchfile uploaded and disclaimer sent via fax (hopes that it is alright).

By: Fabian Mueller (fmueller) 2006-07-14 03:06:25

The patch fixes my problem. Thanks!

By: Tilghman Lesher (tilghman) 2006-07-14 08:34:41

Committed to 1.2; merged to trunk.  Thanks for the patch (and the disclaimer).