diff options
| author | lloyd <[email protected]> | 2011-12-28 18:30:01 +0000 |
|---|---|---|
| committer | lloyd <[email protected]> | 2011-12-28 18:30:01 +0000 |
| commit | 2299fee297f8c3efcc59bd9a9c4ab97d40e39632 (patch) | |
| tree | 768f1411c763941bcde9d87aa8e23245cf7dc8c5 /src | |
| parent | 29033dac5b201c486048a6cfc75d72296caa0a92 (diff) | |
Fixes for DSA auth
Diffstat (limited to 'src')
| -rw-r--r-- | src/tls/cert_ver.cpp | 17 | ||||
| -rw-r--r-- | src/tls/s_kex.cpp | 2 |
2 files changed, 11 insertions, 8 deletions
diff --git a/src/tls/cert_ver.cpp b/src/tls/cert_ver.cpp index 023c6ccd7..73cd7343b 100644 --- a/src/tls/cert_ver.cpp +++ b/src/tls/cert_ver.cpp @@ -7,6 +7,7 @@ #include <botan/internal/tls_messages.h> #include <botan/internal/tls_reader.h> +#include <botan/internal/assert.h> #include <botan/tls_exceptn.h> #include <botan/pubkey.h> #include <botan/rsa.h> @@ -16,7 +17,7 @@ namespace Botan { -/** +/* * Create a new Certificate Verify message */ Certificate_Verify::Certificate_Verify(RandomNumberGenerator& rng, @@ -24,6 +25,8 @@ Certificate_Verify::Certificate_Verify(RandomNumberGenerator& rng, TLS_Handshake_Hash& hash, const Private_Key* priv_key) { + BOTAN_ASSERT_NONNULL(priv_key, ""); + std::string padding = ""; Signature_Format format = IEEE_1363; @@ -31,7 +34,7 @@ Certificate_Verify::Certificate_Verify(RandomNumberGenerator& rng, padding = "EMSA3(TLS.Digest.0)"; else if(priv_key->algo_name() == "DSA") { - padding == "EMSA1(SHA-1)"; + padding = "EMSA1(SHA-1)"; format = DER_SEQUENCE; } else @@ -44,7 +47,7 @@ Certificate_Verify::Certificate_Verify(RandomNumberGenerator& rng, send(writer, hash); } -/** +/* * Serialize a Certificate Verify message */ MemoryVector<byte> Certificate_Verify::serialize() const @@ -59,7 +62,7 @@ MemoryVector<byte> Certificate_Verify::serialize() const return buf; } -/** +/* * Deserialize a Certificate Verify message */ void Certificate_Verify::deserialize(const MemoryRegion<byte>& buf) @@ -68,7 +71,7 @@ void Certificate_Verify::deserialize(const MemoryRegion<byte>& buf) signature = reader.get_range<byte>(2, 0, 65535); } -/** +/* * Verify a Certificate Verify message */ bool Certificate_Verify::verify(const X509_Certificate& cert, @@ -76,15 +79,15 @@ bool Certificate_Verify::verify(const X509_Certificate& cert, Version_Code version, const SecureVector<byte>& master_secret) { - // FIXME: duplicate of Server_Key_Exchange::verify - std::auto_ptr<Public_Key> key(cert.subject_public_key()); std::string padding = ""; Signature_Format format = IEEE_1363; if(key->algo_name() == "RSA") + { padding = "EMSA3(TLS.Digest.0)"; + } else if(key->algo_name() == "DSA") { if(version == SSL_V3) diff --git a/src/tls/s_kex.cpp b/src/tls/s_kex.cpp index bb2ed7c1d..dd79e527e 100644 --- a/src/tls/s_kex.cpp +++ b/src/tls/s_kex.cpp @@ -161,7 +161,7 @@ bool Server_Key_Exchange::verify(const X509_Certificate& cert, padding = "EMSA3(TLS.Digest.0)"; else if(key->algo_name() == "DSA") { - padding == "EMSA1(SHA-1)"; + padding = "EMSA1(SHA-1)"; format = DER_SEQUENCE; } else |