Skip to content

Commit 902164c

Browse files
committed
wolfcrypt/src/rng_bank.c: fixes for typography and s/wc_FreeRng(rng2)/wc_rng_free(rng2)/ in random_bank_test();
wolfcrypt/src/rng_bank.c: tweaks to silence benign Coverity CHECKED_RETURN.
1 parent d0e32f8 commit 902164c

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

wolfcrypt/src/rng_bank.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -645,7 +645,7 @@ WOLFSSL_API int wc_rng_bank_reseed(struct wc_rng_bank *bank,
645645
"ERROR: wc_crng_reseed() wc_RNG_GenerateBlock() "
646646
"for DRBG #%d returned %d.", n, ret);
647647
#endif
648-
wc_rng_bank_checkin(bank, &drbg);
648+
(void)wc_rng_bank_checkin(bank, &drbg);
649649
if (ret == WC_NO_ERR_TRACE(WC_TIMEOUT_E))
650650
return ret;
651651
ret = WC_CHECK_FOR_INTR_SIGNALS();
@@ -654,7 +654,7 @@ WOLFSSL_API int wc_rng_bank_reseed(struct wc_rng_bank *bank,
654654
WC_RELAX_LONG_LOOP();
655655
}
656656
else {
657-
wc_rng_bank_checkin(bank, &drbg);
657+
(void)wc_rng_bank_checkin(bank, &drbg);
658658
}
659659
}
660660

wolfcrypt/test/test.c

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20523,7 +20523,8 @@ WOLFSSL_TEST_SUBROUTINE wc_test_ret_t random_bank_test(void)
2052320523
if (ret != WC_NO_ERR_TRACE(BUSY_E))
2052420524
ERROR_OUT(WC_TEST_RET_ENC_NC, out);
2052520525

20526-
wc_FreeRng(rng2);
20526+
wc_rng_free(rng2);
20527+
rng2 = NULL;
2052720528

2052820529
if (wolfSSL_RefCur(bank2->refcount) != 1)
2052920530
ERROR_OUT(WC_TEST_RET_ENC_NC, out);
@@ -20536,7 +20537,7 @@ WOLFSSL_TEST_SUBROUTINE wc_test_ret_t random_bank_test(void)
2053620537
if (bank2 != NULL)
2053720538
ERROR_OUT(WC_TEST_RET_ENC_NC, out);
2053820539

20539-
#endif /* WC_RNG_BANK_STATIC */
20540+
#endif /* !WC_RNG_BANK_STATIC */
2054020541

2054120542
out:
2054220543

@@ -20546,9 +20547,7 @@ WOLFSSL_TEST_SUBROUTINE wc_test_ret_t random_bank_test(void)
2054620547
#ifdef WC_DRBG_BANKREF
2054720548
cleanup_ret = wc_FreeRng(rng);
2054820549
if ((cleanup_ret != 0) && (ret == 0))
20549-
{
2055020550
ret = WC_TEST_RET_ENC_EC(cleanup_ret);
20551-
}
2055220551
WC_FREE_VAR_EX(rng, HEAP_HINT, DYNAMIC_TYPE_TMP_BUFFER);
2055320552
#endif /* WC_DRBG_BANKREF */
2055420553
if (rng_inst) {

0 commit comments

Comments
 (0)