Skip to content

Commit 237f4d9

Browse files
TofMassilia13320Grom-
authored andcommitted
[fix][services] use STSAFE family index
1 parent ab3d296 commit 237f4d9

5 files changed

Lines changed: 15 additions & 11 deletions

File tree

api/stse_random.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,7 @@ stse_ReturnCode_t stse_generate_random(
4343
if (pSTSE->device_type != STSAFE_L010) {
4444
#endif /* STSE_CONF_STSAFE_L_SUPPORT */
4545
while (0 < random_size) {
46-
PLAT_UI16 chunk = (random_size < STSAFEA_MAXIMUM_RNG_SIZE) ?
47-
random_size : STSAFEA_MAXIMUM_RNG_SIZE;
46+
PLAT_UI16 chunk = (random_size < STSAFEA_MAXIMUM_RNG_SIZE) ? random_size : STSAFEA_MAXIMUM_RNG_SIZE;
4847

4948
ret = stsafea_generate_random(pSTSE, pRandom, chunk);
5049

core/stse_device.h

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,13 @@ typedef enum stse_device_t {
5858
#endif /* STSE_CONF_STSAFE_L_SUPPORT */
5959
} stse_device_t;
6060

61+
#ifdef STSE_CONF_STSAFE_A_SUPPORT
6162
#define STSE_DEVICE_STSAFEA_FAMILY_INDEX STSAFE_A100
63+
#endif /* STSE_CONF_STSAFE_A_SUPPORT */
64+
65+
#ifdef STSE_CONF_STSAFE_L_SUPPORT
6266
#define STSE_DEVICE_STSAFEL_FAMILY_INDEX STSAFE_L010
67+
#endif /* STSE_CONF_STSAFE_L_SUPPORT */
6368

6469
typedef struct stse_perso_info_t {
6570
PLAT_UI32 cmd_encryption_status;

services/stsafea/stsafea_frame_transfer.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ stse_ReturnCode_t stsafea_frame_transmit(stse_Handler_t *pSTSE, stse_frame_t *pF
4444
return STSE_SERVICE_INVALID_PARAMETER;
4545
}
4646
/*- Verify Device type */
47-
if (pSTSE->device_type >= (STSAFE_A100 + STSAFEA_PRODUCT_COUNT)) {
47+
if (pSTSE->device_type >= (STSE_DEVICE_STSAFEA_FAMILY_INDEX + STSAFEA_PRODUCT_COUNT)) {
4848
return STSE_SERVICE_INVALID_PARAMETER;
4949
}
5050
/*- Verify Frame length */
@@ -56,7 +56,7 @@ stse_ReturnCode_t stsafea_frame_transmit(stse_Handler_t *pSTSE, stse_frame_t *pF
5656
return STSE_SERVICE_INVALID_FRAME;
5757
}
5858
/*- Verify Frame overflow */
59-
if (pFrame->length > stsafea_maximum_frame_length[pSTSE->device_type - STSAFE_A100]) {
59+
if (pFrame->length > stsafea_maximum_frame_length[pSTSE->device_type - STSE_DEVICE_STSAFEA_FAMILY_INDEX]) {
6060
return STSE_SERVICE_FRAME_SIZE_ERROR;
6161
}
6262
/*- Compute frame crc */
@@ -144,7 +144,7 @@ stse_ReturnCode_t stsafea_frame_receive(stse_Handler_t *pSTSE, stse_frame_t *pFr
144144
return (STSE_SERVICE_INVALID_PARAMETER);
145145
}
146146
/* - Verify Device type */
147-
if (pSTSE->device_type >= (STSAFE_A100 + STSAFEA_PRODUCT_COUNT)) {
147+
if (pSTSE->device_type >= (STSE_DEVICE_STSAFEA_FAMILY_INDEX + STSAFEA_PRODUCT_COUNT)) {
148148
return STSE_SERVICE_INVALID_PARAMETER;
149149
}
150150

@@ -206,7 +206,7 @@ stse_ReturnCode_t stsafea_frame_receive(stse_Handler_t *pSTSE, stse_frame_t *pFr
206206
received_length = ((length_value[0] << 8) + length_value[1]) - STSE_FRAME_CRC_SIZE + STSE_RSP_FRAME_HEADER_SIZE;
207207

208208
/*- Verify Frame overflow */
209-
if (received_length > stsafea_maximum_frame_length[pSTSE->device_type - STSAFE_A100]) {
209+
if (received_length > stsafea_maximum_frame_length[pSTSE->device_type - STSE_DEVICE_STSAFEA_FAMILY_INDEX]) {
210210
return STSE_SERVICE_FRAME_SIZE_ERROR;
211211
}
212212

services/stsafea/stsafea_put_query.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ stse_ReturnCode_t stsafea_put_i2c_parameters(
9292
}
9393

9494
#ifdef STSE_CONF_STSAFE_L_SUPPORT
95-
if (pSTSE->device_type == STSAFE_L010) {
95+
if (pSTSE->device_type >= STSE_DEVICE_STSAFEL_FAMILY_INDEX) {
9696
return STSE_SERVICE_INVALID_PARAMETER;
9797
}
9898
#endif

services/stsafel/stsafel_frame_transfer.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ stse_ReturnCode_t stsafel_frame_transmit(stse_Handler_t *pSTSE, stse_frame_t *pF
4141
return STSE_SERVICE_INVALID_PARAMETER;
4242
}
4343
/*- Verify Device type */
44-
if (pSTSE->device_type >= (STSAFE_L010 + STSAFEL_PRODUCT_COUNT)) {
44+
if (pSTSE->device_type >= (STSE_DEVICE_STSAFEL_FAMILY_INDEX + STSAFEL_PRODUCT_COUNT)) {
4545
return STSE_SERVICE_INVALID_PARAMETER;
4646
}
4747
/*- Verify Frame length */
@@ -53,7 +53,7 @@ stse_ReturnCode_t stsafel_frame_transmit(stse_Handler_t *pSTSE, stse_frame_t *pF
5353
return STSE_SERVICE_INVALID_FRAME;
5454
}
5555
/*- Verify Frame overflow */
56-
if (pFrame->length > stsafel_maximum_frame_length[pSTSE->device_type - STSAFE_L010]) {
56+
if (pFrame->length > stsafel_maximum_frame_length[pSTSE->device_type - STSE_DEVICE_STSAFEL_FAMILY_INDEX]) {
5757
return STSE_SERVICE_FRAME_SIZE_ERROR;
5858
}
5959
/*- Compute frame crc */
@@ -142,7 +142,7 @@ stse_ReturnCode_t stsafel_i2c_frame_receive(stse_Handler_t *pSTSE, stse_frame_t
142142
return (STSE_SERVICE_INVALID_PARAMETER);
143143
}
144144
/* - Verify Device type */
145-
if (pSTSE->device_type >= (STSAFE_L010 + STSAFEL_PRODUCT_COUNT)) {
145+
if (pSTSE->device_type >= (STSE_DEVICE_STSAFEL_FAMILY_INDEX + STSAFEL_PRODUCT_COUNT)) {
146146
return STSE_SERVICE_INVALID_PARAMETER;
147147
}
148148

@@ -182,7 +182,7 @@ stse_ReturnCode_t stsafel_i2c_frame_receive(stse_Handler_t *pSTSE, stse_frame_t
182182
received_length = ((length_value[0] << 8) + length_value[1]) - STSE_FRAME_CRC_SIZE;
183183

184184
/*- Verify Frame overflow */
185-
if (pFrame->length > stsafel_maximum_frame_length[pSTSE->device_type - STSAFE_L010]) {
185+
if (pFrame->length > stsafel_maximum_frame_length[pSTSE->device_type - STSE_DEVICE_STSAFEL_FAMILY_INDEX]) {
186186
return STSE_SERVICE_FRAME_SIZE_ERROR;
187187
}
188188

0 commit comments

Comments
 (0)