X-Git-Url: https://git.openssl.org/?p=openssl.git;a=blobdiff_plain;f=test%2Fsslapitest.c;h=ae5c4c0bd183684c04e6b79da70b3c365fbf96af;hp=0e2bdbefaff2a2c2b22f07bed2456b9ab6c64cad;hb=a599574be17579496877bcdcc5e1eeaf5b014cf3;hpb=a37008d90fc4d2bd31a747aebdaf59eaa4e6efaf diff --git a/test/sslapitest.c b/test/sslapitest.c index 0e2bdbefaf..ae5c4c0bd1 100644 --- a/test/sslapitest.c +++ b/test/sslapitest.c @@ -17,17 +17,17 @@ #include "ssltestlib.h" #include "testutil.h" -#include "test_main_custom.h" #include "e_os.h" +#include "../ssl/ssl_locl.h" static char *cert = NULL; static char *privkey = NULL; #define LOG_BUFFER_SIZE 1024 static char server_log_buffer[LOG_BUFFER_SIZE + 1] = {0}; -static int server_log_buffer_index = 0; +static size_t server_log_buffer_index = 0; static char client_log_buffer[LOG_BUFFER_SIZE + 1] = {0}; -static int client_log_buffer_index = 0; +static size_t client_log_buffer_index = 0; static int error_writing_log = 0; #ifndef OPENSSL_NO_OCSP @@ -54,64 +54,69 @@ struct sslapitest_log_counts { unsigned int server_application_secret_count; }; -static void client_keylog_callback(const SSL *ssl, const char *line) { + +static unsigned char serverinfov1[] = { + 0xff, 0xff, /* Dummy extension type */ + 0x00, 0x01, /* Extension length is 1 byte */ + 0xff /* Dummy extension data */ +}; + +static unsigned char serverinfov2[] = { + 0x00, 0x00, 0x00, + (unsigned char)(SSL_EXT_CLIENT_HELLO & 0xff), /* Dummy context - 4 bytes */ + 0xff, 0xff, /* Dummy extension type */ + 0x00, 0x01, /* Extension length is 1 byte */ + 0xff /* Dummy extension data */ +}; + +static void client_keylog_callback(const SSL *ssl, const char *line) +{ int line_length = strlen(line); /* If the log doesn't fit, error out. */ - if ((client_log_buffer_index + line_length) > LOG_BUFFER_SIZE) { - printf("No room in client log\n"); + if (client_log_buffer_index + line_length > sizeof(client_log_buffer) - 1) { + TEST_info("Client log too full"); error_writing_log = 1; return; } strcat(client_log_buffer, line); client_log_buffer_index += line_length; - client_log_buffer[client_log_buffer_index] = '\n'; - client_log_buffer_index += 1; - - return; + client_log_buffer[client_log_buffer_index++] = '\n'; } -static void server_keylog_callback(const SSL *ssl, const char *line) { +static void server_keylog_callback(const SSL *ssl, const char *line) +{ int line_length = strlen(line); /* If the log doesn't fit, error out. */ - if ((server_log_buffer_index + line_length) > LOG_BUFFER_SIZE) { - printf("No room in server log\n"); + if (server_log_buffer_index + line_length > sizeof(server_log_buffer) - 1) { + TEST_info("Server og too full"); error_writing_log = 1; return; } strcat(server_log_buffer, line); server_log_buffer_index += line_length; - server_log_buffer[server_log_buffer_index] = '\n'; - server_log_buffer_index += 1; - - return; + server_log_buffer[server_log_buffer_index++] = '\n'; } static int compare_hex_encoded_buffer(const char *hex_encoded, size_t hex_length, const uint8_t *raw, - size_t raw_length) { - size_t i; - size_t j; - - /* One byte too big, just to be safe. */ - char hexed[3] = {0}; + size_t raw_length) +{ + size_t i, j; + char hexed[3]; - if ((raw_length * 2) != hex_length) { - printf("Inconsistent hex encoded lengths.\n"); + if (!TEST_size_t_eq(raw_length * 2, hex_length)) return 1; - } - for (i = j = 0; (i < raw_length) && ((j + 1) < hex_length); i++) { + for (i = j = 0; i < raw_length && j + 1 < hex_length; i++, j += 2) { sprintf(hexed, "%02x", raw[i]); - if ((hexed[0] != hex_encoded[j]) || (hexed[1] != hex_encoded[j + 1])) { - printf("Hex output does not match.\n"); + if (!TEST_int_eq(hexed[0], hex_encoded[j]) + || !TEST_int_eq(hexed[1], hex_encoded[j + 1])) return 1; - } - j += 2; } return 0; @@ -119,7 +124,8 @@ static int compare_hex_encoded_buffer(const char *hex_encoded, static int test_keylog_output(char *buffer, const SSL *ssl, const SSL_SESSION *session, - struct sslapitest_log_counts *expected) { + struct sslapitest_log_counts *expected) +{ char *token = NULL; unsigned char actual_client_random[SSL3_RANDOM_SIZE] = {0}; size_t client_random_size = SSL3_RANDOM_SIZE; @@ -132,28 +138,20 @@ static int test_keylog_output(char *buffer, const SSL *ssl, unsigned int client_application_secret_count = 0; unsigned int server_application_secret_count = 0; - token = strtok(buffer, " \n"); - while (token) { + for (token = strtok(buffer, " \n"); token != NULL; + token = strtok(NULL, " \n")) { if (strcmp(token, "RSA") == 0) { /* * Premaster secret. Tokens should be: 16 ASCII bytes of * hex-encoded encrypted secret, then the hex-encoded pre-master * secret. */ - token = strtok(NULL, " \n"); - if (!token) { - printf("Unexpectedly short premaster secret log.\n"); + if (!TEST_ptr(token = strtok(NULL, " \n"))) return 0; - } - if (strlen(token) != 16) { - printf("Bad value for encrypted secret: %s\n", token); + if (!TEST_size_t_eq(strlen(token), 16)) return 0; - } - token = strtok(NULL, " \n"); - if (!token) { - printf("Unexpectedly short premaster secret log.\n"); + if (!TEST_ptr(token = strtok(NULL, " \n"))) return 0; - } /* * We can't sensibly check the log because the premaster secret is * transient, and OpenSSL doesn't keep hold of it once the master @@ -168,51 +166,34 @@ static int test_keylog_output(char *buffer, const SSL *ssl, client_random_size = SSL_get_client_random(ssl, actual_client_random, SSL3_RANDOM_SIZE); - if (client_random_size != SSL3_RANDOM_SIZE) { - printf("Unexpected short client random.\n"); + if (!TEST_size_t_eq(client_random_size, SSL3_RANDOM_SIZE)) return 0; - } - token = strtok(NULL, " \n"); - if (!token) { - printf("Unexpected short master secret log.\n"); + if (!TEST_ptr(token = strtok(NULL, " \n"))) return 0; - } - if (strlen(token) != 64) { - printf("Bad value for client random: %s\n", token); + if (!TEST_size_t_eq(strlen(token), 64)) return 0; - } - if (compare_hex_encoded_buffer(token, 64, actual_client_random, - client_random_size)) { - printf("Bad value for client random: %s\n", token); + if (!TEST_false(compare_hex_encoded_buffer(token, 64, + actual_client_random, + client_random_size))) return 0; - } - token = strtok(NULL, " \n"); - if (!token) { - printf("Unexpectedly short master secret log.\n"); + if (!TEST_ptr(token = strtok(NULL, " \n"))) return 0; - } - master_key_size = SSL_SESSION_get_master_key(session, actual_master_key, master_key_size); - if (!master_key_size) { - printf("Error getting master key to compare.\n"); + if (!TEST_size_t_ne(master_key_size, 0)) return 0; - } - if (compare_hex_encoded_buffer(token, strlen(token), - actual_master_key, - master_key_size)) { - printf("Bad value for master key: %s\n", token); + if (!TEST_false(compare_hex_encoded_buffer(token, strlen(token), + actual_master_key, + master_key_size))) return 0; - } - master_secret_count++; - } else if ((strcmp(token, "CLIENT_HANDSHAKE_TRAFFIC_SECRET") == 0) || - (strcmp(token, "SERVER_HANDSHAKE_TRAFFIC_SECRET") == 0) || - (strcmp(token, "CLIENT_TRAFFIC_SECRET_0") == 0) || - (strcmp(token, "SERVER_TRAFFIC_SECRET_0") == 0)) { + } else if (strcmp(token, "CLIENT_HANDSHAKE_TRAFFIC_SECRET") == 0 + || strcmp(token, "SERVER_HANDSHAKE_TRAFFIC_SECRET") == 0 + || strcmp(token, "CLIENT_TRAFFIC_SECRET_0") == 0 + || strcmp(token, "SERVER_TRAFFIC_SECRET_0") == 0) { /* * TLSv1.3 secret. Tokens should be: 64 ASCII bytes of hex-encoded * client random, and then the hex-encoded secret. In this case, @@ -231,127 +212,95 @@ static int test_keylog_output(char *buffer, const SSL *ssl, client_random_size = SSL_get_client_random(ssl, actual_client_random, SSL3_RANDOM_SIZE); - if (client_random_size != SSL3_RANDOM_SIZE) { - printf("Unexpected short client random.\n"); + if (!TEST_size_t_eq(client_random_size, SSL3_RANDOM_SIZE)) return 0; - } - token = strtok(NULL, " \n"); - if (!token) { - printf("Unexpected short client handshake secret log.\n"); + if (!TEST_ptr(token = strtok(NULL, " \n"))) return 0; - } - if (strlen(token) != 64) { - printf("Bad value for client random: %s\n", token); + if (!TEST_size_t_eq(strlen(token), 64)) return 0; - } - if (compare_hex_encoded_buffer(token, 64, actual_client_random, - client_random_size)) { - printf("Bad value for client random: %s\n", token); + if (!TEST_false(compare_hex_encoded_buffer(token, 64, + actual_client_random, + client_random_size))) return 0; - } - token = strtok(NULL, " \n"); - if (!token) { - printf("Unexpectedly short master secret log.\n"); + if (!TEST_ptr(token = strtok(NULL, " \n"))) return 0; - } /* * TODO(TLS1.3): test that application traffic secrets are what * we expect */ } else { - printf("Unexpected token in buffer: %s\n", token); + TEST_info("Unexpected token %s\n", token); return 0; } - - token = strtok(NULL, " \n"); } - /* Return whether we got what we expected. */ - return ((rsa_key_exchange_count == expected->rsa_key_exchange_count) && - (master_secret_count == expected->master_secret_count) && - (client_handshake_secret_count == expected->client_handshake_secret_count) && - (server_handshake_secret_count == expected->server_handshake_secret_count) && - (client_application_secret_count == expected->client_application_secret_count) && - (server_application_secret_count == expected->server_application_secret_count)); + /* Got what we expected? */ + if (!TEST_size_t_eq(rsa_key_exchange_count, + expected->rsa_key_exchange_count) + || !TEST_size_t_eq(master_secret_count, + expected->master_secret_count) + || !TEST_size_t_eq(client_handshake_secret_count, + expected->client_handshake_secret_count) + || !TEST_size_t_eq(server_handshake_secret_count, + expected->server_handshake_secret_count) + || !TEST_size_t_eq(client_application_secret_count, + expected->client_application_secret_count) + || !TEST_size_t_eq(server_application_secret_count, + expected->server_application_secret_count)) + return 0; + return 1; } -static int test_keylog(void) { +static int test_keylog(void) +{ SSL_CTX *cctx = NULL, *sctx = NULL; SSL *clientssl = NULL, *serverssl = NULL; int testresult = 0; - int rc; struct sslapitest_log_counts expected = {0}; /* Clean up logging space */ - memset(client_log_buffer, 0, LOG_BUFFER_SIZE + 1); - memset(server_log_buffer, 0, LOG_BUFFER_SIZE + 1); + memset(client_log_buffer, 0, sizeof(client_log_buffer)); + memset(server_log_buffer, 0, sizeof(server_log_buffer)); client_log_buffer_index = 0; server_log_buffer_index = 0; error_writing_log = 0; - if (!create_ssl_ctx_pair(TLS_server_method(), TLS_client_method(), &sctx, - &cctx, cert, privkey)) { - printf("Unable to create SSL_CTX pair\n"); + if (!TEST_true(create_ssl_ctx_pair(TLS_server_method(), + TLS_client_method(), + &sctx, &cctx, cert, privkey))) return 0; - } /* We cannot log the master secret for TLSv1.3, so we should forbid it. */ SSL_CTX_set_options(cctx, SSL_OP_NO_TLSv1_3); SSL_CTX_set_options(sctx, SSL_OP_NO_TLSv1_3); /* We also want to ensure that we use RSA-based key exchange. */ - rc = SSL_CTX_set_cipher_list(cctx, "RSA"); - if (rc == 0) { - printf("Unable to restrict to RSA key exchange.\n"); + if (!TEST_true(SSL_CTX_set_cipher_list(cctx, "RSA"))) goto end; - } - if (SSL_CTX_get_keylog_callback(cctx)) { - printf("Unexpected initial value for client " - "SSL_CTX_get_keylog_callback()\n"); - goto end; - } - if (SSL_CTX_get_keylog_callback(sctx)) { - printf("Unexpected initial value for server " - "SSL_CTX_get_keylog_callback()\n"); + if (!TEST_true(SSL_CTX_get_keylog_callback(cctx) == NULL) + || !TEST_true(SSL_CTX_get_keylog_callback(sctx) == NULL)) goto end; - } - SSL_CTX_set_keylog_callback(cctx, client_keylog_callback); - SSL_CTX_set_keylog_callback(sctx, server_keylog_callback); - - if (SSL_CTX_get_keylog_callback(cctx) != client_keylog_callback) { - printf("Unexpected set value for client " - "SSL_CTX_get_keylog_callback()\n"); - } - - if (SSL_CTX_get_keylog_callback(sctx) != server_keylog_callback) { - printf("Unexpected set value for server " - "SSL_CTX_get_keylog_callback()\n"); - } - - /* Now do a handshake and check that the logs have been written to. */ - if (!create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, NULL)) { - printf("Unable to create SSL objects\n"); + if (!TEST_true(SSL_CTX_get_keylog_callback(cctx) + == client_keylog_callback)) goto end; - } - - if (!create_ssl_connection(serverssl, clientssl, SSL_ERROR_NONE)) { - printf("Unable to create SSL connection\n"); - goto end; - } - - if (error_writing_log) { - printf("Error encountered while logging\n"); + SSL_CTX_set_keylog_callback(sctx, server_keylog_callback); + if (!TEST_true(SSL_CTX_get_keylog_callback(sctx) + == server_keylog_callback)) goto end; - } - if ((client_log_buffer_index == 0) || (server_log_buffer_index == 0)) { - printf("No logs written\n"); + /* Now do a handshake and check that the logs have been written to. */ + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, + &clientssl, NULL, NULL)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE)) + || !TEST_false(error_writing_log) + || !TEST_int_gt(client_log_buffer_index, 0) + || !TEST_int_gt(server_log_buffer_index, 0)) goto end; - } /* * Now we want to test that our output data was vaguely sensible. We @@ -361,18 +310,14 @@ static int test_keylog(void) { */ expected.rsa_key_exchange_count = 1; expected.master_secret_count = 1; - if (!test_keylog_output(client_log_buffer, clientssl, - SSL_get_session(clientssl), &expected)) { - printf("Error encountered in client log buffer\n"); + if (!TEST_true(test_keylog_output(client_log_buffer, clientssl, + SSL_get_session(clientssl), &expected))) goto end; - } expected.rsa_key_exchange_count = 0; - if (!test_keylog_output(server_log_buffer, serverssl, - SSL_get_session(serverssl), &expected)) { - printf("Error encountered in server log buffer\n"); + if (!TEST_true(test_keylog_output(server_log_buffer, serverssl, + SSL_get_session(serverssl), &expected))) goto end; - } testresult = 1; @@ -386,64 +331,46 @@ end: } #ifndef OPENSSL_NO_TLS1_3 -static int test_keylog_no_master_key(void) { +static int test_keylog_no_master_key(void) +{ SSL_CTX *cctx = NULL, *sctx = NULL; SSL *clientssl = NULL, *serverssl = NULL; int testresult = 0; struct sslapitest_log_counts expected = {0}; /* Clean up logging space */ - memset(client_log_buffer, 0, LOG_BUFFER_SIZE + 1); - memset(server_log_buffer, 0, LOG_BUFFER_SIZE + 1); + memset(client_log_buffer, 0, sizeof(client_log_buffer)); + memset(server_log_buffer, 0, sizeof(server_log_buffer)); client_log_buffer_index = 0; server_log_buffer_index = 0; error_writing_log = 0; - if (!create_ssl_ctx_pair(TLS_server_method(), TLS_client_method(), &sctx, - &cctx, cert, privkey)) { - printf("Unable to create SSL_CTX pair\n"); + if (!TEST_true(create_ssl_ctx_pair(TLS_server_method(), + TLS_client_method(), &sctx, + &cctx, cert, privkey))) return 0; - } - if (SSL_CTX_get_keylog_callback(cctx)) { - printf("Unexpected initial value for client " - "SSL_CTX_get_keylog_callback()\n"); - goto end; - } - if (SSL_CTX_get_keylog_callback(sctx)) { - printf("Unexpected initial value for server " - "SSL_CTX_get_keylog_callback()\n"); + if (!TEST_true(SSL_CTX_get_keylog_callback(cctx) == NULL) + || !TEST_true(SSL_CTX_get_keylog_callback(sctx) == NULL)) goto end; - } SSL_CTX_set_keylog_callback(cctx, client_keylog_callback); - SSL_CTX_set_keylog_callback(sctx, server_keylog_callback); - - if (SSL_CTX_get_keylog_callback(cctx) != client_keylog_callback) { - printf("Unexpected set value for client " - "SSL_CTX_get_keylog_callback()\n"); - } - - if (SSL_CTX_get_keylog_callback(sctx) != server_keylog_callback) { - printf("Unexpected set value for server " - "SSL_CTX_get_keylog_callback()\n"); - } - - /* Now do a handshake and check that the logs have been written to. */ - if (!create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, NULL)) { - printf("Unable to create SSL objects\n"); + if (!TEST_true(SSL_CTX_get_keylog_callback(cctx) + == client_keylog_callback)) goto end; - } - if (!create_ssl_connection(serverssl, clientssl, SSL_ERROR_NONE)) { - printf("Unable to create SSL connection\n"); + SSL_CTX_set_keylog_callback(sctx, server_keylog_callback); + if (!TEST_true(SSL_CTX_get_keylog_callback(sctx) + == server_keylog_callback)) goto end; - } - if (error_writing_log) { - printf("Error encountered while logging\n"); + /* Now do a handshake and check that the logs have been written to. */ + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, + &clientssl, NULL, NULL)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE)) + || !TEST_false(error_writing_log)) goto end; - } /* * Now we want to test that our output data was vaguely sensible. For this @@ -454,16 +381,12 @@ static int test_keylog_no_master_key(void) { expected.server_handshake_secret_count = 1; expected.client_application_secret_count = 1; expected.server_application_secret_count = 1; - if (!test_keylog_output(client_log_buffer, clientssl, - SSL_get_session(clientssl), &expected)) { - printf("Error encountered in client log buffer\n"); - goto end; - } - if (!test_keylog_output(server_log_buffer, serverssl, - SSL_get_session(serverssl), &expected)) { - printf("Error encountered in server log buffer\n"); + if (!TEST_true(test_keylog_output(client_log_buffer, clientssl, + SSL_get_session(clientssl), &expected)) + || !TEST_true(test_keylog_output(server_log_buffer, serverssl, + SSL_get_session(serverssl), + &expected))) goto end; - } testresult = 1; @@ -482,6 +405,7 @@ static int full_early_callback(SSL *s, int *al, void *arg) { int *ctr = arg; const unsigned char *p; + int *exts; /* We only configure two ciphers, but the SCSV is added automatically. */ #ifdef OPENSSL_NO_EC const unsigned char expected_ciphers[] = {0x00, 0x9d, 0x00, 0xff}; @@ -489,6 +413,11 @@ static int full_early_callback(SSL *s, int *al, void *arg) const unsigned char expected_ciphers[] = {0x00, 0x9d, 0xc0, 0x2c, 0x00, 0xff}; #endif + const int expected_extensions[] = { +#ifndef OPENSSL_NO_EC + 11, 10, +#endif + 35, 13, 22, 23}; size_t len; /* Make sure we can defer processing and get called back. */ @@ -496,59 +425,51 @@ static int full_early_callback(SSL *s, int *al, void *arg) return -1; len = SSL_early_get0_ciphers(s, &p); - if (len != sizeof(expected_ciphers) || - memcmp(p, expected_ciphers, len) != 0) { - printf("Early callback expected ciphers mismatch\n"); + if (!TEST_mem_eq(p, len, expected_ciphers, sizeof(expected_ciphers)) + || !TEST_size_t_eq(SSL_early_get0_compression_methods(s, &p), 1) + || !TEST_int_eq(*p, 0)) return 0; - } - len = SSL_early_get0_compression_methods(s, &p); - if (len != 1 || *p != 0) { - printf("Early callback expected compression methods mismatch\n"); + if (!SSL_early_get1_extensions_present(s, &exts, &len)) + return 0; + if (len != OSSL_NELEM(expected_extensions) || + memcmp(exts, expected_extensions, len * sizeof(*exts)) != 0) { + printf("Early callback expected ClientHello extensions mismatch\n"); + OPENSSL_free(exts); return 0; } + OPENSSL_free(exts); return 1; } -static int test_early_cb(void) { +static int test_early_cb(void) +{ SSL_CTX *cctx = NULL, *sctx = NULL; SSL *clientssl = NULL, *serverssl = NULL; int testctr = 0, testresult = 0; - if (!create_ssl_ctx_pair(TLS_server_method(), TLS_client_method(), &sctx, - &cctx, cert, privkey)) { - printf("Unable to create SSL_CTX pair\n"); + if (!TEST_true(create_ssl_ctx_pair(TLS_server_method(), + TLS_client_method(), &sctx, + &cctx, cert, privkey))) goto end; - } - SSL_CTX_set_early_cb(sctx, full_early_callback, &testctr); + /* The gimpy cipher list we configure can't do TLS 1.3. */ SSL_CTX_set_max_proto_version(cctx, TLS1_2_VERSION); - if (!SSL_CTX_set_cipher_list(cctx, - "AES256-GCM-SHA384:ECDHE-ECDSA-AES256-GCM-SHA384")) { - printf("Failed to set cipher list\n"); - goto end; - } - - if (!create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, NULL)) { - printf("Unable to create SSL objects\n"); - goto end; - } - - if (create_ssl_connection(serverssl, clientssl, SSL_ERROR_WANT_EARLY)) { - printf("Creating SSL connection succeeded with async early return\n"); - goto end; - } - - /* Passing a -1 literal is a hack since the real value was lost. */ - if (SSL_get_error(serverssl, -1) != SSL_ERROR_WANT_EARLY) { - printf("Early callback failed to make state SSL_ERROR_WANT_EARLY\n"); - goto end; - } - if (!create_ssl_connection(serverssl, clientssl, SSL_ERROR_NONE)) { - printf("Restarting SSL connection failed\n"); + if (!TEST_true(SSL_CTX_set_cipher_list(cctx, + "AES256-GCM-SHA384:ECDHE-ECDSA-AES256-GCM-SHA384")) + || !TEST_true(create_ssl_objects(sctx, cctx, &serverssl, + &clientssl, NULL, NULL)) + || !TEST_false(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_WANT_EARLY)) + /* + * Passing a -1 literal is a hack since + * the real value was lost. + * */ + || !TEST_int_eq(SSL_get_error(serverssl, -1), SSL_ERROR_WANT_EARLY) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE))) goto end; - } testresult = 1; @@ -569,29 +490,23 @@ static int execute_test_large_message(const SSL_METHOD *smeth, SSL *clientssl = NULL, *serverssl = NULL; int testresult = 0; int i; - BIO *certbio = BIO_new_file(cert, "r"); + BIO *certbio = NULL; X509 *chaincert = NULL; int certlen; - if (certbio == NULL) { - printf("Can't load the certificate file\n"); + if (!TEST_ptr(certbio = BIO_new_file(cert, "r"))) goto end; - } chaincert = PEM_read_bio_X509(certbio, NULL, NULL, NULL); BIO_free(certbio); certbio = NULL; - if (chaincert == NULL) { - printf("Unable to load certificate for chain\n"); + if (!TEST_ptr(chaincert)) goto end; - } - if (!create_ssl_ctx_pair(smeth, cmeth, &sctx, - &cctx, cert, privkey)) { - printf("Unable to create SSL_CTX pair\n"); + if (!TEST_true(create_ssl_ctx_pair(smeth, cmeth, &sctx, + &cctx, cert, privkey))) goto end; - } - if(read_ahead) { + if (read_ahead) { /* * Test that read_ahead works correctly when dealing with large * records @@ -603,42 +518,33 @@ static int execute_test_large_message(const SSL_METHOD *smeth, * We assume the supplied certificate is big enough so that if we add * NUM_EXTRA_CERTS it will make the overall message large enough. The * default buffer size is requested to be 16k, but due to the way BUF_MEM - * works, it ends up allocating a little over 21k (16 * 4/3). So, in this test - * we need to have a message larger than that. + * works, it ends up allocating a little over 21k (16 * 4/3). So, in this + * test we need to have a message larger than that. */ certlen = i2d_X509(chaincert, NULL); - OPENSSL_assert((certlen * NUM_EXTRA_CERTS) - > ((SSL3_RT_MAX_PLAIN_LENGTH * 4) / 3)); + OPENSSL_assert(certlen * NUM_EXTRA_CERTS > + (SSL3_RT_MAX_PLAIN_LENGTH * 4) / 3); for (i = 0; i < NUM_EXTRA_CERTS; i++) { - if (!X509_up_ref(chaincert)) { - printf("Unable to up ref cert\n"); + if (!X509_up_ref(chaincert)) goto end; - } if (!SSL_CTX_add_extra_chain_cert(sctx, chaincert)) { - printf("Unable to add extra chain cert %d\n", i); X509_free(chaincert); goto end; } } - if (!create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, NULL)) { - printf("Unable to create SSL objects\n"); - goto end; - } - - if (!create_ssl_connection(serverssl, clientssl, SSL_ERROR_NONE)) { - printf("Unable to create SSL connection\n"); + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, &clientssl, + NULL, NULL)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE))) goto end; - } /* * Calling SSL_clear() first is not required but this tests that SSL_clear() * doesn't leak (when using enable-crypto-mdebug). */ - if (!SSL_clear(serverssl)) { - printf("Unexpected failure from SSL_clear()\n"); + if (!TEST_true(SSL_clear(serverssl))) goto end; - } testresult = 1; end: @@ -679,7 +585,7 @@ static int test_large_message_dtls(void) static int ocsp_server_cb(SSL *s, void *arg) { int *argi = (int *)arg; - unsigned char *orespdercopy = NULL; + unsigned char *copy = NULL; STACK_OF(OCSP_RESPID) *ids = NULL; OCSP_RESPID *id = NULL; @@ -696,15 +602,11 @@ static int ocsp_server_cb(SSL *s, void *arg) return SSL_TLSEXT_ERR_ALERT_FATAL; } - - orespdercopy = OPENSSL_memdup(orespder, sizeof(orespder)); - if (orespdercopy == NULL) + if (!TEST_ptr(copy = OPENSSL_memdup(orespder, sizeof(orespder)))) return SSL_TLSEXT_ERR_ALERT_FATAL; - SSL_set_tlsext_status_ocsp_resp(s, orespdercopy, sizeof(orespder)); - + SSL_set_tlsext_status_ocsp_resp(s, copy, sizeof(orespder)); ocsp_server_called = 1; - return SSL_TLSEXT_ERR_OK; } @@ -718,12 +620,10 @@ static int ocsp_client_cb(SSL *s, void *arg) return 0; len = SSL_get_tlsext_status_ocsp_resp(s, &respderin); - - if (memcmp(orespder, respderin, len) != 0) + if (!TEST_mem_eq(orespder, len, respderin, len)) return 0; ocsp_client_called = 1; - return 1; } @@ -737,55 +637,32 @@ static int test_tlsext_status_type(void) BIO *certbio = NULL; if (!create_ssl_ctx_pair(TLS_server_method(), TLS_client_method(), &sctx, - &cctx, cert, privkey)) { - printf("Unable to create SSL_CTX pair\n"); + &cctx, cert, privkey)) return 0; - } - if (SSL_CTX_get_tlsext_status_type(cctx) != -1) { - printf("Unexpected initial value for " - "SSL_CTX_get_tlsext_status_type()\n"); + if (SSL_CTX_get_tlsext_status_type(cctx) != -1) goto end; - } /* First just do various checks getting and setting tlsext_status_type */ clientssl = SSL_new(cctx); - if (SSL_get_tlsext_status_type(clientssl) != -1) { - printf("Unexpected initial value for SSL_get_tlsext_status_type()\n"); - goto end; - } - - if (!SSL_set_tlsext_status_type(clientssl, TLSEXT_STATUSTYPE_ocsp)) { - printf("Unexpected fail for SSL_set_tlsext_status_type()\n"); - goto end; - } - - if (SSL_get_tlsext_status_type(clientssl) != TLSEXT_STATUSTYPE_ocsp) { - printf("Unexpected result for SSL_get_tlsext_status_type()\n"); + if (!TEST_int_eq(SSL_get_tlsext_status_type(clientssl), -1) + || !TEST_true(SSL_set_tlsext_status_type(clientssl, + TLSEXT_STATUSTYPE_ocsp)) + || !TEST_int_eq(SSL_get_tlsext_status_type(clientssl), + TLSEXT_STATUSTYPE_ocsp)) goto end; - } SSL_free(clientssl); clientssl = NULL; - if (!SSL_CTX_set_tlsext_status_type(cctx, TLSEXT_STATUSTYPE_ocsp)) { - printf("Unexpected fail for SSL_CTX_set_tlsext_status_type()\n"); - goto end; - } - - if (SSL_CTX_get_tlsext_status_type(cctx) != TLSEXT_STATUSTYPE_ocsp) { - printf("Unexpected result for SSL_CTX_get_tlsext_status_type()\n"); + if (!SSL_CTX_set_tlsext_status_type(cctx, TLSEXT_STATUSTYPE_ocsp) + || SSL_CTX_get_tlsext_status_type(cctx) != TLSEXT_STATUSTYPE_ocsp) goto end; - } clientssl = SSL_new(cctx); - - if (SSL_get_tlsext_status_type(clientssl) != TLSEXT_STATUSTYPE_ocsp) { - printf("Unexpected result for SSL_get_tlsext_status_type() (test 2)\n"); + if (SSL_get_tlsext_status_type(clientssl) != TLSEXT_STATUSTYPE_ocsp) goto end; - } - SSL_free(clientssl); clientssl = NULL; @@ -793,27 +670,17 @@ static int test_tlsext_status_type(void) * Now actually do a handshake and check OCSP information is exchanged and * the callbacks get called */ - SSL_CTX_set_tlsext_status_cb(cctx, ocsp_client_cb); SSL_CTX_set_tlsext_status_arg(cctx, &cdummyarg); SSL_CTX_set_tlsext_status_cb(sctx, ocsp_server_cb); SSL_CTX_set_tlsext_status_arg(sctx, &cdummyarg); - - if (!create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, NULL)) { - printf("Unable to create SSL objects\n"); - goto end; - } - - if (!create_ssl_connection(serverssl, clientssl, SSL_ERROR_NONE)) { - printf("Unable to create SSL connection\n"); - goto end; - } - - if (!ocsp_client_called || !ocsp_server_called) { - printf("OCSP callbacks not called\n"); + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, + &clientssl, NULL, NULL)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE)) + || !TEST_true(ocsp_client_called) + || !TEST_true(ocsp_server_called)) goto end; - } - SSL_free(serverssl); SSL_free(clientssl); serverssl = NULL; @@ -823,23 +690,14 @@ static int test_tlsext_status_type(void) ocsp_client_called = 0; ocsp_server_called = 0; cdummyarg = 0; - - if (!create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, NULL)) { - printf("Unable to create SSL objects\n"); - goto end; - } - - /* This should fail because the callback will fail */ - if (create_ssl_connection(serverssl, clientssl, SSL_ERROR_NONE)) { - printf("Unexpected success creating the connection\n"); - goto end; - } - - if (ocsp_client_called || ocsp_server_called) { - printf("OCSP callbacks successfully called unexpectedly\n"); + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, + &clientssl, NULL, NULL)) + /* This should fail because the callback will fail */ + || !TEST_false(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE)) + || !TEST_false(ocsp_client_called) + || !TEST_false(ocsp_server_called)) goto end; - } - SSL_free(serverssl); SSL_free(clientssl); serverssl = NULL; @@ -852,30 +710,22 @@ static int test_tlsext_status_type(void) ocsp_client_called = 0; ocsp_server_called = 0; cdummyarg = 2; - - if (!create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, NULL)) { - printf("Unable to create SSL objects\n"); + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, + &clientssl, NULL, NULL))) goto end; - } /* * We'll just use any old cert for this test - it doesn't have to be an OCSP * specific one. We'll use the server cert. */ - certbio = BIO_new_file(cert, "r"); - if (certbio == NULL) { - printf("Can't load the certificate file\n"); - goto end; - } - id = OCSP_RESPID_new(); - ids = sk_OCSP_RESPID_new_null(); - ocspcert = PEM_read_bio_X509(certbio, NULL, NULL, NULL); - if (id == NULL || ids == NULL || ocspcert == NULL - || !OCSP_RESPID_set_by_key(id, ocspcert) - || !sk_OCSP_RESPID_push(ids, id)) { - printf("Unable to set OCSP_RESPIDs\n"); + if (!TEST_ptr(certbio = BIO_new_file(cert, "r")) + || !TEST_ptr(id = OCSP_RESPID_new()) + || !TEST_ptr(ids = sk_OCSP_RESPID_new_null()) + || !TEST_ptr(ocspcert = PEM_read_bio_X509(certbio, + NULL, NULL, NULL)) + || !TEST_true(OCSP_RESPID_set_by_key(id, ocspcert)) + || !TEST_true(sk_OCSP_RESPID_push(ids, id))) goto end; - } id = NULL; SSL_set_tlsext_status_ids(clientssl, ids); /* Control has been transferred */ @@ -884,15 +734,11 @@ static int test_tlsext_status_type(void) BIO_free(certbio); certbio = NULL; - if (!create_ssl_connection(serverssl, clientssl, SSL_ERROR_NONE)) { - printf("Unable to create SSL connection\n"); + if (!TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE)) + || !TEST_true(ocsp_client_called) + || !TEST_true(ocsp_server_called)) goto end; - } - - if (!ocsp_client_called || !ocsp_server_called) { - printf("OCSP callbacks not called\n"); - goto end; - } testresult = 1; @@ -940,7 +786,6 @@ static void ssl_session_tear_down(SSL_SESSION_TEST_FIXTURE fixture) static int new_session_cb(SSL *ssl, SSL_SESSION *sess) { new_called++; - return 1; } @@ -960,11 +805,10 @@ static int execute_test_session(SSL_SESSION_TEST_FIXTURE fix) SSL_SESSION *sess1 = NULL, *sess2 = NULL; int testresult = 0; - if (!create_ssl_ctx_pair(TLS_server_method(), TLS_client_method(), &sctx, - &cctx, cert, privkey)) { - printf("Unable to create SSL_CTX pair\n"); + if (!TEST_true(create_ssl_ctx_pair(TLS_server_method(), + TLS_client_method(), &sctx, + &cctx, cert, privkey))) return 0; - } #ifndef OPENSSL_NO_TLS1_2 /* Only allow TLS1.2 so we can force a connection failure later */ @@ -985,89 +829,50 @@ static int execute_test_session(SSL_SESSION_TEST_FIXTURE fix) | SSL_SESS_CACHE_NO_INTERNAL_STORE); } - if (!create_ssl_objects(sctx, cctx, &serverssl1, &clientssl1, NULL, - NULL)) { - printf("Unable to create SSL objects\n"); - goto end; - } - - if (!create_ssl_connection(serverssl1, clientssl1, SSL_ERROR_NONE)) { - printf("Unable to create SSL connection\n"); - goto end; - } - sess1 = SSL_get1_session(clientssl1); - if (sess1 == NULL) { - printf("Unexpected NULL session\n"); - goto end; - } - - if (fix.use_int_cache && SSL_CTX_add_session(cctx, sess1)) { - /* Should have failed because it should already be in the cache */ - printf("Unexpected success adding session to cache\n"); + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl1, &clientssl1, + NULL, NULL)) + || !TEST_true(create_ssl_connection(serverssl1, clientssl1, + SSL_ERROR_NONE)) + || !TEST_ptr(sess1 = SSL_get1_session(clientssl1))) goto end; - } - if (fix.use_ext_cache && (new_called != 1 || remove_called != 0)) { - printf("Session not added to cache\n"); + /* Should fail because it should already be in the cache */ + if (fix.use_int_cache && !TEST_false(SSL_CTX_add_session(cctx, sess1))) goto end; - } - - if (!create_ssl_objects(sctx, cctx, &serverssl2, &clientssl2, NULL, NULL)) { - printf("Unable to create second SSL objects\n"); + if (fix.use_ext_cache && (new_called != 1 || remove_called != 0)) goto end; - } - if (!create_ssl_connection(serverssl2, clientssl2, SSL_ERROR_NONE)) { - printf("Unable to create second SSL connection\n"); + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl2, + &clientssl2, NULL, NULL)) + || !TEST_true(create_ssl_connection(serverssl2, clientssl2, + SSL_ERROR_NONE))) goto end; - } - sess2 = SSL_get1_session(clientssl2); - if (sess2 == NULL) { - printf("Unexpected NULL session from clientssl2\n"); + if (!TEST_ptr(sess2 = SSL_get1_session(clientssl2))) goto end; - } - if (fix.use_ext_cache && (new_called != 2 || remove_called != 0)) { - printf("Remove session callback unexpectedly called\n"); + if (fix.use_ext_cache && (new_called != 2 || remove_called != 0)) goto end; - } /* - * This should clear sess2 from the cache because it is a "bad" session. See - * SSL_set_session() documentation. + * This should clear sess2 from the cache because it is a "bad" session. + * See SSL_set_session() documentation. */ - if (!SSL_set_session(clientssl2, sess1)) { - printf("Unexpected failure setting session\n"); + if (!TEST_true(SSL_set_session(clientssl2, sess1))) goto end; - } - - if (fix.use_ext_cache && (new_called != 2 || remove_called != 1)) { - printf("Failed to call callback to remove session\n"); + if (fix.use_ext_cache && (new_called != 2 || remove_called != 1)) goto end; - } - - - if (SSL_get_session(clientssl2) != sess1) { - printf("Unexpected session found\n"); + if (!TEST_ptr_eq(SSL_get_session(clientssl2), sess1)) goto end; - } if (fix.use_int_cache) { - if (!SSL_CTX_add_session(cctx, sess2)) { - /* - * Should have succeeded because it should not already be in the cache - */ - printf("Unexpected failure adding session to cache\n"); - goto end; - } - - if (!SSL_CTX_remove_session(cctx, sess2)) { - printf("Unexpected failure removing session from cache\n"); + /* Should succeeded because it should not already be in the cache */ + if (!TEST_true(SSL_CTX_add_session(cctx, sess2)) + || !TEST_true(SSL_CTX_remove_session(cctx, sess2))) goto end; - } - /* This is for the purposes of internal cache testing...ignore the + /* + * This is for the purposes of internal cache testing...ignore the * counter for external cache */ if (fix.use_ext_cache) @@ -1075,50 +880,30 @@ static int execute_test_session(SSL_SESSION_TEST_FIXTURE fix) } /* This shouldn't be in the cache so should fail */ - if (SSL_CTX_remove_session(cctx, sess2)) { - printf("Unexpected success removing session from cache\n"); + if (!TEST_false(SSL_CTX_remove_session(cctx, sess2))) goto end; - } - if (fix.use_ext_cache && (new_called != 2 || remove_called != 2)) { - printf("Failed to call callback to remove session #2\n"); + if (fix.use_ext_cache && (new_called != 2 || remove_called != 2)) goto end; - } #if !defined(OPENSSL_NO_TLS1_1) && !defined(OPENSSL_NO_TLS1_2) /* Force a connection failure */ SSL_CTX_set_max_proto_version(sctx, TLS1_1_VERSION); - - if (!create_ssl_objects(sctx, cctx, &serverssl3, &clientssl3, NULL, NULL)) { - printf("Unable to create third SSL objects\n"); - goto end; - } - - if (!SSL_set_session(clientssl3, sess1)) { - printf("Unable to set session for third connection\n"); - goto end; - } - - /* This should fail because of the mismatched protocol versions */ - if (create_ssl_connection(serverssl3, clientssl3, SSL_ERROR_NONE)) { - printf("Unable to create third SSL connection\n"); + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl3, + &clientssl3, NULL, NULL)) + || !TEST_true(SSL_set_session(clientssl3, sess1)) + /* This should fail because of the mismatched protocol versions */ + || !TEST_false(create_ssl_connection(serverssl3, clientssl3, + SSL_ERROR_NONE))) goto end; - } - /* We should have automatically removed the session from the cache */ - if (fix.use_ext_cache && (new_called != 2 || remove_called != 3)) { - printf("Failed to call callback to remove session #2\n"); + if (fix.use_ext_cache && (new_called != 2 || remove_called != 3)) goto end; - } - if (fix.use_int_cache && !SSL_CTX_add_session(cctx, sess2)) { - /* - * Should have succeeded because it should not already be in the cache - */ - printf("Unexpected failure adding session to cache #2\n"); + /* Should succeed because it should not already be in the cache */ + if (fix.use_int_cache && !SSL_CTX_add_session(cctx, sess2)) goto end; - } #endif testresult = 1; @@ -1134,6 +919,7 @@ static int execute_test_session(SSL_SESSION_TEST_FIXTURE fix) #endif SSL_SESSION_free(sess1); SSL_SESSION_free(sess2); + /* * Check if we need to remove any sessions up-refed for the external cache */ @@ -1150,25 +936,20 @@ static int execute_test_session(SSL_SESSION_TEST_FIXTURE fix) static int test_session_with_only_int_cache(void) { SETUP_TEST_FIXTURE(SSL_SESSION_TEST_FIXTURE, ssl_session_set_up); - fixture.use_ext_cache = 0; - EXECUTE_TEST(execute_test_session, ssl_session_tear_down); } static int test_session_with_only_ext_cache(void) { SETUP_TEST_FIXTURE(SSL_SESSION_TEST_FIXTURE, ssl_session_set_up); - fixture.use_int_cache = 0; - EXECUTE_TEST(execute_test_session, ssl_session_tear_down); } static int test_session_with_both_cache(void) { SETUP_TEST_FIXTURE(SSL_SESSION_TEST_FIXTURE, ssl_session_set_up); - EXECUTE_TEST(execute_test_session, ssl_session_tear_down); } @@ -1195,7 +976,7 @@ static void setupbio(BIO **res, BIO *bio1, BIO *bio2, int type) static int test_ssl_set_bio(int idx) { - SSL_CTX *ctx = SSL_CTX_new(TLS_method()); + SSL_CTX *ctx; BIO *bio1 = NULL; BIO *bio2 = NULL; BIO *irbio = NULL, *iwbio = NULL, *nrbio = NULL, *nwbio = NULL; @@ -1203,17 +984,6 @@ static int test_ssl_set_bio(int idx) int initrbio, initwbio, newrbio, newwbio; int testresult = 0; - if (ctx == NULL) { - printf("Failed to allocate SSL_CTX\n"); - goto end; - } - - ssl = SSL_new(ctx); - if (ssl == NULL) { - printf("Failed to allocate SSL object\n"); - goto end; - } - initrbio = idx % 3; idx /= 3; initwbio = idx % 3; @@ -1221,24 +991,27 @@ static int test_ssl_set_bio(int idx) newrbio = idx % 3; idx /= 3; newwbio = idx; - OPENSSL_assert(newwbio <= 2); + if (!TEST_int_le(newwbio, 2)) + return 0; + + if (!TEST_ptr(ctx = SSL_CTX_new(TLS_method())) + || !TEST_ptr(ssl = SSL_new(ctx))) + goto end; - if (initrbio == USE_BIO_1 || initwbio == USE_BIO_1 || newrbio == USE_BIO_1 + if (initrbio == USE_BIO_1 + || initwbio == USE_BIO_1 + || newrbio == USE_BIO_1 || newwbio == USE_BIO_1) { - bio1 = BIO_new(BIO_s_mem()); - if (bio1 == NULL) { - printf("Failed to allocate bio1\n"); + if (!TEST_ptr(bio1 = BIO_new(BIO_s_mem()))) goto end; - } } - if (initrbio == USE_BIO_2 || initwbio == USE_BIO_2 || newrbio == USE_BIO_2 + if (initrbio == USE_BIO_2 + || initwbio == USE_BIO_2 + || newrbio == USE_BIO_2 || newwbio == USE_BIO_2) { - bio2 = BIO_new(BIO_s_mem()); - if (bio2 == NULL) { - printf("Failed to allocate bio2\n"); + if (!TEST_ptr(bio2 = BIO_new(BIO_s_mem()))) goto end; - } } setupbio(&irbio, bio1, bio2, initrbio); @@ -1263,9 +1036,13 @@ static int test_ssl_set_bio(int idx) * SSL_set_bio() has some really complicated ownership rules where BIOs have * already been set! */ - if (nrbio != NULL && nrbio != irbio && (nwbio != iwbio || nrbio != nwbio)) + if (nrbio != NULL + && nrbio != irbio + && (nwbio != iwbio || nrbio != nwbio)) BIO_up_ref(nrbio); - if (nwbio != NULL && nwbio != nrbio && (nwbio != iwbio || (nwbio == iwbio && irbio == iwbio))) + if (nwbio != NULL + && nwbio != nrbio + && (nwbio != iwbio || (nwbio == iwbio && irbio == iwbio))) BIO_up_ref(nwbio); SSL_set_bio(ssl, nrbio, nwbio); @@ -1276,6 +1053,7 @@ static int test_ssl_set_bio(int idx) SSL_free(ssl); BIO_free(bio1); BIO_free(bio2); + /* * This test is checking that the ref counting for SSL_set_bio is correct. * If we get here and we did too many frees then we will fail in the above @@ -1283,7 +1061,6 @@ static int test_ssl_set_bio(int idx) * a crypto-mdebug build */ SSL_CTX_free(ctx); - return testresult; } @@ -1300,7 +1077,6 @@ static SSL_BIO_TEST_FIXTURE ssl_bio_set_up(const char *const test_case_name) fixture.test_case_name = test_case_name; fixture.pop_ssl = 0; fixture.change_bio = NO_BIO_CHANGE; - return fixture; } @@ -1311,28 +1087,15 @@ static void ssl_bio_tear_down(SSL_BIO_TEST_FIXTURE fixture) static int execute_test_ssl_bio(SSL_BIO_TEST_FIXTURE fix) { BIO *sslbio = NULL, *membio1 = NULL, *membio2 = NULL; - SSL_CTX *ctx = SSL_CTX_new(TLS_method()); + SSL_CTX *ctx; SSL *ssl = NULL; int testresult = 0; - if (ctx == NULL) { - printf("Failed to allocate SSL_CTX\n"); - return 0; - } - - ssl = SSL_new(ctx); - if (ssl == NULL) { - printf("Failed to allocate SSL object\n"); - goto end; - } - - sslbio = BIO_new(BIO_f_ssl()); - membio1 = BIO_new(BIO_s_mem()); - - if (sslbio == NULL || membio1 == NULL) { - printf("Malloc failure creating BIOs\n"); + if (!TEST_ptr(ctx = SSL_CTX_new(TLS_method())) + || !TEST_ptr(ssl = SSL_new(ctx)) + || !TEST_ptr(sslbio = BIO_new(BIO_f_ssl())) + || !TEST_ptr(membio1 = BIO_new(BIO_s_mem()))) goto end; - } BIO_set_ssl(sslbio, ssl, BIO_CLOSE); @@ -1344,11 +1107,8 @@ static int execute_test_ssl_bio(SSL_BIO_TEST_FIXTURE fix) /* Verify changing the rbio/wbio directly does not cause leaks */ if (fix.change_bio != NO_BIO_CHANGE) { - membio2 = BIO_new(BIO_s_mem()); - if (membio2 == NULL) { - printf("Malloc failure creating membio2\n"); + if (!TEST_ptr(membio2 = BIO_new(BIO_s_mem()))) goto end; - } if (fix.change_bio == CHANGE_RBIO) SSL_set0_rbio(ssl, membio2); else @@ -1374,34 +1134,27 @@ static int execute_test_ssl_bio(SSL_BIO_TEST_FIXTURE fix) static int test_ssl_bio_pop_next_bio(void) { SETUP_TEST_FIXTURE(SSL_BIO_TEST_FIXTURE, ssl_bio_set_up); - EXECUTE_TEST(execute_test_ssl_bio, ssl_bio_tear_down); } static int test_ssl_bio_pop_ssl_bio(void) { SETUP_TEST_FIXTURE(SSL_BIO_TEST_FIXTURE, ssl_bio_set_up); - fixture.pop_ssl = 1; - EXECUTE_TEST(execute_test_ssl_bio, ssl_bio_tear_down); } static int test_ssl_bio_change_rbio(void) { SETUP_TEST_FIXTURE(SSL_BIO_TEST_FIXTURE, ssl_bio_set_up); - fixture.change_bio = CHANGE_RBIO; - EXECUTE_TEST(execute_test_ssl_bio, ssl_bio_tear_down); } static int test_ssl_bio_change_wbio(void) { SETUP_TEST_FIXTURE(SSL_BIO_TEST_FIXTURE, ssl_bio_set_up); - fixture.change_bio = CHANGE_WBIO; - EXECUTE_TEST(execute_test_ssl_bio, ssl_bio_tear_down); } @@ -1445,7 +1198,8 @@ static const sigalgs_list testsigalgs[] = { {NULL, 0, "RSA", 0, 0}, {NULL, 0, "SHA256", 0, 0}, {NULL, 0, "RSA+SHA256:SHA256", 0, 0}, - {NULL, 0, "Invalid", 0, 0}}; + {NULL, 0, "Invalid", 0, 0} +}; static int test_set_sigalgs(int idx) { @@ -1456,18 +1210,17 @@ static int test_set_sigalgs(int idx) int testctx; /* Should never happen */ - if ((size_t)idx >= OSSL_NELEM(testsigalgs) * 2) + if (!TEST_size_t_le((size_t)idx, OSSL_NELEM(testsigalgs) * 2)) return 0; testctx = ((size_t)idx < OSSL_NELEM(testsigalgs)); curr = testctx ? &testsigalgs[idx] : &testsigalgs[idx - OSSL_NELEM(testsigalgs)]; - if (!create_ssl_ctx_pair(TLS_server_method(), TLS_client_method(), &sctx, - &cctx, cert, privkey)) { - printf("Unable to create SSL_CTX pair\n"); + if (!TEST_true(create_ssl_ctx_pair(TLS_server_method(), + TLS_client_method(), &sctx, + &cctx, cert, privkey))) return 0; - } /* * TODO(TLS1.3): These APIs cannot set TLSv1.3 sig algs so we just test it @@ -1477,6 +1230,7 @@ static int test_set_sigalgs(int idx) if (testctx) { int ret; + if (curr->list != NULL) ret = SSL_CTX_set1_sigalgs(cctx, curr->list, curr->listlen); else @@ -1484,22 +1238,20 @@ static int test_set_sigalgs(int idx) if (!ret) { if (curr->valid) - printf("Unexpected failure setting sigalgs in SSL_CTX (%d)\n", - idx); + TEST_info("Failure setting sigalgs in SSL_CTX (%d)\n", idx); else testresult = 1; goto end; } if (!curr->valid) { - printf("Unexpected success setting sigalgs in SSL_CTX (%d)\n", idx); + TEST_info("Not-failed setting sigalgs in SSL_CTX (%d)\n", idx); goto end; } } - if (!create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, NULL)) { - printf("Unable to create SSL objects\n"); + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, + &clientssl, NULL, NULL))) goto end; - } if (!testctx) { int ret; @@ -1510,21 +1262,19 @@ static int test_set_sigalgs(int idx) ret = SSL_set1_sigalgs_list(clientssl, curr->liststr); if (!ret) { if (curr->valid) - printf("Unexpected failure setting sigalgs in SSL (%d)\n", idx); + TEST_info("Failure setting sigalgs in SSL (%d)\n", idx); else testresult = 1; goto end; } - if (!curr->valid) { - printf("Unexpected success setting sigalgs in SSL (%d)\n", idx); + if (!curr->valid) goto end; - } } - if (curr->connsuccess != create_ssl_connection(serverssl, clientssl, SSL_ERROR_NONE)) { - printf("Unexpected return value creating SSL connection (%d)\n", idx); + if (!TEST_int_eq(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE), + curr->connsuccess)) goto end; - } testresult = 1; @@ -1554,11 +1304,10 @@ static int test_set_sigalgs(int idx) static int setupearly_data_test(SSL_CTX **cctx, SSL_CTX **sctx, SSL **clientssl, SSL **serverssl, SSL_SESSION **sess, int idx) { - if (!create_ssl_ctx_pair(TLS_server_method(), TLS_client_method(), sctx, - cctx, cert, privkey)) { - printf("Unable to create SSL_CTX pair\n"); + if (!TEST_true(create_ssl_ctx_pair(TLS_server_method(), + TLS_client_method(), sctx, + cctx, cert, privkey))) return 0; - } /* When idx == 1 we repeat the tests with read_ahead set */ if (idx > 0) { @@ -1566,34 +1315,23 @@ static int setupearly_data_test(SSL_CTX **cctx, SSL_CTX **sctx, SSL **clientssl, SSL_CTX_set_read_ahead(*sctx, 1); } - if (!create_ssl_objects(*sctx, *cctx, serverssl, clientssl, NULL, NULL)) { - printf("Unable to create SSL objects\n"); - return 0; - } - - if (!create_ssl_connection(*serverssl, *clientssl, SSL_ERROR_NONE)) { - printf("Unable to create SSL connection\n"); + if (!TEST_true(create_ssl_objects(*sctx, *cctx, serverssl, clientssl, + NULL, NULL)) + || !TEST_true(create_ssl_connection(*serverssl, *clientssl, + SSL_ERROR_NONE))) return 0; - } *sess = SSL_get1_session(*clientssl); - SSL_shutdown(*clientssl); SSL_shutdown(*serverssl); - SSL_free(*serverssl); SSL_free(*clientssl); *serverssl = *clientssl = NULL; - if (!create_ssl_objects(*sctx, *cctx, serverssl, clientssl, NULL, NULL)) { - printf("Unable to create SSL objects (2)\n"); - return 0; - } - - if (!SSL_set_session(*clientssl, *sess)) { - printf("Failed setting session\n"); + if (!TEST_true(create_ssl_objects(*sctx, *cctx, serverssl, + clientssl, NULL, NULL)) + || !TEST_true(SSL_set_session(*clientssl, *sess))) return 0; - } return 1; } @@ -1608,90 +1346,63 @@ static int test_early_data_read_write(int idx) size_t readbytes, written, eoedlen, rawread, rawwritten; BIO *rbio; - if (!setupearly_data_test(&cctx, &sctx, &clientssl, &serverssl, &sess, idx)) + if (!TEST_true(setupearly_data_test(&cctx, &sctx, &clientssl, + &serverssl, &sess, idx))) goto end; /* Write and read some early data */ - if (!SSL_write_early_data(clientssl, MSG1, strlen(MSG1), &written) - || written != strlen(MSG1)) { - printf("Failed writing early data message 1\n"); - goto end; - } - - if (SSL_read_early_data(serverssl, buf, sizeof(buf), &readbytes) - != SSL_READ_EARLY_DATA_SUCCESS - || readbytes != strlen(MSG1) - || memcmp(MSG1, buf, strlen(MSG1))) { - printf("Failed reading early data message 1\n"); + if (!TEST_true(SSL_write_early_data(clientssl, MSG1, strlen(MSG1), + &written)) + || !TEST_size_t_eq(written, strlen(MSG1)) + || !TEST_int_eq(SSL_read_early_data(serverssl, buf, + sizeof(buf), &readbytes), + SSL_READ_EARLY_DATA_SUCCESS) + || !TEST_mem_eq(MSG1, readbytes, buf, strlen(MSG1)) + || !TEST_int_eq(SSL_get_early_data_status(serverssl), + SSL_EARLY_DATA_ACCEPTED)) goto end; - } - - if (SSL_get_early_data_status(serverssl) != SSL_EARLY_DATA_ACCEPTED) { - printf("Unexpected early data status\n"); - goto end; - } /* * Server should be able to write data, and client should be able to * read it. */ - if (!SSL_write_early_data(serverssl, MSG2, strlen(MSG2), &written) - || written != strlen(MSG2)) { - printf("Failed writing message 2\n"); + if (!TEST_true(SSL_write_early_data(serverssl, MSG2, strlen(MSG2), + &written)) + || !TEST_size_t_eq(written, strlen(MSG2)) + || !TEST_true(SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes)) + || !TEST_mem_eq(buf, readbytes, MSG2, strlen(MSG2))) goto end; - } - - if (!SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes) - || readbytes != strlen(MSG2) - || memcmp(MSG2, buf, strlen(MSG2))) { - printf("Failed reading message 2\n"); - goto end; - } /* Even after reading normal data, client should be able write early data */ - if (!SSL_write_early_data(clientssl, MSG3, strlen(MSG3), &written) - || written != strlen(MSG3)) { - printf("Failed writing early data message 3\n"); + if (!TEST_true(SSL_write_early_data(clientssl, MSG3, strlen(MSG3), + &written)) + || !TEST_size_t_eq(written, strlen(MSG3))) goto end; - } /* Server should still be able read early data after writing data */ - if (SSL_read_early_data(serverssl, buf, sizeof(buf), &readbytes) - != SSL_READ_EARLY_DATA_SUCCESS - || readbytes != strlen(MSG3) - || memcmp(MSG3, buf, strlen(MSG3))) { - printf("Failed reading early data message 3\n"); + if (!TEST_int_eq(SSL_read_early_data(serverssl, buf, sizeof(buf), + &readbytes), + SSL_READ_EARLY_DATA_SUCCESS) + || !TEST_mem_eq(buf, readbytes, MSG3, strlen(MSG3))) goto end; - } /* Write more data from server and read it from client */ - if (!SSL_write_early_data(serverssl, MSG4, strlen(MSG4), &written) - || written != strlen(MSG4)) { - printf("Failed writing message 4\n"); + if (!TEST_true(SSL_write_early_data(serverssl, MSG4, strlen(MSG4), + &written)) + || !TEST_size_t_eq(written, strlen(MSG4)) + || !TEST_true(SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes)) + || !TEST_mem_eq(buf, readbytes, MSG4, strlen(MSG4))) goto end; - } - - if (!SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes) - || readbytes != strlen(MSG4) - || memcmp(MSG4, buf, strlen(MSG4))) { - printf("Failed reading message 4\n"); - goto end; - } /* * If client writes normal data it should mean writing early data is no * longer possible. */ - if (!SSL_write_ex(clientssl, MSG5, strlen(MSG5), &written) - || written != strlen(MSG5)) { - printf("Failed writing message 5\n"); + if (!TEST_true(SSL_write_ex(clientssl, MSG5, strlen(MSG5), &written)) + || !TEST_size_t_eq(written, strlen(MSG5)) + || !TEST_int_eq(SSL_get_early_data_status(clientssl), + SSL_EARLY_DATA_ACCEPTED)) goto end; - } - - if (SSL_get_early_data_status(clientssl) != SSL_EARLY_DATA_ACCEPTED) { - printf("Unexpected early data status(2)\n"); - goto end; - } /* * At this point the client has written EndOfEarlyData, ClientFinished and @@ -1700,196 +1411,145 @@ static int test_early_data_read_write(int idx) * in the read BIO, and then just put back the EndOfEarlyData message. */ rbio = SSL_get_rbio(serverssl); - if (!BIO_read_ex(rbio, data, sizeof(data), &rawread) - || rawread >= sizeof(data) - || rawread < SSL3_RT_HEADER_LENGTH) { - printf("Failed reading data from rbio\n"); + if (!TEST_true(BIO_read_ex(rbio, data, sizeof(data), &rawread)) + || !TEST_size_t_lt(rawread, sizeof(data)) + || !TEST_size_t_gt(rawread, SSL3_RT_HEADER_LENGTH)) goto end; - } + /* Record length is in the 4th and 5th bytes of the record header */ eoedlen = SSL3_RT_HEADER_LENGTH + (data[3] << 8 | data[4]); - if (!BIO_write_ex(rbio, data, eoedlen, &rawwritten) - || rawwritten != eoedlen) { - printf("Failed to write the EndOfEarlyData message to server rbio\n"); + if (!TEST_true(BIO_write_ex(rbio, data, eoedlen, &rawwritten)) + || !TEST_size_t_eq(rawwritten, eoedlen)) goto end; - } /* Server should be told that there is no more early data */ - if (SSL_read_early_data(serverssl, buf, sizeof(buf), &readbytes) - != SSL_READ_EARLY_DATA_FINISH - || readbytes != 0) { - printf("Failed finishing read of early data\n"); + if (!TEST_int_eq(SSL_read_early_data(serverssl, buf, sizeof(buf), + &readbytes), + SSL_READ_EARLY_DATA_FINISH) + || !TEST_size_t_eq(readbytes, 0)) goto end; - } /* * Server has not finished init yet, so should still be able to write early * data. */ - if (!SSL_write_early_data(serverssl, MSG6, strlen(MSG6), &written) - || written != strlen(MSG6)) { - printf("Failed writing early data message 6\n"); + if (!TEST_true(SSL_write_early_data(serverssl, MSG6, strlen(MSG6), + &written)) + || !TEST_size_t_eq(written, strlen(MSG6))) goto end; - } /* Push the ClientFinished and the normal data back into the server rbio */ - if (!BIO_write_ex(rbio, data + eoedlen, rawread - eoedlen, &rawwritten) - || rawwritten != rawread - eoedlen) { - printf("Failed to write the ClientFinished and data to server rbio\n"); + if (!TEST_true(BIO_write_ex(rbio, data + eoedlen, rawread - eoedlen, + &rawwritten)) + || !TEST_size_t_eq(rawwritten, rawread - eoedlen)) goto end; - } /* Server should be able to read normal data */ - if (!SSL_read_ex(serverssl, buf, sizeof(buf), &readbytes) - || readbytes != strlen(MSG5)) { - printf("Failed reading message 5\n"); + if (!TEST_true(SSL_read_ex(serverssl, buf, sizeof(buf), &readbytes)) + || !TEST_size_t_eq(readbytes, strlen(MSG5))) goto end; - } /* Client and server should not be able to write/read early data now */ - if (SSL_write_early_data(clientssl, MSG6, strlen(MSG6), &written)) { - printf("Unexpected success writing early data\n"); + if (!TEST_false(SSL_write_early_data(clientssl, MSG6, strlen(MSG6), + &written))) goto end; - } ERR_clear_error(); - - if (SSL_read_early_data(serverssl, buf, sizeof(buf), &readbytes) - != SSL_READ_EARLY_DATA_ERROR) { - printf("Unexpected success reading early data\n"); + if (!TEST_int_eq(SSL_read_early_data(serverssl, buf, sizeof(buf), + &readbytes), + SSL_READ_EARLY_DATA_ERROR)) goto end; - } ERR_clear_error(); /* Client should be able to read the data sent by the server */ - if (!SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes) - || readbytes != strlen(MSG6) - || memcmp(MSG6, buf, strlen(MSG6))) { - printf("Failed reading message 6\n"); + if (!TEST_true(SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes)) + || !TEST_mem_eq(buf, readbytes, MSG6, strlen(MSG6))) goto end; - } + /* * Make sure we process the NewSessionTicket. This arrives post-handshake. * We attempt a read which we do not expect to return any data. */ - if (SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes)) { - printf("Unexpected success doing final client read\n"); + if (!TEST_false(SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes))) goto end; - } /* Server should be able to write normal data */ - if (!SSL_write_ex(serverssl, MSG7, strlen(MSG7), &written) - || written != strlen(MSG7)) { - printf("Failed writing normal data message 7\n"); - goto end; - } - if (!SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes) - || readbytes != strlen(MSG7) - || memcmp(MSG7, buf, strlen(MSG7))) { - printf("Failed reading message 7\n"); + if (!TEST_true(SSL_write_ex(serverssl, MSG7, strlen(MSG7), &written)) + || !TEST_size_t_eq(written, strlen(MSG7)) + || !TEST_true(SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes)) + || !TEST_mem_eq(buf, readbytes, MSG7, strlen(MSG7))) goto end; - } SSL_SESSION_free(sess); sess = SSL_get1_session(clientssl); SSL_shutdown(clientssl); SSL_shutdown(serverssl); - SSL_free(serverssl); SSL_free(clientssl); serverssl = clientssl = NULL; - - if (!create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, NULL)) { - printf("Unable to create SSL objects (3)\n"); - goto end; - } - - if (!SSL_set_session(clientssl, sess)) { - printf("Failed setting session (2)\n"); + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, + &clientssl, NULL, NULL)) + || !TEST_true(SSL_set_session(clientssl, sess))) goto end; - } /* Write and read some early data */ - if (!SSL_write_early_data(clientssl, MSG1, strlen(MSG1), &written) - || written != strlen(MSG1)) { - printf("Failed writing early data message 1\n"); - goto end; - } - - if (SSL_read_early_data(serverssl, buf, sizeof(buf), &readbytes) - != SSL_READ_EARLY_DATA_SUCCESS - || readbytes != strlen(MSG1) - || memcmp(MSG1, buf, strlen(MSG1))) { - printf("Failed reading early data message 1\n"); - goto end; - } - - if (SSL_connect(clientssl) <= 0) { - printf("Unable to complete client handshake\n"); + if (!TEST_true(SSL_write_early_data(clientssl, MSG1, strlen(MSG1), + &written)) + || !TEST_size_t_eq(written, strlen(MSG1)) + || !TEST_int_eq(SSL_read_early_data(serverssl, buf, sizeof(buf), + &readbytes), + SSL_READ_EARLY_DATA_SUCCESS) + || !TEST_mem_eq(buf, readbytes, MSG1, strlen(MSG1))) goto end; - } - if (SSL_accept(serverssl) <= 0) { - printf("Unable to complete server handshake\n"); + if (!TEST_int_gt(SSL_connect(clientssl), 0) + || !TEST_int_gt(SSL_accept(serverssl), 0)) goto end; - } /* Client and server should not be able to write/read early data now */ - if (SSL_write_early_data(clientssl, MSG6, strlen(MSG6), &written)) { - printf("Unexpected success writing early data (2)\n"); + if (!TEST_false(SSL_write_early_data(clientssl, MSG6, strlen(MSG6), + &written))) goto end; - } ERR_clear_error(); - - if (SSL_read_early_data(serverssl, buf, sizeof(buf), &readbytes) - != SSL_READ_EARLY_DATA_ERROR) { - printf("Unexpected success reading early data (2)\n"); + if (!TEST_int_eq(SSL_read_early_data(serverssl, buf, sizeof(buf), + &readbytes), + SSL_READ_EARLY_DATA_ERROR)) goto end; - } ERR_clear_error(); /* Client and server should be able to write/read normal data */ - if (!SSL_write_ex(clientssl, MSG5, strlen(MSG5), &written) - || written != strlen(MSG5)) { - printf("Failed writing message 5 (2)\n"); + if (!TEST_true(SSL_write_ex(clientssl, MSG5, strlen(MSG5), &written)) + || !TEST_size_t_eq(written, strlen(MSG5)) + || !TEST_true(SSL_read_ex(serverssl, buf, sizeof(buf), &readbytes)) + || !TEST_size_t_eq(readbytes, strlen(MSG5))) goto end; - } - - if (!SSL_read_ex(serverssl, buf, sizeof(buf), &readbytes) - || readbytes != strlen(MSG5)) { - printf("Failed reading message 5 (2)\n"); - goto end; - } testresult = 1; end: - if(!testresult) - ERR_print_errors_fp(stdout); SSL_SESSION_free(sess); SSL_free(serverssl); SSL_free(clientssl); SSL_CTX_free(sctx); SSL_CTX_free(cctx); - return testresult; } +/* + * Test that a server attempting to read early data can handle a connection + * from a client where the early data is not acceptable. + */ static int test_early_data_skip(int idx) { SSL_CTX *cctx = NULL, *sctx = NULL; SSL *clientssl = NULL, *serverssl = NULL; int testresult = 0; - SSL_SESSION *sess; + SSL_SESSION *sess = NULL; unsigned char buf[20]; size_t readbytes, written; - /* - * Test that a server attempting to read early data can handle a connection - * from a client where the early data is not acceptable. - */ - - if (!setupearly_data_test(&cctx, &sctx, &clientssl, &serverssl, &sess, idx)) + if (!TEST_true(setupearly_data_test(&cctx, &sctx, &clientssl, + &serverssl, &sess, idx))) goto end; /* @@ -1897,242 +1557,172 @@ static int test_early_data_skip(int idx) * It could be any value as long as it is not within tolerance. This should * mean the ticket is rejected. */ - if (!SSL_SESSION_set_time(sess, time(NULL) - 20)) { - printf("Unexpected failure setting session creation time\n"); + if (!TEST_true(SSL_SESSION_set_time(sess, time(NULL) - 20))) goto end; - } /* Write some early data */ - if (!SSL_write_early_data(clientssl, MSG1, strlen(MSG1), &written) - || written != strlen(MSG1)) { - printf("Failed writing early data message 1\n"); + if (!TEST_true(SSL_write_early_data(clientssl, MSG1, strlen(MSG1), + &written)) + || !TEST_size_t_eq(written, strlen(MSG1))) goto end; - } /* Server should reject the early data and skip over it */ - if (SSL_read_early_data(serverssl, buf, sizeof(buf), &readbytes) - != SSL_READ_EARLY_DATA_FINISH - || readbytes != 0) { - printf("Failed reading early data\n"); - goto end; - } - - if (SSL_get_early_data_status(serverssl) != SSL_EARLY_DATA_REJECTED) { - printf("Unexpected early data status\n"); - goto end; - } - - /* - * We should be able to send normal data despite rejection of early data - */ - if (!SSL_write_ex(clientssl, MSG2, strlen(MSG2), &written) - || written != strlen(MSG2)) { - printf("Failed writing message 2\n"); - goto end; - } - - if (SSL_get_early_data_status(clientssl) != SSL_EARLY_DATA_REJECTED) { - printf("Unexpected early data status (2)\n"); + if (!TEST_int_eq(SSL_read_early_data(serverssl, buf, sizeof(buf), + &readbytes), + SSL_READ_EARLY_DATA_FINISH) + || !TEST_size_t_eq(readbytes, 0) + || !TEST_int_eq(SSL_get_early_data_status(serverssl), + SSL_EARLY_DATA_REJECTED)) goto end; - } - if (!SSL_read_ex(serverssl, buf, sizeof(buf), &readbytes) - || readbytes != strlen(MSG2) - || memcmp(MSG2, buf, strlen(MSG2))) { - printf("Failed reading message 2\n"); + /* Should be able to send normal data despite rejection of early data */ + if (!TEST_true(SSL_write_ex(clientssl, MSG2, strlen(MSG2), &written)) + || !TEST_size_t_eq(written, strlen(MSG2)) + || !TEST_int_eq(SSL_get_early_data_status(clientssl), + SSL_EARLY_DATA_REJECTED) + || !TEST_true(SSL_read_ex(serverssl, buf, sizeof(buf), &readbytes)) + || !TEST_mem_eq(buf, readbytes, MSG2, strlen(MSG2))) goto end; - } testresult = 1; end: - if(!testresult) - ERR_print_errors_fp(stdout); SSL_SESSION_free(sess); SSL_free(serverssl); SSL_free(clientssl); SSL_CTX_free(sctx); SSL_CTX_free(cctx); - return testresult; } +/* + * Test that a server attempting to read early data can handle a connection + * from a client that doesn't send any. + */ static int test_early_data_not_sent(int idx) { SSL_CTX *cctx = NULL, *sctx = NULL; SSL *clientssl = NULL, *serverssl = NULL; int testresult = 0; - SSL_SESSION *sess; + SSL_SESSION *sess = NULL; unsigned char buf[20]; size_t readbytes, written; - /* - * Test that a server attempting to read early data can handle a connection - * from a client that doesn't send any. - */ - - if (!setupearly_data_test(&cctx, &sctx, &clientssl, &serverssl, &sess, idx)) + if (!TEST_true(setupearly_data_test(&cctx, &sctx, &clientssl, + &serverssl, &sess, idx))) goto end; /* Write some data - should block due to handshake with server */ SSL_set_connect_state(clientssl); - if (SSL_write_ex(clientssl, MSG1, strlen(MSG1), &written)) { - printf("Unexpected success writing message 1\n"); + if (!TEST_false(SSL_write_ex(clientssl, MSG1, strlen(MSG1), &written))) goto end; - } /* Server should detect that early data has not been sent */ - if (SSL_read_early_data(serverssl, buf, sizeof(buf), &readbytes) - != SSL_READ_EARLY_DATA_FINISH - || readbytes != 0) { - printf("Failed reading early data\n"); - goto end; - } - - if (SSL_get_early_data_status(serverssl) != SSL_EARLY_DATA_NOT_SENT) { - printf("Unexpected early data status\n"); - goto end; - } - - if (SSL_get_early_data_status(clientssl) != SSL_EARLY_DATA_NOT_SENT) { - printf("Unexpected early data status (2)\n"); + if (!TEST_int_eq(SSL_read_early_data(serverssl, buf, sizeof(buf), + &readbytes), + SSL_READ_EARLY_DATA_FINISH) + || !TEST_size_t_eq(readbytes, 0) + || !TEST_int_eq(SSL_get_early_data_status(serverssl), + SSL_EARLY_DATA_NOT_SENT) + || !TEST_int_eq(SSL_get_early_data_status(clientssl), + SSL_EARLY_DATA_NOT_SENT)) goto end; - } /* Continue writing the message we started earlier */ - if (!SSL_write_ex(clientssl, MSG1, strlen(MSG1), &written) - || written != strlen(MSG1)) { - printf("Failed writing message 1\n"); - goto end; - } - - if (!SSL_read_ex(serverssl, buf, sizeof(buf), &readbytes) - || readbytes != strlen(MSG1) - || memcmp(MSG1, buf, strlen(MSG1))) { - printf("Failed reading message 1\n"); + if (!TEST_true(SSL_write_ex(clientssl, MSG1, strlen(MSG1), &written)) + || !TEST_size_t_eq(written, strlen(MSG1)) + || !TEST_true(SSL_read_ex(serverssl, buf, sizeof(buf), &readbytes)) + || !TEST_mem_eq(buf, readbytes, MSG1, strlen(MSG1)) + || !SSL_write_ex(serverssl, MSG2, strlen(MSG2), &written) + || !TEST_size_t_eq(written, strlen(MSG2))) goto end; - } - - if (!SSL_write_ex(serverssl, MSG2, strlen(MSG2), &written) - || written != strlen(MSG2)) { - printf("Failed writing message 2\n"); - goto end; - } /* * Should block due to the NewSessionTicket arrival unless we're using * read_ahead */ if (idx == 0) { - if (SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes)) { - printf("Unexpected success reading message 2\n"); + if (!TEST_false(SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes))) goto end; - } } - if (!SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes) - || readbytes != strlen(MSG2) - || memcmp(MSG2, buf, strlen(MSG2))) { - printf("Failed reading message 2\n"); + if (!TEST_true(SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes)) + || !TEST_mem_eq(buf, readbytes, MSG2, strlen(MSG2))) goto end; - } testresult = 1; end: - if(!testresult) - ERR_print_errors_fp(stdout); SSL_SESSION_free(sess); SSL_free(serverssl); SSL_free(clientssl); SSL_CTX_free(sctx); SSL_CTX_free(cctx); - return testresult; } +/* + * Test that a server that doesn't try to read early data can handle a + * client sending some. + */ static int test_early_data_not_expected(int idx) { SSL_CTX *cctx = NULL, *sctx = NULL; SSL *clientssl = NULL, *serverssl = NULL; int testresult = 0; - SSL_SESSION *sess; + SSL_SESSION *sess = NULL; unsigned char buf[20]; size_t readbytes, written; - /* - * Test that a server that doesn't try to read early data can handle a - * client sending some. - */ - if (!setupearly_data_test(&cctx, &sctx, &clientssl, &serverssl, &sess, idx)) + if (!TEST_true(setupearly_data_test(&cctx, &sctx, &clientssl, + &serverssl, &sess, idx))) goto end; /* Write some early data */ - if (!SSL_write_early_data(clientssl, MSG1, strlen(MSG1), &written)) { - printf("Unexpected failure writing message 1\n"); + if (!TEST_true(SSL_write_early_data(clientssl, MSG1, strlen(MSG1), + &written))) goto end; - } /* * Server should skip over early data and then block waiting for client to * continue handshake */ - if (SSL_accept(serverssl) > 0) { - printf("Unexpected success setting up server connection\n"); + if (!TEST_int_le(SSL_accept(serverssl), 0) + || !TEST_int_gt(SSL_connect(clientssl), 0) + || !TEST_int_eq(SSL_get_early_data_status(serverssl), + SSL_EARLY_DATA_REJECTED) + || !TEST_int_gt(SSL_accept(serverssl), 0) + || !TEST_int_eq(SSL_get_early_data_status(clientssl), + SSL_EARLY_DATA_REJECTED)) goto end; - } - - if (SSL_connect(clientssl) <= 0) { - printf("Failed setting up client connection\n"); - goto end; - } - - if (SSL_get_early_data_status(serverssl) != SSL_EARLY_DATA_REJECTED) { - printf("Unexpected early data status\n"); - goto end; - } - - if (SSL_accept(serverssl) <= 0) { - printf("Failed setting up server connection\n"); - goto end; - } - - if (SSL_get_early_data_status(clientssl) != SSL_EARLY_DATA_REJECTED) { - printf("Unexpected early data status (2)\n"); - goto end; - } /* Send some normal data from client to server */ - if (!SSL_write_ex(clientssl, MSG2, strlen(MSG2), &written) - || written != strlen(MSG2)) { - printf("Failed writing message 2\n"); + if (!TEST_true(SSL_write_ex(clientssl, MSG2, strlen(MSG2), &written)) + || !TEST_size_t_eq(written, strlen(MSG2))) goto end; - } - if (!SSL_read_ex(serverssl, buf, sizeof(buf), &readbytes) - || readbytes != strlen(MSG2) - || memcmp(MSG2, buf, strlen(MSG2))) { - printf("Failed reading message 2\n"); + if (!TEST_true(SSL_read_ex(serverssl, buf, sizeof(buf), &readbytes)) + || !TEST_mem_eq(buf, readbytes, MSG2, strlen(MSG2))) goto end; - } testresult = 1; end: - if(!testresult) - ERR_print_errors_fp(stdout); SSL_SESSION_free(sess); SSL_free(serverssl); SSL_free(clientssl); SSL_CTX_free(sctx); SSL_CTX_free(cctx); - return testresult; } # ifndef OPENSSL_NO_TLS1_2 +/* + * Test that a server attempting to read early data can handle a connection + * from a TLSv1.2 client. + */ static int test_early_data_tls1_2(int idx) { SSL_CTX *cctx = NULL, *sctx = NULL; @@ -2141,16 +1731,10 @@ static int test_early_data_tls1_2(int idx) unsigned char buf[20]; size_t readbytes, written; - /* - * Test that a server attempting to read early data can handle a connection - * from a TLSv1.2 client. - */ - - if (!create_ssl_ctx_pair(TLS_server_method(), TLS_client_method(), &sctx, - &cctx, cert, privkey)) { - printf("Unable to create SSL_CTX pair\n"); + if (!TEST_true(create_ssl_ctx_pair(TLS_server_method(), + TLS_client_method(), &sctx, + &cctx, cert, privkey))) goto end; - } /* When idx == 1 we repeat the tests with read_ahead set */ if (idx > 0) { @@ -2158,97 +1742,373 @@ static int test_early_data_tls1_2(int idx) SSL_CTX_set_read_ahead(sctx, 1); } - if (!create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, NULL)) { - printf("Unable to create SSL objects\n"); + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, + &clientssl, NULL, NULL))) goto end; - } /* Write some data - should block due to handshake with server */ SSL_set_max_proto_version(clientssl, TLS1_2_VERSION); SSL_set_connect_state(clientssl); - if (SSL_write_ex(clientssl, MSG1, strlen(MSG1), &written)) { - printf("Unexpected success writing message 1\n"); + if (!TEST_false(SSL_write_ex(clientssl, MSG1, strlen(MSG1), &written))) goto end; - } /* * Server should do TLSv1.2 handshake. First it will block waiting for more * messages from client after ServerDone. Then SSL_read_early_data should * finish and detect that early data has not been sent */ - if (SSL_read_early_data(serverssl, buf, sizeof(buf), &readbytes) - != SSL_READ_EARLY_DATA_ERROR) { - printf("Unexpected success reading early data\n"); + if (!TEST_int_eq(SSL_read_early_data(serverssl, buf, sizeof(buf), + &readbytes), + SSL_READ_EARLY_DATA_ERROR)) goto end; - } /* * Continue writing the message we started earlier. Will still block waiting * for the CCS/Finished from server */ - if (SSL_write_ex(clientssl, MSG1, strlen(MSG1), &written)) { - printf("Unexpected success writing message 1\n"); + if (!TEST_false(SSL_write_ex(clientssl, MSG1, strlen(MSG1), &written)) + || !TEST_int_eq(SSL_read_early_data(serverssl, buf, sizeof(buf), + &readbytes), + SSL_READ_EARLY_DATA_FINISH) + || !TEST_size_t_eq(readbytes, 0) + || !TEST_int_eq(SSL_get_early_data_status(serverssl), + SSL_EARLY_DATA_NOT_SENT)) goto end; - } - if (SSL_read_early_data(serverssl, buf, sizeof(buf), &readbytes) - != SSL_READ_EARLY_DATA_FINISH - || readbytes != 0) { - printf("Failed reading early data\n"); + /* Continue writing the message we started earlier */ + if (!TEST_true(SSL_write_ex(clientssl, MSG1, strlen(MSG1), &written)) + || !TEST_size_t_eq(written, strlen(MSG1)) + || !TEST_int_eq(SSL_get_early_data_status(clientssl), + SSL_EARLY_DATA_NOT_SENT) + || !TEST_true(SSL_read_ex(serverssl, buf, sizeof(buf), &readbytes)) + || !TEST_mem_eq(buf, readbytes, MSG1, strlen(MSG1)) + || !TEST_true(SSL_write_ex(serverssl, MSG2, strlen(MSG2), &written)) + || !TEST_size_t_eq(written, strlen(MSG2)) + || !SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes) + || !TEST_mem_eq(buf, readbytes, MSG2, strlen(MSG2))) goto end; - } - if (SSL_get_early_data_status(serverssl) != SSL_EARLY_DATA_NOT_SENT) { - printf("Unexpected early data status\n"); + testresult = 1; + + end: + SSL_free(serverssl); + SSL_free(clientssl); + SSL_CTX_free(sctx); + SSL_CTX_free(cctx); + + return testresult; +} +# endif /* OPENSSL_NO_TLS1_2 */ + +static int test_ciphersuite_change(void) +{ + SSL_CTX *cctx = NULL, *sctx = NULL; + SSL *clientssl = NULL, *serverssl = NULL; + SSL_SESSION *clntsess = NULL; + int testresult = 0; + const SSL_CIPHER *aes_128_gcm_sha256 = NULL; + + /* Create a session based on SHA-256 */ + if (!TEST_true(create_ssl_ctx_pair(TLS_server_method(), + TLS_client_method(), &sctx, + &cctx, cert, privkey)) + || !TEST_true(SSL_CTX_set_cipher_list(cctx, + "TLS13-AES-128-GCM-SHA256")) + || !TEST_true(create_ssl_objects(sctx, cctx, &serverssl, + &clientssl, NULL, NULL)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE))) + goto end; + + clntsess = SSL_get1_session(clientssl); + /* Save for later */ + aes_128_gcm_sha256 = SSL_SESSION_get0_cipher(clntsess); + SSL_shutdown(clientssl); + SSL_shutdown(serverssl); + SSL_free(serverssl); + SSL_free(clientssl); + serverssl = clientssl = NULL; + + /* Check we can resume a session with a different SHA-256 ciphersuite */ + if (!TEST_true(SSL_CTX_set_cipher_list(cctx, + "TLS13-CHACHA20-POLY1305-SHA256")) + || !TEST_true(create_ssl_objects(sctx, cctx, &serverssl, &clientssl, + NULL, NULL)) + || !TEST_true(SSL_set_session(clientssl, clntsess)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE)) + || !TEST_true(SSL_session_reused(clientssl))) goto end; - } - /* Continue writing the message we started earlier */ - if (!SSL_write_ex(clientssl, MSG1, strlen(MSG1), &written) - || written != strlen(MSG1)) { - printf("Failed writing message 1\n"); + SSL_SESSION_free(clntsess); + clntsess = SSL_get1_session(clientssl); + SSL_shutdown(clientssl); + SSL_shutdown(serverssl); + SSL_free(serverssl); + SSL_free(clientssl); + serverssl = clientssl = NULL; + + /* + * Check attempting to resume a SHA-256 session with no SHA-256 ciphersuites + * succeeds but does not resume. + */ + if (!TEST_true(SSL_CTX_set_cipher_list(cctx, "TLS13-AES-256-GCM-SHA384")) + || !TEST_true(create_ssl_objects(sctx, cctx, &serverssl, &clientssl, + NULL, NULL)) + || !TEST_true(SSL_set_session(clientssl, clntsess)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_SSL)) + || !TEST_false(SSL_session_reused(clientssl))) goto end; - } - if (SSL_get_early_data_status(clientssl) != SSL_EARLY_DATA_NOT_SENT) { - printf("Unexpected early data status (2)\n"); + SSL_SESSION_free(clntsess); + clntsess = NULL; + SSL_shutdown(clientssl); + SSL_shutdown(serverssl); + SSL_free(serverssl); + SSL_free(clientssl); + serverssl = clientssl = NULL; + + /* Create a session based on SHA384 */ + if (!TEST_true(SSL_CTX_set_cipher_list(cctx, "TLS13-AES-256-GCM-SHA384")) + || !TEST_true(create_ssl_objects(sctx, cctx, &serverssl, + &clientssl, NULL, NULL)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE))) goto end; - } - if (!SSL_read_ex(serverssl, buf, sizeof(buf), &readbytes) - || readbytes != strlen(MSG1) - || memcmp(MSG1, buf, strlen(MSG1))) { - printf("Failed reading message 1\n"); + clntsess = SSL_get1_session(clientssl); + SSL_shutdown(clientssl); + SSL_shutdown(serverssl); + SSL_free(serverssl); + SSL_free(clientssl); + serverssl = clientssl = NULL; + + if (!TEST_true(SSL_CTX_set_cipher_list(cctx, + "TLS13-AES-128-GCM-SHA256:TLS13-AES-256-GCM-SHA384")) + || !TEST_true(SSL_CTX_set_cipher_list(sctx, + "TLS13-AES-256-GCM-SHA384")) + || !TEST_true(create_ssl_objects(sctx, cctx, &serverssl, &clientssl, + NULL, NULL)) + || !TEST_true(SSL_set_session(clientssl, clntsess)) + /* + * We use SSL_ERROR_WANT_READ below so that we can pause the + * connection after the initial ClientHello has been sent to + * enable us to make some session changes. + */ + || !TEST_false(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_WANT_READ))) + goto end; + + /* Trick the client into thinking this session is for a different digest */ + clntsess->cipher = aes_128_gcm_sha256; + clntsess->cipher_id = clntsess->cipher->id; + + /* + * Continue the previously started connection. Server has selected a SHA-384 + * ciphersuite, but client thinks the session is for SHA-256, so it should + * bail out. + */ + if (!TEST_false(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_SSL)) + || !TEST_int_eq(ERR_GET_REASON(ERR_get_error()), + SSL_R_CIPHERSUITE_DIGEST_HAS_CHANGED)) goto end; + + testresult = 1; + + end: + SSL_SESSION_free(clntsess); + SSL_free(serverssl); + SSL_free(clientssl); + SSL_CTX_free(sctx); + SSL_CTX_free(cctx); + + return testresult; +} + + +static SSL_SESSION *psk = NULL; +static const char *pskid = "Identity"; +static const char *srvid; + +static int use_session_cb_cnt = 0; +static int find_session_cb_cnt = 0; + +static int use_session_cb(SSL *ssl, const EVP_MD *md, const unsigned char **id, + size_t *idlen, SSL_SESSION **sess) +{ + switch (++use_session_cb_cnt) { + case 1: + /* The first call should always have a NULL md */ + if (md != NULL) + return 0; + break; + + case 2: + /* The second call should always have an md */ + if (md == NULL) + return 0; + break; + + default: + /* We should only be called a maximum of twice */ + return 0; } - if (!SSL_write_ex(serverssl, MSG2, strlen(MSG2), &written) - || written != strlen(MSG2)) { - printf("Failed writing message 2\n"); - goto end; + if (psk != NULL) + SSL_SESSION_up_ref(psk); + + *sess = psk; + *id = (const unsigned char *)pskid; + *idlen = strlen(pskid); + + return 1; +} + +static int find_session_cb(SSL *ssl, const unsigned char *identity, + size_t identity_len, SSL_SESSION **sess) +{ + find_session_cb_cnt++; + + /* We should only ever be called a maximum of twice per connection */ + if (find_session_cb_cnt > 2) + return 0; + + if (psk == NULL) + return 0; + + /* Identity should match that set by the client */ + if (strlen(srvid) != identity_len + || strncmp(srvid, (const char *)identity, identity_len) != 0) { + /* No PSK found, continue but without a PSK */ + *sess = NULL; + return 1; } - if (!SSL_read_ex(clientssl, buf, sizeof(buf), &readbytes) - || readbytes != strlen(MSG2) - || memcmp(MSG2, buf, strlen(MSG2))) { - printf("Failed reading message 2\n"); + SSL_SESSION_up_ref(psk); + *sess = psk; + + return 1; +} + +#define TLS13_AES_256_GCM_SHA384_BYTES ((const unsigned char *)"\x13\x02") + +static int test_tls13_psk(void) +{ + SSL_CTX *sctx = NULL, *cctx = NULL; + SSL *serverssl = NULL, *clientssl = NULL; + const SSL_CIPHER *cipher = NULL; + const unsigned char key[] = { + 0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08, 0x09, 0x0a, 0x0b, + 0x0c, 0x0d, 0x0e, 0x0f, 0x10, 0x11, 0x12, 0x13, 0x14, 0x15, 0x16, 0x17, + 0x18, 0x19, 0x1a, 0x1b, 0x1c, 0x1d, 0x1e, 0x1f + }; + int testresult = 0; + + if (!TEST_true(create_ssl_ctx_pair(TLS_server_method(), + TLS_client_method(), &sctx, + &cctx, cert, privkey))) goto end; - } + SSL_CTX_set_psk_use_session_callback(cctx, use_session_cb); + SSL_CTX_set_psk_find_session_callback(sctx, find_session_cb); + srvid = pskid; + + /* Check we can create a connection if callback decides not to send a PSK */ + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, &clientssl, + NULL, NULL)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE)) + || !TEST_false(SSL_session_reused(clientssl)) + || !TEST_false(SSL_session_reused(serverssl)) + || !TEST_true(use_session_cb_cnt == 1) + || !TEST_true(find_session_cb_cnt == 0)) + goto end; + + shutdown_ssl_connection(serverssl, clientssl); + serverssl = clientssl = NULL; + use_session_cb_cnt = 0; + + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, &clientssl, + NULL, NULL))) + goto end; + + /* Create the PSK */ + cipher = SSL_CIPHER_find(clientssl, TLS13_AES_256_GCM_SHA384_BYTES); + psk = SSL_SESSION_new(); + if (!TEST_ptr(psk) + || !TEST_ptr(cipher) + || !TEST_true(SSL_SESSION_set1_master_key(psk, key, sizeof(key))) + || !TEST_true(SSL_SESSION_set_cipher(psk, cipher)) + || !TEST_true(SSL_SESSION_set_protocol_version(psk, + TLS1_3_VERSION))) + goto end; + + /* Check we can create a connection and the PSK is used */ + if (!TEST_true(create_ssl_connection(serverssl, clientssl, SSL_ERROR_NONE)) + || !TEST_true(SSL_session_reused(clientssl)) + || !TEST_true(SSL_session_reused(serverssl)) + || !TEST_true(use_session_cb_cnt == 1) + || !TEST_true(find_session_cb_cnt == 1)) + goto end; + + shutdown_ssl_connection(serverssl, clientssl); + serverssl = clientssl = NULL; + use_session_cb_cnt = find_session_cb_cnt = 0; + + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, &clientssl, + NULL, NULL))) + goto end; + + /* Force an HRR */ + if (!TEST_true(SSL_set1_groups_list(serverssl, "P-256"))) + goto end; + + /* + * Check we can create a connection, the PSK is used and the callbacks are + * called twice. + */ + if (!TEST_true(create_ssl_connection(serverssl, clientssl, SSL_ERROR_NONE)) + || !TEST_true(SSL_session_reused(clientssl)) + || !TEST_true(SSL_session_reused(serverssl)) + || !TEST_true(use_session_cb_cnt == 2) + || !TEST_true(find_session_cb_cnt == 2)) + goto end; + + shutdown_ssl_connection(serverssl, clientssl); + serverssl = clientssl = NULL; + use_session_cb_cnt = find_session_cb_cnt = 0; + + /* + * Check that if the server rejects the PSK we can still connect, but with + * a full handshake + */ + srvid = "Dummy Identity"; + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, &clientssl, + NULL, NULL)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE)) + || !TEST_false(SSL_session_reused(clientssl)) + || !TEST_false(SSL_session_reused(serverssl)) + || !TEST_true(use_session_cb_cnt == 1) + || !TEST_true(find_session_cb_cnt == 1)) + goto end; + + shutdown_ssl_connection(serverssl, clientssl); + serverssl = clientssl = NULL; testresult = 1; end: - if(!testresult) - ERR_print_errors_fp(stdout); + SSL_SESSION_free(psk); SSL_free(serverssl); SSL_free(clientssl); SSL_CTX_free(sctx); SSL_CTX_free(cctx); - return testresult; } -# endif -#endif + +#endif /* OPENSSL_NO_TLS1_3 */ static int clntaddoldcb = 0; static int clntparseoldcb = 0; @@ -2258,6 +2118,7 @@ static int clntaddnewcb = 0; static int clntparsenewcb = 0; static int srvaddnewcb = 0; static int srvparsenewcb = 0; +static int snicb = 0; #define TEST_EXT_TYPE1 0xff00 @@ -2272,17 +2133,13 @@ static int old_add_cb(SSL *s, unsigned int ext_type, const unsigned char **out, else clntaddoldcb++; - if (*server != SSL_is_server(s)) - return -1; - - data = OPENSSL_malloc(sizeof(char)); - if (data == NULL) + if (*server != SSL_is_server(s) + || (data = OPENSSL_malloc(sizeof(*data))) == NULL) return -1; *data = 1; *out = data; *outlen = sizeof(char); - return 1; } @@ -2302,10 +2159,9 @@ static int old_parse_cb(SSL *s, unsigned int ext_type, const unsigned char *in, else clntparseoldcb++; - if (*server != SSL_is_server(s)) - return -1; - - if (inlen != sizeof(char) || *in != 1) + if (*server != SSL_is_server(s) + || inlen != sizeof(char) + || *in != 1) return -1; return 1; @@ -2323,17 +2179,13 @@ static int new_add_cb(SSL *s, unsigned int ext_type, unsigned int context, else clntaddnewcb++; - if (*server != SSL_is_server(s)) - return -1; - - data = OPENSSL_malloc(sizeof(char)); - if (data == NULL) + if (*server != SSL_is_server(s) + || (data = OPENSSL_malloc(sizeof(*data))) == NULL) return -1; *data = 1; *out = data; - *outlen = sizeof(char); - + *outlen = sizeof(*data); return 1; } @@ -2354,23 +2206,36 @@ static int new_parse_cb(SSL *s, unsigned int ext_type, unsigned int context, else clntparsenewcb++; - if (*server != SSL_is_server(s)) - return -1; - - if (inlen != sizeof(char) || *in != 1) + if (*server != SSL_is_server(s) + || inlen != sizeof(char) || *in != 1) return -1; return 1; } + +static int sni_cb(SSL *s, int *al, void *arg) +{ + SSL_CTX *ctx = (SSL_CTX *)arg; + + if (SSL_set_SSL_CTX(s, ctx) == NULL) { + *al = SSL_AD_INTERNAL_ERROR; + return SSL_TLSEXT_ERR_ALERT_FATAL; + } + snicb++; + return SSL_TLSEXT_ERR_OK; +} + /* * Custom call back tests. * Test 0: Old style callbacks in TLSv1.2 * Test 1: New style callbacks in TLSv1.2 - * Test 2: New style callbacks in TLSv1.3. Extensions in CH and EE - * Test 3: New style callbacks in TLSv1.3. Extensions in CH, SH, EE, Cert + NST + * Test 2: New style callbacks in TLSv1.2 with SNI + * Test 3: New style callbacks in TLSv1.3. Extensions in CH and EE + * Test 4: New style callbacks in TLSv1.3. Extensions in CH, SH, EE, Cert + NST */ -static int test_custom_exts(int tst) { - SSL_CTX *cctx = NULL, *sctx = NULL; +static int test_custom_exts(int tst) +{ + SSL_CTX *cctx = NULL, *sctx = NULL, *sctx2 = NULL; SSL *clientssl = NULL, *serverssl = NULL; int testresult = 0; static int server = 1; @@ -2381,142 +2246,149 @@ static int test_custom_exts(int tst) { /* Reset callback counters */ clntaddoldcb = clntparseoldcb = srvaddoldcb = srvparseoldcb = 0; clntaddnewcb = clntparsenewcb = srvaddnewcb = srvparsenewcb = 0; + snicb = 0; + + if (!TEST_true(create_ssl_ctx_pair(TLS_server_method(), + TLS_client_method(), &sctx, + &cctx, cert, privkey))) + goto end; + + if (tst == 2 + && !TEST_true(create_ssl_ctx_pair(TLS_server_method(), NULL, &sctx2, + NULL, cert, privkey))) + goto end; - if (!create_ssl_ctx_pair(TLS_server_method(), TLS_client_method(), &sctx, - &cctx, cert, privkey)) { - printf("Unable to create SSL_CTX pair\n"); - return 0; - } - if (tst < 2) { + if (tst < 3) { SSL_CTX_set_options(cctx, SSL_OP_NO_TLSv1_3); SSL_CTX_set_options(sctx, SSL_OP_NO_TLSv1_3); + if (sctx2 != NULL) + SSL_CTX_set_options(sctx2, SSL_OP_NO_TLSv1_3); } - if (tst == 3) { - context = SSL_EXT_CLIENT_HELLO | SSL_EXT_TLS1_2_SERVER_HELLO + if (tst == 4) { + context = SSL_EXT_CLIENT_HELLO + | SSL_EXT_TLS1_2_SERVER_HELLO | SSL_EXT_TLS1_3_SERVER_HELLO | SSL_EXT_TLS1_3_ENCRYPTED_EXTENSIONS | SSL_EXT_TLS1_3_CERTIFICATE | SSL_EXT_TLS1_3_NEW_SESSION_TICKET; } else { - context = SSL_EXT_CLIENT_HELLO | SSL_EXT_TLS1_2_SERVER_HELLO + context = SSL_EXT_CLIENT_HELLO + | SSL_EXT_TLS1_2_SERVER_HELLO | SSL_EXT_TLS1_3_ENCRYPTED_EXTENSIONS; } /* Create a client side custom extension */ if (tst == 0) { - if (!SSL_CTX_add_client_custom_ext(cctx, TEST_EXT_TYPE1, old_add_cb, - old_free_cb, &client, old_parse_cb, - &client)) { - printf("Unable to create old style client side custom extension\n"); - return 0; - } + if (!TEST_true(SSL_CTX_add_client_custom_ext(cctx, TEST_EXT_TYPE1, + old_add_cb, old_free_cb, + &client, old_parse_cb, + &client))) + goto end; } else { - if (!SSL_CTX_add_custom_ext(cctx, TEST_EXT_TYPE1, context, new_add_cb, - new_free_cb, &client, new_parse_cb, - &client)) { - printf("Unable to create new style client side custom extension\n"); - return 0; - } + if (!TEST_true(SSL_CTX_add_custom_ext(cctx, TEST_EXT_TYPE1, context, + new_add_cb, new_free_cb, + &client, new_parse_cb, &client))) + goto end; } /* Should not be able to add duplicates */ - if (SSL_CTX_add_client_custom_ext(cctx, TEST_EXT_TYPE1, old_add_cb, - old_free_cb, &client, old_parse_cb, - &client)) { - printf("Unexpected success adding duplicate client custom extension\n"); - return 0; - } - if (SSL_CTX_add_custom_ext(cctx, TEST_EXT_TYPE1, context, new_add_cb, - new_free_cb, &client, new_parse_cb, &client)) { - printf("Unexpected success adding duplicate client custom extension\n"); - return 0; - } + if (!TEST_false(SSL_CTX_add_client_custom_ext(cctx, TEST_EXT_TYPE1, + old_add_cb, old_free_cb, + &client, old_parse_cb, + &client)) + || !TEST_false(SSL_CTX_add_custom_ext(cctx, TEST_EXT_TYPE1, + context, new_add_cb, + new_free_cb, &client, + new_parse_cb, &client))) + goto end; /* Create a server side custom extension */ if (tst == 0) { - if (!SSL_CTX_add_server_custom_ext(sctx, TEST_EXT_TYPE1, old_add_cb, - old_free_cb, &server, old_parse_cb, - &server)) { - printf("Unable to create old style server side custom extension\n"); - return 0; - } + if (!TEST_true(SSL_CTX_add_server_custom_ext(sctx, TEST_EXT_TYPE1, + old_add_cb, old_free_cb, + &server, old_parse_cb, + &server))) + goto end; } else { - if (!SSL_CTX_add_custom_ext(sctx, TEST_EXT_TYPE1, context, new_add_cb, - new_free_cb, &server, new_parse_cb, - &server)) { - printf("Unable to create new style server side custom extension\n"); - return 0; - } + if (!TEST_true(SSL_CTX_add_custom_ext(sctx, TEST_EXT_TYPE1, context, + new_add_cb, new_free_cb, + &server, new_parse_cb, &server))) + goto end; + if (sctx2 != NULL + && !TEST_true(SSL_CTX_add_custom_ext(sctx2, TEST_EXT_TYPE1, + context, new_add_cb, + new_free_cb, &server, + new_parse_cb, &server))) + goto end; } /* Should not be able to add duplicates */ - if (SSL_CTX_add_server_custom_ext(sctx, TEST_EXT_TYPE1, old_add_cb, - old_free_cb, &server, old_parse_cb, - &server)) { - printf("Unexpected success adding duplicate server custom extension\n"); - return 0; - } - if (SSL_CTX_add_custom_ext(sctx, TEST_EXT_TYPE1, context, new_add_cb, - new_free_cb, &server, new_parse_cb, &server)) { - printf("Unexpected success adding duplicate server custom extension\n"); - return 0; - } - - if (!create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, NULL)) { - printf("Unable to create SSL objects\n"); - goto end; + if (!TEST_false(SSL_CTX_add_server_custom_ext(sctx, TEST_EXT_TYPE1, + old_add_cb, old_free_cb, + &server, old_parse_cb, + &server)) + || !TEST_false(SSL_CTX_add_custom_ext(sctx, TEST_EXT_TYPE1, + context, new_add_cb, + new_free_cb, &server, + new_parse_cb, &server))) + goto end; + + if (tst == 2) { + /* Set up SNI */ + if (!TEST_true(SSL_CTX_set_tlsext_servername_callback(sctx, sni_cb)) + || !TEST_true(SSL_CTX_set_tlsext_servername_arg(sctx, sctx2))) + goto end; } - if (!create_ssl_connection(serverssl, clientssl, SSL_ERROR_NONE)) { - printf("Unable to create SSL connection\n"); + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, + &clientssl, NULL, NULL)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE))) goto end; - } if (tst == 0) { - if (clntaddoldcb != 1 || clntparseoldcb != 1 || srvaddoldcb != 1 - || srvparseoldcb != 1) { - printf("Custom extension callbacks not called\n"); + if (clntaddoldcb != 1 + || clntparseoldcb != 1 + || srvaddoldcb != 1 + || srvparseoldcb != 1) goto end; - } - } else if (tst == 1 || tst == 2) { - if (clntaddnewcb != 1 || clntparsenewcb != 1 || srvaddnewcb != 1 - || srvparsenewcb != 1) { - printf("Custom extension callbacks not called\n"); + } else if (tst == 1 || tst == 2 || tst == 3) { + if (clntaddnewcb != 1 + || clntparsenewcb != 1 + || srvaddnewcb != 1 + || srvparsenewcb != 1 + || (tst != 2 && snicb != 0) + || (tst == 2 && snicb != 1)) goto end; - } } else { - if (clntaddnewcb != 1 || clntparsenewcb != 4 || srvaddnewcb != 4 - || srvparsenewcb != 1) { - printf("Custom extension callbacks not called\n"); + if (clntaddnewcb != 1 + || clntparsenewcb != 4 + || srvaddnewcb != 4 + || srvparsenewcb != 1) goto end; - } } sess = SSL_get1_session(clientssl); - SSL_shutdown(clientssl); SSL_shutdown(serverssl); - SSL_free(serverssl); SSL_free(clientssl); serverssl = clientssl = NULL; - if (!create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, NULL)) { - printf("Unable to create SSL objects (2)\n"); - goto end; - } - - if (!SSL_set_session(clientssl, sess)) { - printf("Failed setting session\n"); + if (tst == 3) { + /* We don't bother with the resumption aspects for this test */ + testresult = 1; goto end; } - if (!create_ssl_connection(serverssl, clientssl, SSL_ERROR_NONE)) { - printf("Unable to create SSL connection (2)\n"); + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, &clientssl, + NULL, NULL)) + || !TEST_true(SSL_set_session(clientssl, sess)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE))) goto end; - } /* * For a resumed session we expect to add the ClientHello extension. For the @@ -2525,24 +2397,24 @@ static int test_custom_exts(int tst) { * them. */ if (tst == 0) { - if (clntaddoldcb != 2 || clntparseoldcb != 1 || srvaddoldcb != 1 - || srvparseoldcb != 1) { - printf("Unexpected custom extension callback calls\n"); + if (clntaddoldcb != 2 + || clntparseoldcb != 1 + || srvaddoldcb != 1 + || srvparseoldcb != 1) goto end; - } - } else if (tst == 1 || tst == 2) { - if (clntaddnewcb != 2 || clntparsenewcb != 2 || srvaddnewcb != 2 - || srvparsenewcb != 2) { - printf("Unexpected custom extension callback calls\n"); + } else if (tst == 1 || tst == 2 || tst == 3) { + if (clntaddnewcb != 2 + || clntparsenewcb != 2 + || srvaddnewcb != 2 + || srvparsenewcb != 2) goto end; - } } else { /* No Certificate message extensions in the resumption handshake */ - if (clntaddnewcb != 2 || clntparsenewcb != 7 || srvaddnewcb != 7 - || srvparsenewcb != 2) { - printf("Unexpected custom extension callback calls\n"); + if (clntaddnewcb != 2 + || clntparsenewcb != 7 + || srvaddnewcb != 7 + || srvparsenewcb != 2) goto end; - } } testresult = 1; @@ -2551,6 +2423,189 @@ end: SSL_SESSION_free(sess); SSL_free(serverssl); SSL_free(clientssl); + SSL_CTX_free(sctx2); + SSL_CTX_free(sctx); + SSL_CTX_free(cctx); + return testresult; +} + +/* + * Test loading of serverinfo data in various formats. test_sslmessages actually + * tests to make sure the extensions appear in the handshake + */ +static int test_serverinfo(int tst) +{ + unsigned int version; + unsigned char *sibuf; + size_t sibuflen; + int ret, expected, testresult = 0; + SSL_CTX *ctx; + + ctx = SSL_CTX_new(TLS_method()); + if (!TEST_ptr(ctx)) + goto end; + + if ((tst & 0x01) == 0x01) + version = SSL_SERVERINFOV2; + else + version = SSL_SERVERINFOV1; + + if ((tst & 0x02) == 0x02) { + sibuf = serverinfov2; + sibuflen = sizeof(serverinfov2); + expected = (version == SSL_SERVERINFOV2); + } else { + sibuf = serverinfov1; + sibuflen = sizeof(serverinfov1); + expected = (version == SSL_SERVERINFOV1); + } + + if ((tst & 0x04) == 0x04) { + ret = SSL_CTX_use_serverinfo_ex(ctx, version, sibuf, sibuflen); + } else { + ret = SSL_CTX_use_serverinfo(ctx, sibuf, sibuflen); + + /* + * The version variable is irrelevant in this case - it's what is in the + * buffer that matters + */ + if ((tst & 0x02) == 0x02) + expected = 0; + else + expected = 1; + } + + if (!TEST_true(ret == expected)) + goto end; + + testresult = 1; + + end: + SSL_CTX_free(ctx); + + return testresult; +} + +/* + * Test that SSL_export_keying_material() produces expected results. There are + * no test vectors so all we do is test that both sides of the communication + * produce the same results for different protocol versions. + */ +static int test_export_key_mat(int tst) +{ + int testresult = 0; + SSL_CTX *cctx = NULL, *sctx = NULL, *sctx2 = NULL; + SSL *clientssl = NULL, *serverssl = NULL; + const char label[] = "test label"; + const unsigned char context[] = "context"; + const unsigned char *emptycontext = NULL; + unsigned char ckeymat1[80], ckeymat2[80], ckeymat3[80]; + unsigned char skeymat1[80], skeymat2[80], skeymat3[80]; + const int protocols[] = { + TLS1_VERSION, + TLS1_1_VERSION, + TLS1_2_VERSION, + TLS1_3_VERSION + }; + +#ifdef OPENSSL_NO_TLS1 + if (tst == 0) + return 1; +#endif +#ifdef OPENSSL_NO_TLS1_1 + if (tst == 1) + return 1; +#endif +#ifdef OPENSSL_NO_TLS1_2 + if (tst == 2) + return 1; +#endif +#ifdef OPENSSL_NO_TLS1_3 + if (tst == 3) + return 1; +#endif + if (!TEST_true(create_ssl_ctx_pair(TLS_server_method(), + TLS_client_method(), &sctx, + &cctx, cert, privkey))) + goto end; + + OPENSSL_assert(tst >= 0 && (size_t)tst < OSSL_NELEM(protocols)); + SSL_CTX_set_max_proto_version(cctx, protocols[tst]); + SSL_CTX_set_min_proto_version(cctx, protocols[tst]); + + if (!TEST_true(create_ssl_objects(sctx, cctx, &serverssl, &clientssl, NULL, + NULL)) + || !TEST_true(create_ssl_connection(serverssl, clientssl, + SSL_ERROR_NONE))) + goto end; + + if (!TEST_int_eq(SSL_export_keying_material(clientssl, ckeymat1, + sizeof(ckeymat1), label, + sizeof(label) - 1, context, + sizeof(context) - 1, 1), 1) + || !TEST_int_eq(SSL_export_keying_material(clientssl, ckeymat2, + sizeof(ckeymat2), label, + sizeof(label) - 1, + emptycontext, + 0, 1), 1) + || !TEST_int_eq(SSL_export_keying_material(clientssl, ckeymat3, + sizeof(ckeymat3), label, + sizeof(label) - 1, + NULL, 0, 0), 1) + || !TEST_int_eq(SSL_export_keying_material(serverssl, skeymat1, + sizeof(skeymat1), label, + sizeof(label) - 1, + context, + sizeof(context) -1, 1), + 1) + || !TEST_int_eq(SSL_export_keying_material(serverssl, skeymat2, + sizeof(skeymat2), label, + sizeof(label) - 1, + emptycontext, + 0, 1), 1) + || !TEST_int_eq(SSL_export_keying_material(serverssl, skeymat3, + sizeof(skeymat3), label, + sizeof(label) - 1, + NULL, 0, 0), 1) + /* + * Check that both sides created the same key material with the + * same context. + */ + || !TEST_mem_eq(ckeymat1, sizeof(ckeymat1), skeymat1, + sizeof(skeymat1)) + /* + * Check that both sides created the same key material with an + * empty context. + */ + || !TEST_mem_eq(ckeymat2, sizeof(ckeymat2), skeymat2, + sizeof(skeymat2)) + /* + * Check that both sides created the same key material without a + * context. + */ + || !TEST_mem_eq(ckeymat3, sizeof(ckeymat3), skeymat3, + sizeof(skeymat3)) + /* Different contexts should produce different results */ + || !TEST_mem_ne(ckeymat1, sizeof(ckeymat1), ckeymat2, + sizeof(ckeymat2))) + goto end; + + /* + * Check that an empty context and no context produce different results in + * protocols less than TLSv1.3. In TLSv1.3 they should be the same. + */ + if ((tst != 3 && !TEST_mem_ne(ckeymat2, sizeof(ckeymat2), ckeymat3, + sizeof(ckeymat3))) + || (tst ==3 && !TEST_mem_eq(ckeymat2, sizeof(ckeymat2), ckeymat3, + sizeof(ckeymat3)))) + goto end; + + testresult = 1; + + end: + SSL_free(serverssl); + SSL_free(clientssl); + SSL_CTX_free(sctx2); SSL_CTX_free(sctx); SSL_CTX_free(cctx); @@ -2562,8 +2617,8 @@ int test_main(int argc, char *argv[]) int testresult = 1; if (argc != 3) { - printf("Invalid argument count\n"); - return 1; + TEST_error("Wrong argument count"); + return 0; } cert = argv[1]; @@ -2602,7 +2657,15 @@ int test_main(int argc, char *argv[]) ADD_ALL_TESTS(test_early_data_tls1_2, 2); # endif #endif - ADD_ALL_TESTS(test_custom_exts, 4); +#ifndef OPENSSL_NO_TLS1_3 + ADD_TEST(test_ciphersuite_change); + ADD_TEST(test_tls13_psk); + ADD_ALL_TESTS(test_custom_exts, 5); +#else + ADD_ALL_TESTS(test_custom_exts, 3); +#endif + ADD_ALL_TESTS(test_serverinfo, 8); + ADD_ALL_TESTS(test_export_key_mat, 4); testresult = run_tests(argv[0]);