Skip to content

Commit f42fb58

Browse files
committed
Address coverity issue 394706
1 parent ac52660 commit f42fb58

File tree

1 file changed

+31
-31
lines changed

1 file changed

+31
-31
lines changed

wolfcrypt/test/test.c

Lines changed: 31 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -28587,39 +28587,39 @@ WOLFSSL_TEST_SUBROUTINE wc_test_ret_t srtpkdf_test(void)
2858728587
return WC_TEST_RET_ENC_NC;
2858828588
for (i = 0; i < 32; i++) {
2858928589
word32 kdr = 1U << i;
28590-
idx = wc_SRTP_KDF_kdr_to_idx(kdr);
28591-
if (idx != i)
28592-
return WC_TEST_RET_ENC_NC;
28593-
}
2859428590

28595-
/* SRTCP w/ 48-bit IDX, 128-bit key test */
28596-
if (i == 0) {
28597-
ret = wc_SRTCP_KDF_ex(mk48_1, (word32)sizeof(mk48_1),
28598-
ms48_1, (word32)sizeof(ms48_1),
28599-
kdr_48_1, srtcp48idx_1, keyE, tv[i].keSz,
28600-
keyA, tv[i].kaSz, keyS, tv[i].ksSz,
28601-
WC_SRTCP_48BIT_IDX);
28602-
if (ret != 0)
28603-
return WC_TEST_RET_ENC_EC(ret);
28604-
if (XMEMCMP(keyE, srtcpKe_48_1, tv[i].keSz) != 0)
28605-
return WC_TEST_RET_ENC_NC;
28606-
if (XMEMCMP(keyA, srtcpKa_48_1, tv[i].kaSz) != 0)
28607-
return WC_TEST_RET_ENC_NC;
28608-
if (XMEMCMP(keyS, srtcpKs_48_1, tv[i].ksSz) != 0)
28609-
return WC_TEST_RET_ENC_NC;
28591+
/* SRTCP w/ 48-bit IDX, 128-bit key test */
28592+
if (i == 0) {
28593+
ret = wc_SRTCP_KDF_ex(mk48_1, (word32)sizeof(mk48_1),
28594+
ms48_1, (word32)sizeof(ms48_1),
28595+
kdr_48_1, srtcp48idx_1, keyE, tv[i].keSz,
28596+
keyA, tv[i].kaSz, keyS, tv[i].ksSz,
28597+
WC_SRTCP_48BIT_IDX);
28598+
if (ret != 0)
28599+
return WC_TEST_RET_ENC_EC(ret);
28600+
if (XMEMCMP(keyE, srtcpKe_48_1, tv[i].keSz) != 0)
28601+
return WC_TEST_RET_ENC_NC;
28602+
if (XMEMCMP(keyA, srtcpKa_48_1, tv[i].kaSz) != 0)
28603+
return WC_TEST_RET_ENC_NC;
28604+
if (XMEMCMP(keyS, srtcpKs_48_1, tv[i].ksSz) != 0)
28605+
return WC_TEST_RET_ENC_NC;
2861028606

28611-
ret = wc_SRTCP_KDF_ex(mk48_2, (word32)sizeof(mk48_2),
28612-
ms48_2, (word32)sizeof(ms48_2),
28613-
kdr_48_2, srtcp48idx_2, keyE, tv[i].keSz,
28614-
keyA, tv[i].kaSz, keyS, tv[i].ksSz,
28615-
WC_SRTCP_48BIT_IDX);
28616-
if (ret != 0)
28617-
return WC_TEST_RET_ENC_EC(ret);
28618-
if (XMEMCMP(keyE, srtcpKe_48_2, tv[i].keSz) != 0)
28619-
return WC_TEST_RET_ENC_NC;
28620-
if (XMEMCMP(keyA, srtcpKa_48_2, tv[i].kaSz) != 0)
28621-
return WC_TEST_RET_ENC_NC;
28622-
if (XMEMCMP(keyS, srtcpKs_48_2, tv[i].ksSz) != 0)
28607+
ret = wc_SRTCP_KDF_ex(mk48_2, (word32)sizeof(mk48_2),
28608+
ms48_2, (word32)sizeof(ms48_2),
28609+
kdr_48_2, srtcp48idx_2, keyE, tv[i].keSz,
28610+
keyA, tv[i].kaSz, keyS, tv[i].ksSz,
28611+
WC_SRTCP_48BIT_IDX);
28612+
if (ret != 0)
28613+
return WC_TEST_RET_ENC_EC(ret);
28614+
if (XMEMCMP(keyE, srtcpKe_48_2, tv[i].keSz) != 0)
28615+
return WC_TEST_RET_ENC_NC;
28616+
if (XMEMCMP(keyA, srtcpKa_48_2, tv[i].kaSz) != 0)
28617+
return WC_TEST_RET_ENC_NC;
28618+
if (XMEMCMP(keyS, srtcpKs_48_2, tv[i].ksSz) != 0)
28619+
return WC_TEST_RET_ENC_NC;
28620+
}
28621+
idx = wc_SRTP_KDF_kdr_to_idx(kdr);
28622+
if (idx != i)
2862328623
return WC_TEST_RET_ENC_NC;
2862428624
}
2862528625

0 commit comments

Comments
 (0)