Skip to content

Commit 5d1cb85

Browse files
Add missing dependency on "same seed" test case
The test case passes even if the two clients have different RNG seeds, but in that case, it doesn't do what it claims, but instead it duplicates the "2 successive clients, TLS 1.3" test case above. So run "2 successive clients with same seed, TLS 1.3" only when there is no entropy, like "2 successive clients with same seed, TLS 1.3". Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
1 parent 5d91ec5 commit 5d1cb85

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

tests/opt-testcases/sample.sh

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -248,6 +248,9 @@ run_test "Sample: ssl_fork_server, 2 successive clients with same seed, TLS 1
248248
-C "error" \
249249
-v 'distinct_server_random'
250250

251+
requires_config_enabled MBEDTLS_ENTROPY_NO_SOURCES_OK
252+
requires_config_disabled MBEDTLS_PSA_BUILTIN_GET_ENTROPY
253+
requires_config_disabled MBEDTLS_PSA_DRIVER_GET_ENTROPY
251254
run_test "Sample: ssl_fork_server, 2 successive clients with same seed, TLS 1.3: unique random" \
252255
-P 4433 \
253256
"server_with_own_seedfile $PROGRAMS_DIR/ssl_fork_server" \

0 commit comments

Comments
 (0)