Skip to content

Commit 766c82e

Browse files
committed
New test function for large ClientHello
Signed-off-by: Manuel Pégourié-Gonnard <[email protected]>
1 parent 140579f commit 766c82e

File tree

2 files changed

+133
-0
lines changed

2 files changed

+133
-0
lines changed

tests/suites/test_suite_ssl.data

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3606,3 +3606,24 @@ inject_client_content_on_the_wire:MBEDTLS_PK_ECDSA:MBEDTLS_SSL_CLIENT_HELLO:"160
36063606
Inject ClientHello - TLS 1.3 fragmented 4 + invalid type + 72 rejected
36073607
depends_on:MBEDTLS_SSL_PROTO_TLS1_3:MBEDTLS_SSL_TLS1_3_KEY_EXCHANGE_MODE_EPHEMERAL_ENABLED:MBEDTLS_SSL_HAVE_AES:MBEDTLS_MD_CAN_SHA256:MBEDTLS_SSL_HAVE_GCM:MBEDTLS_ECP_HAVE_SECP256R1:MBEDTLS_ECP_HAVE_SECP384R1:MBEDTLS_PK_CAN_ECDSA_SIGN:MBEDTLS_PK_CAN_ECDSA_VERIFY
36083608
inject_client_content_on_the_wire:MBEDTLS_PK_ECDSA:MBEDTLS_SSL_CLIENT_HELLO:"1603030004010000481003030002015a160303004803030123456789abcdef0123456789abcdef0123456789abcdef0123456789abcdef00000213010100001d000a000400020017002b0003020304000d000400020403003300020000":"unknown record type":MBEDTLS_ERR_SSL_INVALID_RECORD
3609+
3610+
Send large fragmented ClientHello: 4 bytes too large
3611+
send_large_fragmented_hello:MBEDTLS_SSL_IN_CONTENT_LEN:0:"requesting more data than fits":MBEDTLS_ERR_SSL_BAD_INPUT_DATA
3612+
3613+
Send large fragmented ClientHello: 1 byte too large
3614+
send_large_fragmented_hello:MBEDTLS_SSL_IN_CONTENT_LEN - 3:0:"requesting more data than fits":MBEDTLS_ERR_SSL_BAD_INPUT_DATA
3615+
3616+
Send large fragmented ClientHello: would fit without overhead #1
3617+
send_large_fragmented_hello:MBEDTLS_SSL_IN_CONTENT_LEN - 4:0:"requesting more data than fits":MBEDTLS_ERR_SSL_BAD_INPUT_DATA
3618+
3619+
Send large fragmented ClientHello: would fit without overhead #2
3620+
send_large_fragmented_hello:MBEDTLS_SSL_IN_CONTENT_LEN - 4:1:"requesting more data than fits":MBEDTLS_ERR_SSL_BAD_INPUT_DATA
3621+
3622+
Send large fragmented ClientHello: would fit without overhead #3
3623+
send_large_fragmented_hello:MBEDTLS_SSL_IN_CONTENT_LEN - 4:2:"requesting more data than fits":MBEDTLS_ERR_SSL_BAD_INPUT_DATA
3624+
3625+
Send large fragmented ClientHello: would fit without overhead #4
3626+
send_large_fragmented_hello:MBEDTLS_SSL_IN_CONTENT_LEN - 4:3:"requesting more data than fits":MBEDTLS_ERR_SSL_BAD_INPUT_DATA
3627+
3628+
Send large fragmented ClientHello: would fit without overhead #5
3629+
send_large_fragmented_hello:MBEDTLS_SSL_IN_CONTENT_LEN - 4:4:"requesting more data than fits":MBEDTLS_ERR_SSL_BAD_INPUT_DATA

tests/suites/test_suite_ssl.function

Lines changed: 112 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5112,3 +5112,115 @@ exit:
51125112
PSA_DONE();
51135113
}
51145114
/* END_CASE */
5115+
5116+
/* BEGIN_CASE depends_on:MBEDTLS_SSL_PROTO_TLS1_3:MBEDTLS_DEBUG_C:MBEDTLS_SSL_HANDSHAKE_WITH_CERT_ENABLED */
5117+
void send_large_fragmented_hello(int hs_len_int, int first_frag_content_len_int,
5118+
char *log_pattern, int expected_ret)
5119+
{
5120+
/* This function sends a long message (claiming to be a ClientHello)
5121+
* fragmented in 1-byte fragments (except the initial fragment).
5122+
* The purpose is to test how the stack reacts when receiving:
5123+
* - a message larger than our buffer;
5124+
* - a message smaller than our buffer, but where the intermediate size of
5125+
* holding all the fragments (including overhead) is larger than our
5126+
* buffer.
5127+
*/
5128+
enum { BUFFSIZE = 16384 };
5129+
mbedtls_test_ssl_endpoint server, client;
5130+
mbedtls_platform_zeroize(&server, sizeof(server));
5131+
mbedtls_platform_zeroize(&client, sizeof(client));
5132+
5133+
mbedtls_test_handshake_test_options options;
5134+
mbedtls_test_init_handshake_options(&options);
5135+
5136+
mbedtls_test_ssl_log_pattern srv_pattern;
5137+
memset(&srv_pattern, 0, sizeof(srv_pattern));
5138+
5139+
unsigned char *first_frag = NULL;
5140+
int ret = -1;
5141+
5142+
size_t hs_len = (size_t) hs_len_int;
5143+
size_t first_frag_content_len = (size_t) first_frag_content_len_int;
5144+
5145+
PSA_INIT();
5146+
5147+
srv_pattern.pattern = log_pattern;
5148+
options.srv_log_obj = &srv_pattern;
5149+
options.srv_log_fun = mbedtls_test_ssl_log_analyzer;
5150+
mbedtls_debug_set_threshold(5);
5151+
5152+
ret = mbedtls_test_ssl_endpoint_init(&server, MBEDTLS_SSL_IS_SERVER,
5153+
&options, NULL, NULL, NULL);
5154+
TEST_EQUAL(ret, 0);
5155+
5156+
ret = mbedtls_test_ssl_endpoint_init(&client, MBEDTLS_SSL_IS_CLIENT,
5157+
&options, NULL, NULL, NULL);
5158+
TEST_EQUAL(ret, 0);
5159+
5160+
ret = mbedtls_test_mock_socket_connect(&server.socket, &client.socket,
5161+
BUFFSIZE);
5162+
TEST_EQUAL(ret, 0);
5163+
5164+
/* Make the server move past the initial dummy state */
5165+
ret = mbedtls_test_move_handshake_to_state(&client.ssl, &server.ssl,
5166+
MBEDTLS_SSL_CLIENT_HELLO);
5167+
TEST_EQUAL(ret, 0);
5168+
5169+
/* Prepare initial fragment */
5170+
const size_t first_len = 5 // record header, see below
5171+
+ 4 // handshake header, see balow
5172+
+ first_frag_content_len;
5173+
TEST_CALLOC(first_frag, first_len);
5174+
unsigned char *p = first_frag;
5175+
// record header
5176+
// record type: handshake
5177+
*p++ = 0x16,
5178+
// record version (actually common to TLS 1.2 and TLS 1.3)
5179+
*p++ = 0x03,
5180+
*p++ = 0x03,
5181+
// record length: two bytes
5182+
*p++ = (unsigned char) (((4 + first_frag_content_len) >> 8) & 0xff);
5183+
*p++ = (unsigned char) (((4 + first_frag_content_len) >> 0) & 0xff);
5184+
// handshake header
5185+
// handshake type: ClientHello
5186+
*p++ = 0x01,
5187+
// handshake length: three bytes
5188+
*p++ = (unsigned char) ((hs_len >> 16) & 0xff);
5189+
*p++ = (unsigned char) ((hs_len >> 8) & 0xff);
5190+
*p++ = (unsigned char) ((hs_len >> 0) & 0xff);
5191+
// handshake content: dummy value
5192+
memset(p, 0x2a, first_frag_content_len);
5193+
5194+
/* Send initial fragment and have the server process it. */
5195+
ret = mbedtls_test_mock_tcp_send_b(&client.socket, first_frag, first_len);
5196+
TEST_ASSERT(ret >= 0 && (size_t) ret == first_len);
5197+
5198+
ret = mbedtls_ssl_handshake_step(&server.ssl);
5199+
TEST_EQUAL(ret, MBEDTLS_ERR_SSL_WANT_READ);
5200+
5201+
/* Dummy 1-byte fragment to repeatedly send next */
5202+
const unsigned char next[] = {
5203+
0x16, 0x03, 0x03, 0x00, 0x01, // record header (see above)
5204+
0x2a, // Dummy handshake message content
5205+
};
5206+
for (size_t left = hs_len - first_frag_content_len; left != 0; left--) {
5207+
ret = mbedtls_test_mock_tcp_send_b(&client.socket, next, sizeof(next));
5208+
TEST_ASSERT(ret >= 0 && (size_t) ret == sizeof(next));
5209+
5210+
ret = mbedtls_ssl_handshake_step(&server.ssl);
5211+
if (ret != MBEDTLS_ERR_SSL_WANT_READ) {
5212+
break;
5213+
}
5214+
}
5215+
TEST_EQUAL(ret, expected_ret);
5216+
TEST_EQUAL(srv_pattern.counter, 1);
5217+
5218+
exit:
5219+
mbedtls_test_free_handshake_options(&options);
5220+
mbedtls_test_ssl_endpoint_free(&server, NULL);
5221+
mbedtls_test_ssl_endpoint_free(&client, NULL);
5222+
mbedtls_debug_set_threshold(0);
5223+
mbedtls_free(first_frag);
5224+
PSA_DONE();
5225+
}
5226+
/* END_CASE */

0 commit comments

Comments
 (0)