[Feature][T8TSK-291]modify the sensitive information in syslog-2
Only Configure: No
Affected branch: GSW3.0-No-Connman
Affected module: RIL
Is it affected on both ZXIC and MTK: only MTK
Self-test: Yes
Doc Update: No
Change-Id: I9dd5c80912e28f5975059e34e91a2e188f75082e
diff --git a/src/telephonyware/3.0/hardware/ril/platformlib/telematics/libmtkproperty/mtk_properties.c b/src/telephonyware/3.0/hardware/ril/platformlib/telematics/libmtkproperty/mtk_properties.c
index 9dc57f6..c723dd6 100644
--- a/src/telephonyware/3.0/hardware/ril/platformlib/telematics/libmtkproperty/mtk_properties.c
+++ b/src/telephonyware/3.0/hardware/ril/platformlib/telematics/libmtkproperty/mtk_properties.c
@@ -1,4 +1,5 @@
#include <mtk_log.h>
+#include <string.h>
#include "mtk_properties.h"
#define LOG_TAG "mtk_properties"
@@ -13,7 +14,23 @@
int mtk_property_get(const char *key, char *value, const char *default_value)
{
int ali_pro_res = property_get(key, value, default_value);
- mtkLogD(LOG_TAG, "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))
+ {
+ mtkLogD(LOG_TAG, "get key is %s, value is %s", key, value);
+ }
+ else
+ {
+ if(value != NULL && strlen(value) > 0)
+ {
+ mtkLogD(LOG_TAG, "get key is %s, value is (has got)", key);
+ }
+ else
+ {
+ mtkLogD(LOG_TAG, "get key is %s, value is ", key);
+ }
+ }
+//xf.li@20231120 modify for T8TSK-291 end
return ali_pro_res;
}
@@ -25,6 +42,22 @@
int mtk_property_set(const char *key, const char *value)
{
int ret_val = property_set(key, value);
- mtkLogD(LOG_TAG, "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))
+ {
+ mtkLogD(LOG_TAG, "set key is %s, value is %s", key, value);
+ }
+ else
+ {
+ if(value != NULL && strlen(value) > 0)
+ {
+ mtkLogD(LOG_TAG, "set key is %s, value is (has got)", key);
+ }
+ else
+ {
+ mtkLogD(LOG_TAG, "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/libvendor-ril/mtk-rilproxy/framework/core/RfxStatusManager.cpp b/src/telephonyware/3.0/libvendor-ril/mtk-rilproxy/framework/core/RfxStatusManager.cpp
index 2b01455..1b6cdb7 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
@@ -136,13 +136,23 @@
RfxVariant old = m_status_list[key]->value;
bool notify = false;
bool is_dif = (old != value);
-
- 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 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
if ((is_dif && !is_default) || force_notify) {
// if setting default value and not force notify, don't notify
notify = true;