Remove some flags that are unused in the shim
authorMatt Caswell <matt@openssl.org>
Wed, 19 Oct 2016 10:22:07 +0000 (11:22 +0100)
committerMatt Caswell <matt@openssl.org>
Fri, 4 Nov 2016 10:38:54 +0000 (10:38 +0000)
Reviewed-by: Richard Levitte <levitte@openssl.org>
test/ossl_shim/ossl_shim.cc
test/ossl_shim/test_config.cc
test/ossl_shim/test_config.h

index 99733f6065cf2b4d983e40d712e616aeec14ef3c..fc2180ef5e8e0b998092da86f1c5e77662a94fcf 100644 (file)
@@ -340,7 +340,7 @@ static int CertCallback(SSL *ssl, void *arg) {
   }
 
   // The certificate will be installed via other means.
   }
 
   // The certificate will be installed via other means.
-  if (!config->async || config->use_early_callback ||
+  if (!config->async ||
       config->use_old_client_cert_callback) {
     return 1;
   }
       config->use_old_client_cert_callback) {
     return 1;
   }
@@ -890,8 +890,7 @@ static bool DoExchange(bssl::UniquePtr<SSL_SESSION> *out_session,
     return false;
   }
   // Install the certificate synchronously if nothing else will handle it.
     return false;
   }
   // Install the certificate synchronously if nothing else will handle it.
-  if (!config->use_early_callback &&
-      !config->use_old_client_cert_callback &&
+  if (!config->use_old_client_cert_callback &&
       !config->async &&
       !InstallCertificate(ssl.get())) {
     return false;
       !config->async &&
       !InstallCertificate(ssl.get())) {
     return false;
index f7ae8e67a7d446d7e91b486a244f194b9fd31c12..6c18f825ac437660702bd7448fcf958eb0b379ae 100644 (file)
@@ -52,7 +52,6 @@ const Flag<bool> kBoolFlags[] = {
   { "-async", &TestConfig::async },
   { "-write-different-record-sizes",
     &TestConfig::write_different_record_sizes },
   { "-async", &TestConfig::async },
   { "-write-different-record-sizes",
     &TestConfig::write_different_record_sizes },
-  { "-cbc-record-splitting", &TestConfig::cbc_record_splitting },
   { "-partial-write", &TestConfig::partial_write },
   { "-no-tls13", &TestConfig::no_tls13 },
   { "-no-tls12", &TestConfig::no_tls12 },
   { "-partial-write", &TestConfig::partial_write },
   { "-no-tls13", &TestConfig::no_tls13 },
   { "-no-tls12", &TestConfig::no_tls12 },
@@ -65,10 +64,6 @@ const Flag<bool> kBoolFlags[] = {
   { "-expect-extended-master-secret",
     &TestConfig::expect_extended_master_secret },
   { "-implicit-handshake", &TestConfig::implicit_handshake },
   { "-expect-extended-master-secret",
     &TestConfig::expect_extended_master_secret },
   { "-implicit-handshake", &TestConfig::implicit_handshake },
-  { "-use-early-callback", &TestConfig::use_early_callback },
-  { "-fail-early-callback", &TestConfig::fail_early_callback },
-  { "-fail-ddos-callback", &TestConfig::fail_ddos_callback },
-  { "-fail-second-ddos-callback", &TestConfig::fail_second_ddos_callback },
   { "-handshake-never-done", &TestConfig::handshake_never_done },
   { "-use-export-context", &TestConfig::use_export_context },
   { "-expect-ticket-renewal", &TestConfig::expect_ticket_renewal },
   { "-handshake-never-done", &TestConfig::handshake_never_done },
   { "-use-export-context", &TestConfig::use_export_context },
   { "-expect-ticket-renewal", &TestConfig::expect_ticket_renewal },
@@ -94,7 +89,6 @@ const Flag<bool> kBoolFlags[] = {
     &TestConfig::use_old_client_cert_callback },
   { "-use-null-client-ca-list", &TestConfig::use_null_client_ca_list },
   { "-peek-then-read", &TestConfig::peek_then_read },
     &TestConfig::use_old_client_cert_callback },
   { "-use-null-client-ca-list", &TestConfig::use_null_client_ca_list },
   { "-peek-then-read", &TestConfig::peek_then_read },
-  { "-enable-grease", &TestConfig::enable_grease },
 };
 
 const Flag<std::string> kStringFlags[] = {
 };
 
 const Flag<std::string> kStringFlags[] = {
@@ -119,9 +113,6 @@ const Flag<std::string> kStringFlags[] = {
 
 const Flag<std::string> kBase64Flags[] = {
   { "-expect-certificate-types", &TestConfig::expected_certificate_types },
 
 const Flag<std::string> kBase64Flags[] = {
   { "-expect-certificate-types", &TestConfig::expected_certificate_types },
-  { "-expect-ocsp-response", &TestConfig::expected_ocsp_response },
-  { "-expect-signed-cert-timestamps",
-    &TestConfig::expected_signed_cert_timestamps },
 };
 
 const Flag<int> kIntFlags[] = {
 };
 
 const Flag<int> kIntFlags[] = {
@@ -132,10 +123,6 @@ const Flag<int> kIntFlags[] = {
   { "-mtu", &TestConfig::mtu },
   { "-export-keying-material", &TestConfig::export_keying_material },
   { "-expect-total-renegotiations", &TestConfig::expect_total_renegotiations },
   { "-mtu", &TestConfig::mtu },
   { "-export-keying-material", &TestConfig::export_keying_material },
   { "-expect-total-renegotiations", &TestConfig::expect_total_renegotiations },
-  { "-expect-peer-signature-algorithm",
-    &TestConfig::expect_peer_signature_algorithm },
-  { "-expect-curve-id", &TestConfig::expect_curve_id },
-  { "-expect-dhe-group-size", &TestConfig::expect_dhe_group_size },
   { "-max-cert-list", &TestConfig::max_cert_list },
 };
 
   { "-max-cert-list", &TestConfig::max_cert_list },
 };
 
index eecf171a73f19354327182dde45de693f8c947e5..3f18c11fb383ae3c4e14fa60cf9a89057f1d3cb8 100644 (file)
@@ -35,7 +35,6 @@ struct TestConfig {
   std::string select_next_proto;
   bool async = false;
   bool write_different_record_sizes = false;
   std::string select_next_proto;
   bool async = false;
   bool write_different_record_sizes = false;
-  bool cbc_record_splitting = false;
   bool partial_write = false;
   bool no_tls13 = false;
   bool no_tls12 = false;
   bool partial_write = false;
   bool no_tls13 = false;
   bool no_tls12 = false;
@@ -54,16 +53,10 @@ struct TestConfig {
   std::string psk;
   std::string psk_identity;
   std::string srtp_profiles;
   std::string psk;
   std::string psk_identity;
   std::string srtp_profiles;
-  std::string expected_ocsp_response;
-  std::string expected_signed_cert_timestamps;
   int min_version = 0;
   int max_version = 0;
   int mtu = 0;
   bool implicit_handshake = false;
   int min_version = 0;
   int max_version = 0;
   int mtu = 0;
   bool implicit_handshake = false;
-  bool use_early_callback = false;
-  bool fail_early_callback = false;
-  bool fail_ddos_callback = false;
-  bool fail_second_ddos_callback = false;
   std::string cipher;
   bool handshake_never_done = false;
   int export_keying_material = 0;
   std::string cipher;
   bool handshake_never_done = false;
   int export_keying_material = 0;
@@ -85,16 +78,12 @@ struct TestConfig {
   bool expect_verify_result = false;
   int expect_total_renegotiations = 0;
   bool renegotiate_freely = false;
   bool expect_verify_result = false;
   int expect_total_renegotiations = 0;
   bool renegotiate_freely = false;
-  int expect_peer_signature_algorithm = 0;
   bool p384_only = false;
   bool enable_all_curves = false;
   bool use_sparse_dh_prime = false;
   bool p384_only = false;
   bool enable_all_curves = false;
   bool use_sparse_dh_prime = false;
-  int expect_curve_id = 0;
-  int expect_dhe_group_size = 0;
   bool use_old_client_cert_callback = false;
   bool use_null_client_ca_list = false;
   bool peek_then_read = false;
   bool use_old_client_cert_callback = false;
   bool use_null_client_ca_list = false;
   bool peek_then_read = false;
-  bool enable_grease = false;
   int max_cert_list = 0;
 };
 
   int max_cert_list = 0;
 };