diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/tls/c_hello.cpp | 29 | ||||
-rw-r--r-- | src/tls/c_kex.cpp | 3 | ||||
-rw-r--r-- | src/tls/cert_req.cpp | 5 | ||||
-rw-r--r-- | src/tls/cert_ver.cpp | 3 | ||||
-rw-r--r-- | src/tls/finished.cpp | 3 | ||||
-rw-r--r-- | src/tls/next_protocol.cpp | 3 | ||||
-rw-r--r-- | src/tls/rec_wri.cpp | 20 | ||||
-rw-r--r-- | src/tls/s_hello.cpp | 6 | ||||
-rw-r--r-- | src/tls/s_kex.cpp | 3 | ||||
-rw-r--r-- | src/tls/tls_messages.h | 6 | ||||
-rw-r--r-- | src/tls/tls_record.h | 2 |
11 files changed, 44 insertions, 39 deletions
diff --git a/src/tls/c_hello.cpp b/src/tls/c_hello.cpp index ecb6f43d6..59e0670df 100644 --- a/src/tls/c_hello.cpp +++ b/src/tls/c_hello.cpp @@ -27,34 +27,11 @@ MemoryVector<byte> make_hello_random(RandomNumberGenerator& rng) } /* -* Encode and send a Handshake message -*/ -void Handshake_Message::send(Record_Writer& writer, Handshake_Hash& hash) const - { - MemoryVector<byte> buf = serialize(); - MemoryVector<byte> send_buf(4); - - const size_t buf_size = buf.size(); - - send_buf[0] = type(); - - for(size_t i = 1; i != 4; ++i) - send_buf[i] = get_byte<u32bit>(i, buf_size); - - send_buf += buf; - - hash.update(send_buf); - - writer.send(HANDSHAKE, &send_buf[0], send_buf.size()); - } - -/* * Create a new Hello Request message */ Hello_Request::Hello_Request(Record_Writer& writer) { - Handshake_Hash dummy; // FIXME: *UGLY* - send(writer, dummy); + writer.send(*this); } /* @@ -105,7 +82,7 @@ Client_Hello::Client_Hello(Record_Writer& writer, for(size_t j = 0; j != sigs.size(); ++j) m_supported_algos.push_back(std::make_pair(hashes[i], sigs[j])); - send(writer, hash); + hash.update(writer.send(*this)); } /* @@ -130,7 +107,7 @@ Client_Hello::Client_Hello(Record_Writer& writer, // set m_supported_algos + m_supported_curves here? - send(writer, hash); + hash.update(writer.send(*this)); } Client_Hello::Client_Hello(const MemoryRegion<byte>& buf, Handshake_Type type) diff --git a/src/tls/c_kex.cpp b/src/tls/c_kex.cpp index 58e458e81..e0ebfb865 100644 --- a/src/tls/c_kex.cpp +++ b/src/tls/c_kex.cpp @@ -8,6 +8,7 @@ #include <botan/internal/tls_messages.h> #include <botan/internal/tls_reader.h> #include <botan/internal/tls_extensions.h> +#include <botan/tls_record.h> #include <botan/internal/assert.h> #include <botan/credentials_manager.h> #include <botan/pubkey.h> @@ -211,7 +212,7 @@ Client_Key_Exchange::Client_Key_Exchange(Record_Writer& writer, pub_key->algo_name()); } - send(writer, state->hash); + state->hash.update(writer.send(*this)); } /* diff --git a/src/tls/cert_req.cpp b/src/tls/cert_req.cpp index f400a36d2..063cc5902 100644 --- a/src/tls/cert_req.cpp +++ b/src/tls/cert_req.cpp @@ -8,6 +8,7 @@ #include <botan/internal/tls_messages.h> #include <botan/internal/tls_reader.h> #include <botan/internal/tls_extensions.h> +#include <botan/tls_record.h> #include <botan/der_enc.h> #include <botan/ber_dec.h> #include <botan/loadstor.h> @@ -74,7 +75,7 @@ Certificate_Req::Certificate_Req(Record_Writer& writer, m_supported_algos.push_back(std::make_pair(hashes[i], sigs[j])); } - send(writer, hash); + hash.update(writer.send(*this)); } /** @@ -176,7 +177,7 @@ Certificate::Certificate(Record_Writer& writer, const std::vector<X509_Certificate>& cert_list) { certs = cert_list; - send(writer, hash); + hash.update(writer.send(*this)); } /** diff --git a/src/tls/cert_ver.cpp b/src/tls/cert_ver.cpp index 791635b17..388e16c88 100644 --- a/src/tls/cert_ver.cpp +++ b/src/tls/cert_ver.cpp @@ -8,6 +8,7 @@ #include <botan/internal/tls_messages.h> #include <botan/internal/tls_reader.h> #include <botan/internal/tls_extensions.h> +#include <botan/tls_record.h> #include <botan/internal/assert.h> #include <memory> @@ -45,7 +46,7 @@ Certificate_Verify::Certificate_Verify(Record_Writer& writer, signature = signer.sign_message(state->hash.get_contents(), rng); } - send(writer, state->hash); + state->hash.update(writer.send(*this)); } /* diff --git a/src/tls/finished.cpp b/src/tls/finished.cpp index 18cc51b96..3c1c5684c 100644 --- a/src/tls/finished.cpp +++ b/src/tls/finished.cpp @@ -6,6 +6,7 @@ */ #include <botan/internal/tls_messages.h> +#include <botan/tls_record.h> #include <memory> namespace Botan { @@ -70,7 +71,7 @@ Finished::Finished(Record_Writer& writer, Connection_Side side) { verification_data = finished_compute_verify(state, side); - send(writer, state->hash); + state->hash.update(writer.send(*this)); } /* diff --git a/src/tls/next_protocol.cpp b/src/tls/next_protocol.cpp index 97b072440..17b77fb6e 100644 --- a/src/tls/next_protocol.cpp +++ b/src/tls/next_protocol.cpp @@ -8,6 +8,7 @@ #include <botan/internal/tls_messages.h> #include <botan/internal/tls_extensions.h> #include <botan/internal/tls_reader.h> +#include <botan/tls_record.h> namespace Botan { @@ -18,7 +19,7 @@ Next_Protocol::Next_Protocol(Record_Writer& writer, const std::string& protocol) : m_protocol(protocol) { - send(writer, hash); + hash.update(writer.send(*this)); } Next_Protocol::Next_Protocol(const MemoryRegion<byte>& buf) diff --git a/src/tls/rec_wri.cpp b/src/tls/rec_wri.cpp index 602d25397..cc7c6f79a 100644 --- a/src/tls/rec_wri.cpp +++ b/src/tls/rec_wri.cpp @@ -6,6 +6,7 @@ */ #include <botan/tls_record.h> +#include <botan/internal/tls_messages.h> #include <botan/internal/tls_session_key.h> #include <botan/internal/tls_handshake_hash.h> #include <botan/lookup.h> @@ -144,6 +145,25 @@ void Record_Writer::activate(Connection_Side side, throw Invalid_Argument("Record_Writer: Unknown hash " + mac_algo); } +MemoryVector<byte> Record_Writer::send(Handshake_Message& msg) + { + const MemoryVector<byte> buf = msg.serialize(); + MemoryVector<byte> send_buf(4); + + const size_t buf_size = buf.size(); + + send_buf[0] = msg.type(); + + for(size_t i = 1; i != 4; ++i) + send_buf[i] = get_byte<u32bit>(i, buf_size); + + send_buf += buf; + + send(HANDSHAKE, &send_buf[0], send_buf.size()); + + return send_buf; + } + /* * Send one or more records to the other side */ diff --git a/src/tls/s_hello.cpp b/src/tls/s_hello.cpp index 0ad78fc5b..9bcbdb5e9 100644 --- a/src/tls/s_hello.cpp +++ b/src/tls/s_hello.cpp @@ -51,7 +51,7 @@ Server_Hello::Server_Hello(Record_Writer& writer, comp_method = policy.choose_compression(c_hello.compression_methods()); - send(writer, hash); + hash.update(writer.send(*this)); } /* @@ -80,7 +80,7 @@ Server_Hello::Server_Hello(Record_Writer& writer, m_next_protocol(client_has_npn), m_next_protocols(next_protocols) { - send(writer, hash); + hash.update(writer.send(*this)); } /* @@ -174,7 +174,7 @@ MemoryVector<byte> Server_Hello::serialize() const Server_Hello_Done::Server_Hello_Done(Record_Writer& writer, Handshake_Hash& hash) { - send(writer, hash); + hash.update(writer.send(*this)); } /* diff --git a/src/tls/s_kex.cpp b/src/tls/s_kex.cpp index f25205e99..6707d2611 100644 --- a/src/tls/s_kex.cpp +++ b/src/tls/s_kex.cpp @@ -8,6 +8,7 @@ #include <botan/internal/tls_messages.h> #include <botan/internal/tls_reader.h> #include <botan/internal/tls_extensions.h> +#include <botan/tls_record.h> #include <botan/internal/assert.h> #include <botan/credentials_manager.h> #include <botan/loadstor.h> @@ -105,7 +106,7 @@ Server_Key_Exchange::Server_Key_Exchange(Record_Writer& writer, m_signature = signer.signature(rng); } - send(writer, state->hash); + state->hash.update(writer.send(*this)); } /** diff --git a/src/tls/tls_messages.h b/src/tls/tls_messages.h index 7162ece1a..3352c2a62 100644 --- a/src/tls/tls_messages.h +++ b/src/tls/tls_messages.h @@ -33,14 +33,14 @@ class Record_Reader; class Handshake_Message { public: - void send(Record_Writer& writer, Handshake_Hash& hash) const; - + virtual MemoryVector<byte> serialize() const = 0; virtual Handshake_Type type() const = 0; + Handshake_Message() {} virtual ~Handshake_Message() {} private: + Handshake_Message(const Handshake_Message&) {} Handshake_Message& operator=(const Handshake_Message&) { return (*this); } - virtual MemoryVector<byte> serialize() const = 0; }; MemoryVector<byte> make_hello_random(RandomNumberGenerator& rng); diff --git a/src/tls/tls_record.h b/src/tls/tls_record.h index 3bc58a0da..fb27db5e2 100644 --- a/src/tls/tls_record.h +++ b/src/tls/tls_record.h @@ -46,6 +46,8 @@ class BOTAN_DLL Record_Writer void send(byte type, const byte input[], size_t length); void send(byte type, byte val) { send(type, &val, 1); } + MemoryVector<byte> send(class Handshake_Message& msg); + void send_alert(const Alert& alert); void activate(Connection_Side side, |