diff --git a/services/group_manager/inc/group_operation_common.h b/services/group_manager/inc/group_operation_common.h index 82e29837d027dad609f291ae201cc733c1fc25d1..72e3ecf9105384bef72fd11ff11e625b6f4ed1bb 100644 --- a/services/group_manager/inc/group_operation_common.h +++ b/services/group_manager/inc/group_operation_common.h @@ -75,7 +75,8 @@ int32_t AddDeviceToDatabaseByJson(int32_t osAccountId, int32_t (*generateDevPara int32_t DelGroupFromDb(int32_t osAccountId, const char *groupId); int32_t ConvertGroupIdToJsonStr(const char *groupId, char **returnJsonStr); -int32_t GenerateBindSuccessData(const char *peerAuthId, const char *groupId, char **returnDataStr); +int32_t GenerateBindSuccessData(const char *peerAuthId, const char *peerUdid, const char *groupId, + char **returnDataStr); int32_t GenerateUnbindSuccessData(const char *peerAuthId, const char *groupId, char **returnDataStr); int32_t GetCurDeviceNumByGroupId(int32_t osAccountId, const char *groupId); diff --git a/services/group_manager/src/group_operation/group_operation_common.c b/services/group_manager/src/group_operation/group_operation_common.c index 9a92adb7b6f6b3c3ee3b2e8c2fb2adca0a25ece4..ec976a340807ee9939b42ac32c4f42fc958f83e9 100644 --- a/services/group_manager/src/group_operation/group_operation_common.c +++ b/services/group_manager/src/group_operation/group_operation_common.c @@ -982,20 +982,16 @@ int32_t ConvertGroupIdToJsonStr(const char *groupId, char **returnJsonStr) return HC_SUCCESS; } -int32_t GenerateBindSuccessData(const char *peerAuthId, const char *groupId, char **returnDataStr) +int32_t GenerateBindSuccessData(const char *peerAuthId, const char *peerUdid, + const char *groupId, char **returnDataStr) { - if ((peerAuthId == NULL) || (groupId == NULL) || (returnDataStr == NULL)) { + if ((peerAuthId == NULL) || (peerUdid == NULL) || (groupId == NULL) || (returnDataStr == NULL)) { LOGE("The input params contains NULL value!"); return HC_ERR_NULL_PTR; } - LOGI("Bind successfully! [GroupId]: %c%c%c%c****", groupId[DEV_AUTH_ZERO], groupId[DEV_AUTH_ONE], - groupId[DEV_AUTH_TWO], groupId[DEV_AUTH_THREE]); - if (HcStrlen(peerAuthId) >= DESENSITIZATION_LEN) { - LOGI("Bind successfully! [PeerAuthId]: %c%c%c%c****", peerAuthId[DEV_AUTH_ZERO], peerAuthId[DEV_AUTH_ONE], - peerAuthId[DEV_AUTH_TWO], peerAuthId[DEV_AUTH_THREE]); - } else { - LOGI("Bind successfully! [PeerAuthId]: too short"); - } + PRINT_SENSITIVE_DATA("GroupId", groupId); + PRINT_SENSITIVE_DATA("PeerAuthId", peerAuthId); + PRINT_SENSITIVE_DATA("PeerUdid", peerUdid); CJson *jsonData = CreateJson(); if (jsonData == NULL) { LOGE("Failed to allocate jsonData memory!"); @@ -1027,14 +1023,8 @@ int32_t GenerateUnbindSuccessData(const char *peerAuthId, const char *groupId, c LOGE("The input params contains NULL value!"); return HC_ERR_NULL_PTR; } - LOGI("Unbind successfully! [GroupId]: %c%c%c%c****", groupId[DEV_AUTH_ZERO], groupId[DEV_AUTH_ONE], - groupId[DEV_AUTH_TWO], groupId[DEV_AUTH_THREE]); - if (HcStrlen(peerAuthId) >= DESENSITIZATION_LEN) { - LOGI("Unbind successfully! [PeerAuthId]: %c%c%c%c****", peerAuthId[DEV_AUTH_ZERO], peerAuthId[DEV_AUTH_ONE], - peerAuthId[DEV_AUTH_TWO], peerAuthId[DEV_AUTH_THREE]); - } else { - LOGI("Unbind successfully! [PeerAuthId]: too short"); - } + PRINT_SENSITIVE_DATA("GroupId", groupId); + PRINT_SENSITIVE_DATA("PeerAuthId", peerAuthId); CJson *jsonData = CreateJson(); if (jsonData == NULL) { LOGE("Failed to allocate jsonData memory!"); diff --git a/services/session_manager/src/session/v1/compatible_bind_sub_session/compatible_bind_sub_session.c b/services/session_manager/src/session/v1/compatible_bind_sub_session/compatible_bind_sub_session.c index 9ff79366951e69e057f9412d160698f44c29f2a2..ac72e0038b286def8d18f15e7537d2d5c54134e7 100644 --- a/services/session_manager/src/session/v1/compatible_bind_sub_session/compatible_bind_sub_session.c +++ b/services/session_manager/src/session/v1/compatible_bind_sub_session/compatible_bind_sub_session.c @@ -257,7 +257,7 @@ static int32_t SendBindDataToPeer(CompatibleBindSubSession *session, CJson *out) return result; } -static int32_t InformSelfBindSuccess(const char *peerAuthId, const char *groupId, +static int32_t InformSelfBindSuccess(const char *peerAuthId, const char *peerUdid, const char *groupId, const CompatibleBindSubSession *session, CJson *out) { uint8_t sessionKey[DEFAULT_RETURN_KEY_LENGTH] = { 0 }; @@ -268,7 +268,7 @@ static int32_t InformSelfBindSuccess(const char *peerAuthId, const char *groupId } char *jsonDataStr = NULL; - int32_t result = GenerateBindSuccessData(peerAuthId, groupId, &jsonDataStr); + int32_t result = GenerateBindSuccessData(peerAuthId, peerUdid, groupId, &jsonDataStr); if (result != HC_SUCCESS) { LOGE("Failed to generate the data to be sent to the service!"); return result; @@ -471,7 +471,7 @@ static int32_t HandleBindSuccess(const char *peerAuthId, const char *peerUdid, c if (result != HC_SUCCESS) { return result; } - return InformSelfBindSuccess(peerAuthId, groupId, session, out); + return InformSelfBindSuccess(peerAuthId, peerUdid, groupId, session, out); } static int32_t OnBindFinish(const CompatibleBindSubSession *session, const CJson *jsonParams, CJson *out) diff --git a/test/unittest/deviceauth/source/group_operation_common_test.cpp b/test/unittest/deviceauth/source/group_operation_common_test.cpp index 54357de1e4659db77feaca52aa47487bfe536315..005abe06af762fc0a2b32c1555f53fddef52d62e 100644 --- a/test/unittest/deviceauth/source/group_operation_common_test.cpp +++ b/test/unittest/deviceauth/source/group_operation_common_test.cpp @@ -307,20 +307,20 @@ HWTEST_F(GroupOperationTest, GroupOperationTest036, TestSize.Level0) HWTEST_F(GroupOperationTest, GroupOperationTest037, TestSize.Level0) { char *returnJsonStr = nullptr; - int32_t ret = GenerateBindSuccessData(nullptr, g_normalStr, &returnJsonStr); + int32_t ret = GenerateBindSuccessData(nullptr, g_normalStr, g_normalStr, &returnJsonStr); ASSERT_NE(ret, HC_SUCCESS); } HWTEST_F(GroupOperationTest, GroupOperationTest038, TestSize.Level0) { char *returnJsonStr = nullptr; - int32_t ret = GenerateBindSuccessData(g_normalStr, nullptr, &returnJsonStr); + int32_t ret = GenerateBindSuccessData(g_normalStr, nullptr, g_normalStr, &returnJsonStr); ASSERT_NE(ret, HC_SUCCESS); } HWTEST_F(GroupOperationTest, GroupOperationTest039, TestSize.Level0) { - int32_t ret = GenerateBindSuccessData(g_normalStr, g_normalStr, nullptr); + int32_t ret = GenerateBindSuccessData(g_normalStr, g_normalStr, g_normalStr, nullptr); ASSERT_NE(ret, HC_SUCCESS); }