Fix hash functions.

[umurmur.git] / src / ssl.c

diff --git a/src/ssl.c b/src/ssl.c
index d2ef7610dbfdddb2a1ae116fda037d1384eb2188..35b226ffda1f228586a36e6d264d90a5e7a38599 100644 (file)
--- a/src/ssl.c
+++ b/src/ssl.c
@@ -168,14 +168,14 @@ void SSLi_deinit(void)
 }
 
 /* Create SHA1 of last certificate in the peer's chain. */
-void SSLi_getSHA1Hash(SSL_handle_t *ssl, uint8_t *hash)
+bool_t SSLi_getSHA1Hash(SSL_handle_t *ssl, uint8_t *hash)
 {
        x509_cert *cert = ssl->peer_cert;
        if (!ssl->peer_cert) {
-               /* XXX what to do? */
-               return;
+               return false;
        }       
        sha1(cert->raw.p, cert->raw.len, hash);
+       return true;
 }
        
 SSL_handle_t *SSLi_newconnection(int *fd, bool_t *SSLready)
@@ -607,16 +607,15 @@ SSL_handle_t *SSLi_newconnection(int *fd, bool_t *SSLready)
 }
 
 /* Create SHA1 of last certificate in the peer's chain. */
-void SSLi_getSHA1Hash(SSL_handle_t *ssl, uint8_t *hash)
+bool_t SSLi_getSHA1Hash(SSL_handle_t *ssl, uint8_t *hash)
 {
        X509 *x509;
        uint8_t *buf, *p;
        int len;
        
        x509 = SSL_get_peer_certificate(ssl);
-       if (x509) {
-               /* XXX what to do? */
-               return;
+       if (!x509) {
+               return false;
        }       
        
        len = i2d_X509(x509, NULL);
@@ -624,11 +623,13 @@ void SSLi_getSHA1Hash(SSL_handle_t *ssl, uint8_t *hash)
        if (buf == NULL) {
                Log_fatal("malloc");
        }
+
+       p = buf;
+       i2d_X509(x509, &p);     
        
-       i2d_X509(x509, &p);
-       
-       SHA1(p, len, hash);
+       SHA1(buf, len, hash);
        free(buf);
+       return true;
 }
  
 void SSLi_closeconnection(SSL_handle_t *ssl)