=================================================================== RCS file: /cvsrepo/anoncvs/cvs/src/usr.bin/ssh/ssh-rsa.c,v retrieving revision 1.8.2.4 retrieving revision 1.9 diff -u -r1.8.2.4 -r1.9 --- src/usr.bin/ssh/ssh-rsa.c 2002/06/02 22:56:11 1.8.2.4 +++ src/usr.bin/ssh/ssh-rsa.c 2001/06/06 23:13:54 1.9 @@ -23,7 +23,7 @@ */ #include "includes.h" -RCSID("$OpenBSD: ssh-rsa.c,v 1.8.2.4 2002/06/02 22:56:11 miod Exp $"); +RCSID("$OpenBSD: ssh-rsa.c,v 1.9 2001/06/06 23:13:54 markus Exp $"); #include #include @@ -35,18 +35,17 @@ #include "key.h" #include "ssh-rsa.h" #include "compat.h" -#include "ssh.h" /* RSASSA-PKCS1-v1_5 (PKCS #1 v2.0 signature) with SHA1 */ int ssh_rsa_sign( Key *key, - u_char **sigp, u_int *lenp, - u_char *data, u_int datalen) + u_char **sigp, int *lenp, + u_char *data, int datalen) { const EVP_MD *evp_md; EVP_MD_CTX md; - u_char digest[EVP_MAX_MD_SIZE], *sig, *ret; + u_char *digest, *sig, *ret; u_int slen, dlen, len; int ok, nid; Buffer b; @@ -60,15 +59,18 @@ error("ssh_rsa_sign: EVP_get_digestbynid %d failed", nid); return -1; } + dlen = evp_md->md_size; + digest = xmalloc(dlen); EVP_DigestInit(&md, evp_md); EVP_DigestUpdate(&md, data, datalen); - EVP_DigestFinal(&md, digest, &dlen); + EVP_DigestFinal(&md, digest, NULL); slen = RSA_size(key->rsa); sig = xmalloc(slen); ok = RSA_sign(nid, digest, dlen, sig, &len, key->rsa); - memset(digest, 'd', sizeof(digest)); + memset(digest, 'd', dlen); + xfree(digest); if (ok != 1) { int ecode = ERR_get_error(); @@ -107,14 +109,14 @@ int ssh_rsa_verify( Key *key, - u_char *signature, u_int signaturelen, - u_char *data, u_int datalen) + u_char *signature, int signaturelen, + u_char *data, int datalen) { Buffer b; const EVP_MD *evp_md; EVP_MD_CTX md; char *ktype; - u_char digest[EVP_MAX_MD_SIZE], *sigblob; + u_char *sigblob, *digest; u_int len, dlen; int rlen, ret, nid; @@ -122,13 +124,13 @@ error("ssh_rsa_verify: no RSA key"); return -1; } - if (BN_num_bits(key->rsa->n) < SSH_RSA_MINIMUM_MODULUS_SIZE) { + if (BN_num_bits(key->rsa->n) < 768) { error("ssh_rsa_verify: n too small: %d bits", BN_num_bits(key->rsa->n)); return -1; } buffer_init(&b); - buffer_append(&b, signature, signaturelen); + buffer_append(&b, (char *) signature, signaturelen); ktype = buffer_get_string(&b, NULL); if (strcmp("ssh-rsa", ktype) != 0) { error("ssh_rsa_verify: cannot handle type %s", ktype); @@ -137,26 +139,29 @@ return -1; } xfree(ktype); - sigblob = buffer_get_string(&b, &len); + sigblob = (u_char *)buffer_get_string(&b, &len); rlen = buffer_len(&b); buffer_free(&b); - if (rlen != 0) { - error("ssh_rsa_verify: remaining bytes in signature %d", rlen); + if(rlen != 0) { xfree(sigblob); + error("ssh_rsa_verify: remaining bytes in signature %d", rlen); return -1; } nid = (datafellows & SSH_BUG_RSASIGMD5) ? NID_md5 : NID_sha1; if ((evp_md = EVP_get_digestbynid(nid)) == NULL) { - error("ssh_rsa_verify: EVP_get_digestbynid %d failed", nid); xfree(sigblob); + error("ssh_rsa_verify: EVP_get_digestbynid %d failed", nid); return -1; } + dlen = evp_md->md_size; + digest = xmalloc(dlen); EVP_DigestInit(&md, evp_md); EVP_DigestUpdate(&md, data, datalen); - EVP_DigestFinal(&md, digest, &dlen); + EVP_DigestFinal(&md, digest, NULL); ret = RSA_verify(nid, digest, dlen, sigblob, len, key->rsa); - memset(digest, 'd', sizeof(digest)); + memset(digest, 'd', dlen); + xfree(digest); memset(sigblob, 's', len); xfree(sigblob); if (ret == 0) {