Operating system name and version: Ubuntu 12.04.4 LT, x86_64
Version of smsd: 3.1.15
Smsd installed from: sources
Name and model of a modem / phone: MicroHard IPn4G
Interface: serial

My modem only support sms messages in text mode, not PDU. Therefore, I added the option 'sms_mode' in the conf file for this device. In this case, the GSM character conversion must be disabled.

devices = MicroHard_IPn4G logfile = /var/log/syslog infofile=/var/run/smsd/smsd.working pidfile=/var/run/smsd/smsd.pid loglevel = 7 #Default value: 4 for logfile, 7 for syslog. # Settings to run smsd without root priviledges: user = root group = dialout delaytime = 1 outgoing = /var/spool/sms/outgoing checked = /var/spool/sms/checked failed = /var/spool/sms/failed incoming = /var/spool/sms/incoming [MicroHard_IPn4G] device = /dev/ttyS4 rtscts = no pre_init = no read_timeout = 1 serial_login = admin serial_password = password pin = ignore check_network = 0 status_signal_quality = no incoming = no # sms PDU mode (0) or text mode (1) sms_mode = 1 cs_convert = no detect_unexpected_input = no

For more details on the other options, check my previous thread.

Basically, the patch reads the sms_mode in the conf file. In text mode, make_pdu() isn't used and the message buffer is copied as is.
diff --git a/smstools3/src/smsd.c b/smstools3/src/smsd.c index 3cc6377..4175619 100755 --- a/smstools3/src/smsd.c +++ b/smstools3/src/smsd.c @@ -3761,29 +3761,41 @@ int send_part(char* from, char* to, char* text, int textlen, int alphabet, int w } } - // Compose the modem command - make_pdu(to,text,textlen,alphabet,flash,report,with_udh,udh_data,DEVICE.mode,pdu,validity, replace_msg, system_msg, to_type, smsc); - if (strcasecmp(DEVICE.mode,"old")==0) - sprintf(command,"AT+CMGS=%i\r",(int)strlen(pdu)/2); - else - sprintf(command,"AT%s+CMGS=%i\r", (DEVICE.verify_pdu)? "E1" : "", (int)strlen(pdu)/2-1); // 3.1.4: verify_pdu - - sprintf(command2,"%s\x1A",pdu); - - if (store_sent_pdu) + if(DEVICE.sms_mode == 0) { - char *title = "PDU: "; + make_pdu(to,text,textlen,alphabet,flash,report,with_udh,udh_data,DEVICE.mode,pdu,validity, replace_msg, system_msg, to_type, smsc); + if (strcasecmp(DEVICE.mode,"old")==0) + sprintf(command,"AT+CMGS=%i\r",(int)strlen(pdu)/2); + else + sprintf(command,"AT%s+CMGS=%s\r", (DEVICE.verify_pdu)? "E1" : "", to); // 3.1.4: verify_pdu - if (!outgoing_pdu_store) - { - if ((outgoing_pdu_store = (char *)malloc(strlen(title) +strlen(pdu) +2))) - *outgoing_pdu_store = 0; - } - else - outgoing_pdu_store = (char *)realloc((void *)outgoing_pdu_store, strlen(outgoing_pdu_store) +strlen(title) +strlen(pdu) +2); + sprintf(command2,"%s\x1A",text); + + if (store_sent_pdu) + { + char *title = "PDU: "; - if (outgoing_pdu_store) - sprintf(strchr(outgoing_pdu_store, 0), "%s%s\n", title, pdu); + if (!outgoing_pdu_store) + { + if ((outgoing_pdu_store = (char *)malloc(strlen(title) +strlen(pdu) +2))) + *outgoing_pdu_store = 0; + } + else + outgoing_pdu_store = (char *)realloc((void *)outgoing_pdu_store, strlen(outgoing_pdu_store) +strlen(title) +strlen(pdu) +2); + + if (outgoing_pdu_store) + sprintf(strchr(outgoing_pdu_store, 0), "%s%s\n", title, pdu); + } + } + else if(DEVICE.sms_mode == 1) + { + // Compose the modem command in text mode + sprintf(command,"AT%s+CMGS=%s\r", (DEVICE.verify_pdu)? "E1" : "", to); // 3.1.4: verify_pdu + char buf[160]; + strncpy(buf, text, textlen); + buf[textlen] = '\0'; + sprintf(command2,"%s\x1A",buf); + writelogfile(LOG_NOTICE, 0, "sent, Message: %s=>%s, l=%i.", text, buf, textlen); } @@ -3878,11 +3892,11 @@ int send_part(char* from, char* to, char* text, int textlen, int alphabet, int w { char answer2[1024]; - put_command("ATE0\r", answer2 ,sizeof(answer2), 1, EXPECT_OK_ERROR); + put_command("\r", answer2 ,sizeof(answer2), 1, EXPECT_OK_ERROR); } // Check answer if (strstr(answer,"OK")) diff --git a/smstools3/src/smsd_cfg.c b/smstools3/src/smsd_cfg.c index 7884efa..cb3e4bd 100755 --- a/smstools3/src/smsd_cfg.c +++ b/smstools3/src/smsd_cfg.c @@ -335,6 +335,10 @@ void initcfg() devices[i].keep_messages = 0; devices[i].trust_spool = 1; devices[i].smsc_pdu = 0; + devices[i].sms_mode = 0; devices[i].telnet_login[0] = 0; snprintf(devices[i].telnet_login_prompt, sizeof(devices[i].telnet_login_prompt), "%s", TELNET_LOGIN_PROMPT_DEFAULT); snprintf(devices[i].telnet_login_prompt_ignore, sizeof(devices[i].telnet_login_prompt_ignore), "%s", TELNET_LOGIN_PROMPT_IGNORE_DEFAULT); @@ -1647,6 +1651,24 @@ int readcfg() startuperror(yesno_error, name, value); } else + if (strcasecmp(name,"sms_mode")==0) + { + if ((NEWDEVICE.sms_mode = yesno_check(ask_value(NEWDEVICE.name, name, value))) == -1) + startuperror(yesno_error, name, value); + } + else if (strcasecmp(name,"telnet_login")==0) strcpy2(NEWDEVICE.telnet_login, ask_value(NEWDEVICE.name, name, value)); else diff --git a/smstools3/src/smsd_cfg.h b/smstools3/src/smsd_cfg.h index 70221c2..5f6c88e 100755 --- a/smstools3/src/smsd_cfg.h +++ b/smstools3/src/smsd_cfg.h @@ -221,6 +221,10 @@ typedef struct char stopstring[100]; // 3.1.7: Command(s) to send to the modem when a devicespooler is stopping. int trust_spool; // 3.1.9 int smsc_pdu; // 3.1.12: 1 if smsc is included in the PDU. + int sms_mode; // 0 for PDU, 1 for TEXT char telnet_login[64]; // 3.1.12: Settings for telnet. char telnet_login_prompt[64]; char telnet_login_prompt_ignore[64];

Hello,
I think I have the same issue with my gprs modem which is a Siemens MC55 and supports only AT+CMGF=1 for text mode, pdu mode is not supported by this modem.
Can I have instructions for applying this patch to my system ?

Thanks.

Operating system name and version: Debian 8.3, x86_64
Version of smsd: 3.1.15
Smsd installed from: standard repository
Name and model of a modem / phone: Siemens MC55
Interface: serial

Not sure I understand your difficulty.
It's been a while, but we're running the version of smsd, so Linux patch tool should be able to apply the fix on source.
Then you'll probably need to insert your CMGF somewhere.
Does that help?
Thanks

Hello,
I am not using the source but the debian repository package.

It's complicated to support so many different modems. However, the download page shows a recent 3.1.16 beta version which may address this in a better way.

No matter if you try 3.1.16 or patch 3.1.15, you have no choice but downloading the source and rebuild.
Looking at the Makefile, it should be straight forward:
make compile

Just make sure the package is removed before you install the custom tool.

sorry I don't understand this point, there are many modems yes, but only two options CMGF=0 and CMGF=1.
Can you tell instructions to patch the program ?

thanks

pkeroulas wrote

It's complicated to support so many different modems. However, the download page shows a recent 3.1.16 beta version which may address this in a better way.

No matter if you try 3.1.16 or patch 3.1.15, you have no choice but downloading the source and rebuild.
Looking at the Makefile, it should be straight forward:
make compile

[ Select ]

Just make sure the package is removed before you install the custom tool.

I have installed 3.1.16, it doesn't seem to solve the problem

Well, I have attempted to apply your patch by hand. didn't change the behavior of the program. So I have taken drastical path: simply replaced AT+CMGF=0 to AT+CMGF=1 in "Selecting PDU mode" in modeminit I think, this resolved my issue.