Merge "[Feature][T8TSK-291]modify the sensitive information in syslog" into GSW3.0-No-Connman
diff --git a/src/lynq/framework/lynq-ril-service/src/mtk_ril_commands.h b/src/lynq/framework/lynq-ril-service/src/mtk_ril_commands.h
index eb1f8d0..4b10fce 100755
--- a/src/lynq/framework/lynq-ril-service/src/mtk_ril_commands.h
+++ b/src/lynq/framework/lynq-ril-service/src/mtk_ril_commands.h
@@ -78,7 +78,9 @@
 { RIL_REQUEST_MODEM_POWEROFF,dispatchVoid,responseVoid},
 { RIL_REQUEST_MODEM_POWERON,dispatchVoid,responseVoid},
 //SIM
-{ RIL_REQUEST_QUERY_ICCID, dispatchVoid, responseString},
+//xf.li@20231120 modify for T8TSK-291 start
+{ RIL_REQUEST_QUERY_ICCID, dispatchVoid, responseStringNoLog},
+//xf.li@20231120 modify for T8TSK-291 end
 //Keepalive
 #ifdef KEEP_ALIVE
 { RIL_REQUEST_START_KEEPALIVE_PRO,dispatchStartKeepalivePro,responseInts},
diff --git a/src/lynq/framework/lynq-ril-service/src/ril.cpp b/src/lynq/framework/lynq-ril-service/src/ril.cpp
index 011a083..db7d129 100755
--- a/src/lynq/framework/lynq-ril-service/src/ril.cpp
+++ b/src/lynq/framework/lynq-ril-service/src/ril.cpp
@@ -304,6 +304,9 @@
 static int responseFailCause(Parcel &p, void *response, size_t responselen);
 static int responseStrings(Parcel &p, void *response, size_t responselen);
 static int responseString(Parcel &p, void *response, size_t responselen);
+//xf.li@20231120 modify for T8TSK-291 start
+static int responseStringNoLog(Parcel &p, void *response, size_t responselen);
+//xf.li@20231120 modify for T8TSK-291 end
 static int responseVoid(Parcel &p, void *response, size_t responselen);
 static int responseCallList(Parcel &p, void *response, size_t responselen);
 static int responseSMS(Parcel &p, void *response, size_t responselen);
@@ -366,7 +369,10 @@
 void initRequestInfo(RequestInfo *pRI, int  request, int mode, RIL_SOCKET_ID soc_id);
 void getVoiceAndDataRegistrationState(RIL_SOCKET_ID soc_id);
 const int waitResponse(int token);
-
+//xf.li@20231120 modify for T8TSK-291 start
+static void encode_log(char str[], int n);
+static void decode_log(char str[], int n);
+//xf.li@20231120 modify for T8TSK-291 end
 #ifdef RIL_SHLIB
 #if defined(ANDROID_MULTI_SIM)
 extern "C" void RIL_onUnsolicitedResponse(int unsolResponse, const void *data,
@@ -2162,7 +2168,20 @@
     writeStringToParcel(p, (const char *)response);
     return 0;
 }
+//xf.li@20231120 modify for T8TSK-291 start
+static int responseStringNoLog(Parcel &p, void *response, size_t responselen) {
+    /* one string only */
+    startResponse;
+    char log_buf[512] = {0};
+    snprintf(log_buf, sizeof(log_buf), "%s", (char*)response);
+    encode_log(log_buf, 6);
+//    appendPrintBuf("%s%s", printBuf, "(has got)");
+    closeResponse;
 
+    writeStringToParcel(p, (const char *)response);
+    return 0;
+}
+//xf.li@20231120 modify for T8TSK-291 end
 static int responseVoid(Parcel &p, void *response, size_t responselen) {
     startResponse;
     removeLastChar;
@@ -2509,11 +2528,99 @@
     }
 
     startResponse;
-    appendPrintBuf("%slen=%d,%s", printBuf, responselen, (char*)response);
+    //xf.li@20231120 modify for T8TSK-291 start 
+    char log_buf[512] = {0};
+    if(strncmp("+CNUM:", (char*)response, strlen("+CNUM:")) == 0)
+    {
+        snprintf(log_buf, sizeof(log_buf), "len=%d,%s", responselen, (char*)response);
+        encode_log(log_buf, 6);
+    }
+    else
+    {
+        appendPrintBuf("%slen=%d,%s", printBuf, responselen, (char*)response);
+    }
+    //xf.li@20231120 modify for T8TSK-291 end
     closeResponse;
     return 0;
 }
+//xf.li@20231120 modify for T8TSK-291 start
+static void string_to_hex(unsigned char *Buff, unsigned int BuffLen, char *OutputStr)
+{
+    int i = 0;
+    char TempBuff[512*2 + 1] = {0};
+    char strBuff[512*2 + 1] = {0};
 
+    if(BuffLen > 512)
+    {
+        RLOGD("string_to_hex error!");
+        return;
+    }
+    for(i = 0; i<BuffLen;i++)
+    {
+        sprintf(TempBuff,"%02x",(unsigned char)Buff[i]);
+        strncat(strBuff,TempBuff,BuffLen*2);
+    }
+    strncpy(OutputStr, strBuff, BuffLen*2);
+    return;
+}
+
+static void encode_log(char input_str[], int n)
+{
+    char c;
+    int i;
+    char str[512*2 + 1] = {0};
+    string_to_hex((unsigned char *)input_str, strlen(input_str), str);
+    for( i=0; i<strlen(str); i++)
+    {
+        c = str[i];
+        if(c>=' ' && c <= 'z')
+        {
+            if(c + n%90 <= 'z')
+            { 
+                str[i] = (char)(c + n%90);  //encode
+            }
+            else
+            {  //After encoding, the space beyond the end reaches ‘z’, and the cycle starts from the beginning.
+                str[i] = (char)(c + n%90 - 90);//encode
+            }
+        }
+        else
+        {
+            str[i]=c;
+        }
+    }
+
+    appendPrintBuf("%s%s", printBuf, str);
+}
+//decode() only for analysis log
+static void decode_log(char str[], int n)
+{
+    char c;
+    int i;
+    
+    for(i=0; i<strlen(str); i++)
+    {
+        c=str[i];
+        
+        if(c >= ' ' && c <= 'z')
+        {
+            if(c - n%90 >= ' ')
+            {
+                str[i] = (char)(c - n%90); //decode
+            }
+            else
+            {
+                //After decoding, there are over spaces until ‘z’, and the lowercase letters are processed into printable information by looping
+                str[i] = (char)(c - n%90 + 90); //decode
+            }
+        }
+        else
+        {
+            str[i]=c;
+        }
+    }
+}
+//xf.li@20231120 modify for T8TSK-291 end
 
 static int responseSIM_IO(Parcel &p, void *response, size_t responselen) {
     if (response == NULL) {
@@ -5715,7 +5822,7 @@
 {
     char *pos, *pos2;
     int argc = 0;
-    printf("warren test cmd=%s\n",cmd);
+    printf("LYNQ_REQUEST cmd=%s\n",cmd);
     pos = cmd;
     while (1) {
         // Trim the space characters.
@@ -6016,7 +6123,7 @@
                     argc = argc+1;
                     for(int t = 0;t<argc;t++)
                     {
-                        RLOGD("warren test argv[%d]=%s\n",t,argv[t]);
+                        RLOGD("LYNQ_REQUEST argv[%d]=%s\n",t,argv[t]);
                     }
                     COMMAND *command = find_command(argv[0]);
                     if(!command)
diff --git a/src/lynq/framework/lynq-ril-service/src/ril_commands.h b/src/lynq/framework/lynq-ril-service/src/ril_commands.h
index 85878ee..362bb7d 100755
--- a/src/lynq/framework/lynq-ril-service/src/ril_commands.h
+++ b/src/lynq/framework/lynq-ril-service/src/ril_commands.h
@@ -25,7 +25,9 @@
     {RIL_REQUEST_ENTER_NETWORK_DEPERSONALIZATION, dispatchStrings, responseInts},
     {RIL_REQUEST_GET_CURRENT_CALLS, dispatchVoid, responseCallList},
     {RIL_REQUEST_DIAL, dispatchDial, responseVoid},
-    {RIL_REQUEST_GET_IMSI, dispatchStrings, responseString},
+    //xf.li@20231120 modify for T8TSK-291 start
+    {RIL_REQUEST_GET_IMSI, dispatchStrings, responseStringNoLog},
+    //xf.li@20231120 modify for T8TSK-291 end
     {RIL_REQUEST_HANGUP, dispatchInts, responseVoid},
     {RIL_REQUEST_HANGUP_WAITING_OR_BACKGROUND, dispatchVoid, responseVoid},
     {RIL_REQUEST_HANGUP_FOREGROUND_RESUME_BACKGROUND, dispatchVoid, responseVoid},
diff --git a/src/telephonyware/3.0/atcid/atci/src/platform/atcid_common.c b/src/telephonyware/3.0/atcid/atci/src/platform/atcid_common.c
index e775d6b..f1e859f 100755
--- a/src/telephonyware/3.0/atcid/atci/src/platform/atcid_common.c
+++ b/src/telephonyware/3.0/atcid/atci/src/platform/atcid_common.c
@@ -35,7 +35,7 @@
  * any receiver's applicable license agreements with MediaTek Inc.

  */

 

-

+#include <string.h>

 #include "atcid_common.h"

 #include "atcid.h"

 

@@ -44,12 +44,44 @@
 

 int atci_property_get(const char *key, char *value, const char *default_value) {

     int ali_pro_res = property_get(key, value, default_value);

-    LOGATCI(LOG_DEBUG, "get key is %s, value is %s", key, value);

+//xf.li@20231120 modify for T8TSK-291 start

+    if((strcmp(key, "vendor.ril.iccid.sim1") != 0) && (strcmp(key, "vendor.ril.iccid.sim2") != 0))

+    {

+        LOGATCI(LOG_DEBUG, "get key is %s, value is %s", key, value);

+    }

+    else

+    {

+	if(value != NULL && strlen(value) > 0)

+        {

+            LOGATCI(LOG_DEBUG, "get key is %s, value is (has got)", key);

+        }

+        else

+        {

+            LOGATCI(LOG_DEBUG, "get key is %s, value is ", key);

+        }

+    }

+//xf.li@20231120 modify for T8TSK-291 end

     return ali_pro_res;

 }

 

 int atci_property_set(const char *key, const char *value) {

     int ret_val = property_set(key, value);

-    LOGATCI(LOG_DEBUG, "set key is %s, value is %s", key, value);

+//xf.li@20231120 modify for T8TSK-291 start

+    if((strcmp(key, "vendor.ril.iccid.sim1") != 0) && (strcmp(key, "vendor.ril.iccid.sim2") != 0))

+    {

+        LOGATCI(LOG_DEBUG, "set key is %s, value is %s", key, value);

+    }

+    else

+    {

+	if(value != NULL && strlen(value) > 0)

+        {

+            LOGATCI(LOG_DEBUG, "set key is %s, value is (has got)", key);

+        }

+        else

+        {

+            LOGATCI(LOG_DEBUG, "set key is %s, value is ", key);

+        }

+    }

+//xf.li@20231120 modify for T8TSK-291 end

     return ret_val;

 }

diff --git a/src/telephonyware/3.0/hardware/ril/fusion/libril/rilmipc/framework/core/RfxStatusManager.cpp b/src/telephonyware/3.0/hardware/ril/fusion/libril/rilmipc/framework/core/RfxStatusManager.cpp
index 026b7df..1bade4a 100644
--- a/src/telephonyware/3.0/hardware/ril/fusion/libril/rilmipc/framework/core/RfxStatusManager.cpp
+++ b/src/telephonyware/3.0/hardware/ril/fusion/libril/rilmipc/framework/core/RfxStatusManager.cpp
@@ -138,8 +138,23 @@
         //    RFX_LOG_D(RFX_LOG_TAG, "setValue() slot(%d) key = %s, value = [XXX]",
         //                           m_slot_id, getKeyString(key));
         //} else {
-            RFX_LOG_D(RFX_LOG_TAG, "setValue() slot(%d) key = %s, value = [%s]",
+//xf.li@20231120 modify for T8TSK-291 start
+            if(strcmp(getKeyString(key), "RFX_STATUS_KEY_SIM_ICCID") == 0)
+            {
+                RFX_LOG_D(RFX_LOG_TAG, "setValue() slot(%d) key = %s, value = [has got]",
+                                   m_slot_id, getKeyString(key));
+            }
+            else if(strcmp(getKeyString(key), "RFX_STATUS_KEY_GSM_IMSI") == 0)
+            {
+                RFX_LOG_D(RFX_LOG_TAG, "setValue() slot(%d) key = %s, value = [has got]",
+                                   m_slot_id, getKeyString(key));
+            }
+            else
+            {
+                RFX_LOG_D(RFX_LOG_TAG, "setValue() slot(%d) key = %s, value = [%s]",
                                    m_slot_id, getKeyString(key), value.toString().string());
+            }
+//xf.li@20231120 modify for T8TSK-291 end
         //}
     } else {
         RfxVariant old = m_status_list[key]->value;
@@ -154,12 +169,26 @@
             //            force_notify ? "true" : "false",
             //            is_default ? "true" : "false");
             //} else {
-                RFX_LOG_D(RFX_LOG_TAG, "setValue() slot(%d) key = %s, old = [%s], new = [%s],\
+//xf.li@20231120 modify for T8TSK-291 start
+            if((strcmp(getKeyString(key), "RFX_STATUS_KEY_SIM_ICCID") == 0) || (strcmp(getKeyString(key), "RFX_STATUS_KEY_GSM_IMSI") == 0))
+            {
+                    RFX_LOG_D(RFX_LOG_TAG, "setValue() slot(%d) key = %s, old = [(has got)], new = [(has got)],\
+is_force = %s, is_default = %s",
+                            m_slot_id, getKeyString(key),
+                            force_notify ? "true" : "false",
+                            is_default ? "true" : "false");
+            }
+            else
+            {
+                    RFX_LOG_D(RFX_LOG_TAG, "setValue() slot(%d) key = %s, old = [%s], new = [%s],\
 is_force = %s, is_default = %s",
                         m_slot_id, getKeyString(key), old.toString().string(),
                         value.toString().string(),
                         force_notify ? "true" : "false",
                         is_default ? "true" : "false");
+            }
+
+//xf.li@20231120 modify for T8TSK-291 end
             //}
         }
 
diff --git a/src/telephonyware/3.0/hardware/ril/fusion/libril/rilmipc/telephony/sim/RpSimController.cpp b/src/telephonyware/3.0/hardware/ril/fusion/libril/rilmipc/telephony/sim/RpSimController.cpp
index a76d183..ae5896a 100644
--- a/src/telephonyware/3.0/hardware/ril/fusion/libril/rilmipc/telephony/sim/RpSimController.cpp
+++ b/src/telephonyware/3.0/hardware/ril/fusion/libril/rilmipc/telephony/sim/RpSimController.cpp
@@ -48,7 +48,9 @@
 
 static bool isSimInserted(mipc_sim_ps_id_enum sim_ps_id) {
     String8 iccid = RFX_OBJ_GET_INSTANCE(RfxRootController)->getStatusManager(mipc_sim_id_to_slot_id(sim_ps_id))->getString8Value(RFX_STATUS_KEY_SIM_ICCID);
-    RFX_LOG_D(getTag(sim_ps_id).c_str(),"%s, iccid: %s", __FUNCTION__, iccid.string());
+//xf.li@20231120 modify for T8TSK-291 start
+    RFX_LOG_D(getTag(sim_ps_id).c_str(),"%s, iccid: (has got)", __FUNCTION__);
+//xf.li@20231120 modify for T8TSK-291 end
     if ((!iccid.isEmpty())){
         return true;
     }
@@ -199,7 +201,9 @@
         str.append(",eid:").append("");
     }
     if(strlen(ptr->iccid) > 0) {
-        str.append(",iccid:").append(string(ptr->iccid));
+//xf.li@20231120 modify for T8TSK-291 start
+        str.append(",iccid:").append("(has got)");
+//xf.li@20231120 modify for T8TSK-291 end
     } else {
         str.append(",iccid:").append("");
     }
@@ -337,12 +341,15 @@
     string str("");
     str.append("request:").append(requestToString(info->pCI->requestNumber));
     if(strlen(result_ptr->iccid) > 0 ) {
-        str.append("iccid: ").append(result_ptr->iccid);
+//xf.li@20231120 modify for T8TSK-291 start
+        str.append("iccid: ").append("(has got)");
+//xf.li@20231120 modify for T8TSK-291 end
     } else {
         str.append("");
     }
-
-    RFX_LOG_D(getTag(sim_ps_id).c_str(),"%s, iccid: %s, result: %d", __FUNCTION__, str.c_str(), result_ptr->result_code);
+//xf.li@20231120 modify for T8TSK-291 start
+    RFX_LOG_D(getTag(sim_ps_id).c_str(),"%s, iccid: (has got), result: %d", __FUNCTION__, result_ptr->result_code);
+//xf.li@20231120 modify for T8TSK-291 end
     if(result_ptr->result_code != MIPC_RESULT_SUCCESS) {
         //result code to ril error transfer
         rfx_enqueue_response_message(NULL,cb_priv_ptr,mipc_sim_id_to_slot_id(sim_ps_id),result_ptr->result_code);
@@ -424,7 +431,9 @@
     string str("");
     str.append("request:").append(requestToString(info->pCI->requestNumber));
     str.append(",result_code:").append(std::to_string(ptr->result_code));
-    str.append(",imsi:").append(string(ptr->imsi));
+//xf.li@20231120 modify for T8TSK-291 start
+    str.append(",imsi:").append("(has got)");
+//xf.li@20231120 modify for T8TSK-291 end
     str.append(",mnc_len:").append(std::to_string(ptr->mnc_len));
     return str;
 }
@@ -744,7 +753,9 @@
 }
 
 static void mipc_sim_iccid_ind_cb(mipc_sim_ps_id_enum sim_ps_id, mipc_sim_iccid_ind_struct *result_ptr, void *cb_priv_ptr) {
-    RFX_LOG_D(getTag(sim_ps_id).c_str(), "%s MIPC_SIM_ICCID_IND, ICCID=%s", __FUNCTION__,string(result_ptr->iccid).c_str());
+//xf.li@20231120 modify for T8TSK-291 start
+    RFX_LOG_D(getTag(sim_ps_id).c_str(), "%s MIPC_SIM_ICCID_IND, ICCID= has got", __FUNCTION__);
+//xf.li@20231120 modify for T8TSK-291 end
     String8 iccid("");
     iccid.append(string(result_ptr->iccid).c_str());
     RFX_OBJ_GET_INSTANCE(RfxRootController)->getStatusManager(mipc_sim_id_to_slot_id(sim_ps_id))->setString8Value(RFX_STATUS_KEY_SIM_ICCID, iccid);
@@ -755,7 +766,9 @@
 }
 
 static void mipc_sim_imsi_get_after_sim_statechanged_cb(mipc_sim_ps_id_enum sim_ps_id, mipc_sim_imsi_struct *result_ptr, void *cb_priv_ptr) {
-    RFX_LOG_D(getTag(sim_ps_id).c_str(),"%s %s,%d", __FUNCTION__,result_ptr->imsi,result_ptr->result_code);
+    //xf.li@20231120 modify for T8TSK-291 start
+    RFX_LOG_D(getTag(sim_ps_id).c_str(),"%s imsi (has got),%d", __FUNCTION__,result_ptr->result_code);
+    //xf.li@20231120 modify for T8TSK-291 end
     string mccMncKey("");
     char mccmnc[7] = { 0 };
     mccMncKey.append(PROPERTY_GSM_MCC_MNC_PREIFX);
@@ -767,7 +780,9 @@
         rfx_property_get(mccMncKey.c_str(), old_value, "");
         RFX_LOG_D(getTag(sim_ps_id).c_str(),"%s S imsi", __FUNCTION__);
         imsi.append(string(result_ptr->imsi).c_str());
-        RFX_LOG_D(getTag(sim_ps_id).c_str(),"%s E imsi: %s, mnc len: %d", __FUNCTION__,imsi.string(), result_ptr->mnc_len);
+//xf.li@20231120 modify for T8TSK-291 start
+        RFX_LOG_D(getTag(sim_ps_id).c_str(),"%s E imsi: (has got), mnc len: %d", __FUNCTION__, result_ptr->mnc_len);
+//xf.li@20231120 modify for T8TSK-291 end
         memcpy(mccmnc,result_ptr->imsi,3+result_ptr->mnc_len);
         RFX_LOG_D(getTag(sim_ps_id).c_str(), "%s old_mccmcn:%s, new_mccmnc:%s", __FUNCTION__,(old_value==NULL ? "":string(old_value).c_str()), ((mccmnc==NULL ? "":string(mccmnc).c_str())));
         if(string(mccmnc) != string(old_value) ) {
diff --git a/src/telephonyware/3.0/hardware/ril/platformlib/libmipc/lib/src/api/mipc_sim_api.c b/src/telephonyware/3.0/hardware/ril/platformlib/libmipc/lib/src/api/mipc_sim_api.c
index 2b68b17..9ebec1a 100644
--- a/src/telephonyware/3.0/hardware/ril/platformlib/libmipc/lib/src/api/mipc_sim_api.c
+++ b/src/telephonyware/3.0/hardware/ril/platformlib/libmipc/lib/src/api/mipc_sim_api.c
@@ -368,7 +368,9 @@
         mtkLogD(LOG_TAG, "[id%d]val_len %d, MIPC_MAX_IMSI_LEN %d", msg_cnf_ptr->hdr.msg_sim_ps_id, val_len, MIPC_MAX_IMSI_LEN);
         MEMSET(result_ptr->imsi, 0, MIPC_MAX_IMSI_LEN+1);
         MEMCPY(result_ptr->imsi, val_ptr, val_len < (MIPC_MAX_IMSI_LEN) ? val_len:(MIPC_MAX_IMSI_LEN+1));
-        mtkLogD(LOG_TAG, "[id%d]imsi %s", msg_cnf_ptr->hdr.msg_sim_ps_id, result_ptr->imsi);
+//xf.li@20231120 modify for T8TSK-291 start
+        mtkLogD(LOG_TAG, "[id%d]imsi (has got)", msg_cnf_ptr->hdr.msg_sim_ps_id);
+//xf.li@20231120 modify for T8TSK-291 end
         val_ptr = mipc_msg_get_val_ptr(msg_cnf_ptr, MIPC_SIM_IMSI_CNF_T_MNC_LEN, &val_len);
         if (NULL == val_ptr) break;
         result_ptr->mnc_len =  (uint8_t)(*((uint32_t*)val_ptr));
diff --git a/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/framework/base/RfxTransferUtils.cpp b/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/framework/base/RfxTransferUtils.cpp
index 5f0362a..ad71e64 100755
--- a/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/framework/base/RfxTransferUtils.cpp
+++ b/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/framework/base/RfxTransferUtils.cpp
@@ -1725,7 +1725,20 @@
     dataStart;
     if (p->dataAvail() > 0) {
         string8 = strdupReadString(p);
-        appendPrintBuf("%s",string8);
+//xf.li@20231120 modify for T8TSK-291 start
+        if(strcmp(requestToString(id), "RIL_REQUEST_QUERY_ICCID") == 0)
+        {
+                appendPrintBuf("(has got)");
+        }
+        else if(strcmp(requestToString(id), "GET_IMSI") == 0)
+        {
+                appendPrintBuf("(has got)");
+        }
+        else
+        {
+                appendPrintBuf("%s",string8);
+        }
+//xf.li@20231120 modify for T8TSK-291 end
         stringlen = strlen(string8);
     }
 
@@ -2247,7 +2260,16 @@
                 goto calloc_error;
             }
             p->read(line, size);
-            appendPrintBuf("line=%s,", line);
+//xf.li@20231120 modify for T8TSK-291 start
+            if(strncmp("+CNUM:", line, strlen("+CNUM:")) == 0)
+            {
+	        appendPrintBuf("line=+CNUM: (has got),");
+            }
+            else
+            {
+                appendPrintBuf("line=%s,", line);
+            }
+//xf.li@20231120 modify for T8TSK-291 end
         }
     }
     dataEnd;
@@ -2263,7 +2285,16 @@
 
     dataStart;
     if( line != NULL){
-        appendPrintBuf("size=%d, line=%s", size,line);
+//xf.li@20231120 modify for T8TSK-291 start
+        if(strncmp("+CNUM:", line, strlen("+CNUM:")) == 0)
+        {
+	    appendPrintBuf("size=%d, line=+CNUM: (has got)", size);
+        }
+        else
+        {
+            appendPrintBuf("size=%d, line=%s", size,line);
+        }
+//xf.li@20231120 modify for T8TSK-291 end
     }
 
     dataEnd;
diff --git a/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/framework/core/RfxStatusManager.cpp b/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/framework/core/RfxStatusManager.cpp
index 0888c20..2b01455 100644
--- a/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/framework/core/RfxStatusManager.cpp
+++ b/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/framework/core/RfxStatusManager.cpp
@@ -120,8 +120,18 @@
     if (m_status_list[key] == NULL) {
         m_status_list[key] = new StatusListEntry();
         m_status_list[key]->value = value;
-        RFX_LOG_D(RFX_LOG_TAG, "setValue() slot(%d) key = %s, value = [%s]",
+//xf.li@20231120 modify for T8TSK-291 start
+        if((strcmp(getKeyString(key), "RFX_STATUS_KEY_SIM_ICCID") == 0) || (strcmp(getKeyString(key), "RFX_STATUS_KEY_GSM_IMSI") == 0))
+        {
+            RFX_LOG_D(RFX_LOG_TAG, "setValue() slot(%d) key = %s, value = (has got)",
+                               m_slot_id, getKeyString(key));
+        }
+        else
+        {
+            RFX_LOG_D(RFX_LOG_TAG, "setValue() slot(%d) key = %s, value = [%s]",
                                m_slot_id, getKeyString(key), value.toString().string());
+        }
+//xf.li@20231120 modify for T8TSK-291 end
     } else {
         RfxVariant old = m_status_list[key]->value;
         bool notify = false;
diff --git a/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/telephony/sim/RpSimController.cpp b/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/telephony/sim/RpSimController.cpp
index cb837bf..1f57a7b 100644
--- a/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/telephony/sim/RpSimController.cpp
+++ b/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/telephony/sim/RpSimController.cpp
@@ -901,7 +901,9 @@
     if (message->getError() == RIL_E_SUCCESS) {
         // Get IMSI then notify.
         String8 str8(req->readString16Inplace(&s16Len));
-        RLOGD("[RpSimController] handleGetImsiRsp : %s", str8.string());
+//xf.li@20231120 modify for T8TSK-291 start
+        RLOGD("[RpSimController] handleGetImsiRsp : (has got)");
+//xf.li@20231120 modify for T8TSK-291 end
     #if  (defined(TELEPHONYWARE) || defined(TELE_FWK))
         source = choiceDestByCardType();
     #else