From: Caiwen Zhang <caiwen.zhang@intel.com>
Date: Thu, 9 Jan 2014 18:55:55 +0000 (+0800)
Subject: Fix index mismatch issue
X-Git-Tag: accepted/tizen/generic/20140130.163213~1
X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Ftelephony%2Ftel-plugin-imc.git;a=commitdiff_plain;h=d8583a4816677e294ee1039359353df123ea536d

Fix index mismatch issue

Upper layer will cache the SIM SMS index information, When
read/write/update/delete SIM SMS, the SIM index (start from 1)
will be used, it isn't necessary to change the index.

Change-Id: I32be7417e6e0facaf0b1ac970e2234ad56fb0970
Signed-off-by: Caiwen Zhang <caiwen.zhang@intel.com>
---

diff --git a/src/s_sms.c b/src/s_sms.c
index 813fa02..1129c5d 100644
--- a/src/s_sms.c
+++ b/src/s_sms.c
@@ -628,7 +628,7 @@ static void on_response_sms_save_msg(TcorePending *p, int data_len, const void *
 			pResp = g_slist_nth_data(tokens, 0);
 			if (pResp) {
 				dbg("0: %s", pResp);
-		 		saveMsgInfo.index = (atoi(pResp) - 1); /* IMC index starts from 1 */
+		 		saveMsgInfo.index = atoi(pResp); 
 				saveMsgInfo.result = SMS_SENDSMS_SUCCESS;
 			} else {
 				dbg("No Tokens");
@@ -2114,7 +2114,7 @@ static void _response_get_efsms_data(TcorePending *p, int data_len, const void *
 			util_byte_to_hex((const char *)&msg_status, (char *)encoded_data, 1);
 
 			//Update EF-SMS with just status byte overwritten, rest 175 bytes are same as received in read information
-			cmd_str = g_strdup_printf("AT+CRSM=220,28476,%d, 4, %d, \"%s\"", (req_msg_status->index+1), PDU_LEN_MAX, encoded_data);
+			cmd_str = g_strdup_printf("AT+CRSM=220,28476,%d, 4, %d, \"%s\"", req_msg_status->index, PDU_LEN_MAX, encoded_data);
 			atreq = tcore_at_request_new((const char *)cmd_str, "+CRSM", TCORE_AT_SINGLELINE);
 			pending = tcore_pending_new(tcore_pending_ref_core_object(pending), 0);
 			if (NULL == cmd_str || NULL == atreq || NULL == pending) {
@@ -2433,7 +2433,7 @@ static TReturn delete_msg(CoreObject *obj, UserRequest *ur)
 	if (delete_msg->index == -1) {
 		cmd_str = g_strdup_printf("AT+CMGD=0,4"); // Delete All Messages
 	} else {
-		cmd_str = g_strdup_printf("AT+CMGD=%d,0", delete_msg->index + 1); // Delete specified index
+		cmd_str = g_strdup_printf("AT+CMGD=%d,0", delete_msg->index); // Delete specified index
 	}
 
 	pending = tcore_pending_new(obj, 0);
@@ -2898,7 +2898,7 @@ static TReturn set_msg_status(CoreObject *obj, UserRequest *ur)
 	}
 	msg_status = tcore_user_request_ref_data(ur, NULL);
 
-	cmd_str = g_strdup_printf("AT+CRSM=178,28476,%d,4,%d", (msg_status->index+1), PDU_LEN_MAX);
+	cmd_str = g_strdup_printf("AT+CRSM=178,28476,%d,4,%d", msg_status->index, PDU_LEN_MAX);
 	atreq = tcore_at_request_new((const char *)cmd_str, "+CRSM", TCORE_AT_SINGLELINE);
 	pending = tcore_pending_new(obj, 0);
 	if (NULL == cmd_str || NULL == atreq || NULL == pending) {