Fix improper part number in multipart SMS text messages

[project/uqmi.git] / commands-wms.c

diff --git a/commands-wms.c b/commands-wms.c
index 6fc34df..9b9aac3 100644 (file)
--- a/commands-wms.c
+++ b/commands-wms.c
@@ -178,7 +178,7 @@ static void decode_udh(const unsigned char *data)
                switch (type) {
                case 0:
                        blobmsg_add_u32(&status, "concat_ref", (uint32_t) val[0]);
-                       blobmsg_add_u32(&status, "concat_part", (uint32_t) val[2] + 1);
+                       blobmsg_add_u32(&status, "concat_part", (uint32_t) val[2]);
                        blobmsg_add_u32(&status, "concat_parts", (uint32_t) val[1]);
                        break;
                default:
@@ -246,9 +246,9 @@ pdu_decode_address(char *str, unsigned char *data, int len)
        *str = 0;
 }
 
-static void wms_decode_address(char *str, char *name, unsigned char *data, int len)
+static void wms_decode_address(char *name, unsigned char *data, int len)
 {
-       str = blobmsg_alloc_string_buffer(&status, name, len * 2 + 2);
+       char *str = blobmsg_alloc_string_buffer(&status, name, len * 2 + 2);
        pdu_decode_address(str, data, len);
        blobmsg_add_string_buffer(&status);
 }
@@ -273,7 +273,7 @@ static void cmd_wms_get_message_cb(struct qmi_dev *qmi, struct qmi_request *req,
                return;
 
        if (cur_len) {
-               wms_decode_address(str, "smsc", data, cur_len - 1);
+               wms_decode_address("smsc", data, cur_len - 1);
                data += cur_len;
        }
 
@@ -291,7 +291,7 @@ static void cmd_wms_get_message_cb(struct qmi_dev *qmi, struct qmi_request *req,
 
        if (cur_len) {
                cur_len = (cur_len + 1) / 2;
-               wms_decode_address(str, sent ? "receiver" : "sender", data, cur_len);
+               wms_decode_address(sent ? "receiver" : "sender", data, cur_len);
                data += cur_len + 1;
        }
 
@@ -389,7 +389,7 @@ static void cmd_wms_get_raw_message_cb(struct qmi_dev *qmi, struct qmi_request *
        data = (unsigned char *) res.data.raw_message_data.raw_data;
        str = blobmsg_alloc_string_buffer(&status, NULL, res.data.raw_message_data.raw_data_n * 3);
        for (i = 0; i < res.data.raw_message_data.raw_data_n; i++) {
-               str += sprintf(str, " %02x" + (i ? 0 : 1), data[i]);
+               str += sprintf(str, &" %02x"[i ? 0 : 1], data[i]);
        }
        blobmsg_add_string_buffer(&status);
 }
@@ -401,14 +401,14 @@ static struct {
        const char *smsc;
        const char *target;
        bool flash;
-} send;
+} _send;
 
 
 #define cmd_wms_send_message_smsc_cb no_cb
 static enum qmi_cmd_result
 cmd_wms_send_message_smsc_prepare(struct qmi_dev *qmi, struct qmi_request *req, struct qmi_msg *msg, char *arg)
 {
-       send.smsc = arg;
+       _send.smsc = arg;
        return QMI_CMD_DONE;
 }
 
@@ -416,7 +416,7 @@ cmd_wms_send_message_smsc_prepare(struct qmi_dev *qmi, struct qmi_request *req,
 static enum qmi_cmd_result
 cmd_wms_send_message_target_prepare(struct qmi_dev *qmi, struct qmi_request *req, struct qmi_msg *msg, char *arg)
 {
-       send.target = arg;
+       _send.target = arg;
        return QMI_CMD_DONE;
 }
 
@@ -424,7 +424,7 @@ cmd_wms_send_message_target_prepare(struct qmi_dev *qmi, struct qmi_request *req
 static enum qmi_cmd_result
 cmd_wms_send_message_flash_prepare(struct qmi_dev *qmi, struct qmi_request *req, struct qmi_msg *msg, char *arg)
 {
-       send.flash = true;
+       _send.flash = true;
        return QMI_CMD_DONE;
 }
 
@@ -546,24 +546,24 @@ cmd_wms_send_message_prepare(struct qmi_dev *qmi, struct qmi_request *req, struc
        unsigned char protocol_id = 0x00;
        unsigned char dcs = 0x00;
 
-       if (!send.smsc || !*send.smsc || !send.target || !*send.target) {
+       if (!_send.smsc || !*_send.smsc || !_send.target || !*_send.target) {
                uqmi_add_error("Missing argument");
                return QMI_CMD_EXIT;
        }
 
-       if (strlen(send.smsc) > 16 || strlen(send.target) > 16 || strlen(arg) > 160) {
+       if (strlen(_send.smsc) > 16 || strlen(_send.target) > 16 || strlen(arg) > 160) {
                uqmi_add_error("Argument too long");
                return QMI_CMD_EXIT;
        }
 
-       if (send.flash)
+       if (_send.flash)
                dcs |= 0x10;
 
-       cur += pdu_encode_number(cur, send.smsc, true);
+       cur += pdu_encode_number(cur, _send.smsc, true);
        *(cur++) = first_octet;
        *(cur++) = 0; /* reference */
 
-       cur += pdu_encode_number(cur, send.target, false);
+       cur += pdu_encode_number(cur, _send.target, false);
        *(cur++) = protocol_id;
        *(cur++) = dcs;