Skip to content

Commit 67200b6

Browse files
karel-mkarel-m
committed
ecc_sign+verify_hash_raw > ecc_sign+verify_hash_rfc7518
1 parent c14bcf4 commit 67200b6

3 files changed

Lines changed: 154 additions & 144 deletions

File tree

‎src/headers/tomcrypt_pk.h‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -314,17 +314,17 @@ int ecc_decrypt_key(const unsigned char *in, unsigned long inlen,
314314
unsigned char *out, unsigned long *outlen,
315315
ecc_key *key);
316316

317-
int ecc_sign_hash_raw(const unsigned char *in, unsigned long inlen,
318-
void *r, void *s,
319-
prng_state *prng, int wprng, ecc_key *key);
317+
int ecc_sign_hash_rfc7518(const unsigned char *in, unsigned long inlen,
318+
unsigned char *out, unsigned long *outlen,
319+
prng_state *prng, int wprng, ecc_key *key);
320320

321321
int ecc_sign_hash(const unsigned char *in, unsigned long inlen,
322322
unsigned char *out, unsigned long *outlen,
323323
prng_state *prng, int wprng, ecc_key *key);
324324

325-
int ecc_verify_hash_raw( void *r, void *s,
326-
const unsigned char *hash, unsigned long hashlen,
327-
int *stat, ecc_key *key);
325+
int ecc_verify_hash_rfc7518(const unsigned char *sig, unsigned long siglen,
326+
const unsigned char *hash, unsigned long hashlen,
327+
int *stat, ecc_key *key);
328328

329329
int ecc_verify_hash(const unsigned char *sig, unsigned long siglen,
330330
const unsigned char *hash, unsigned long hashlen,

‎src/pk/ecc/ecc_sign_hash.c‎

Lines changed: 82 additions & 69 deletions
Original file line numberDiff line numberDiff line change
@@ -7,43 +7,29 @@
77
* guarantee it works.
88
*/
99

10-
/* Implements ECC over Z/pZ for curve y^2 = x^3 - 3x + b
11-
*
12-
* All curves taken from NIST recommendation paper of July 1999
13-
* Available at http://csrc.nist.gov/cryptval/dss.htm
14-
*/
1510
#include "tomcrypt.h"
1611

12+
#ifdef LTC_MECC
13+
1714
/**
1815
@file ecc_sign_hash.c
1916
ECC Crypto, Tom St Denis
2017
*/
2118

22-
#ifdef LTC_MECC
23-
24-
/**
25-
Sign a hash with ECC
26-
@param in The hash to sign
27-
@param inlen The length of the hash to sign
28-
@param r The "r" integer of the signature (caller must initialize with mp_init() first)
29-
@param s The "s" integer of the signature (caller must initialize with mp_init() first)
30-
@param prng An active PRNG state
31-
@param wprng The index of the PRNG desired
32-
@param key A private ECC key
33-
@return CRYPT_OK if successful
34-
*/
35-
int ecc_sign_hash_raw(const unsigned char *in, unsigned long inlen,
36-
void *r, void *s,
37-
prng_state *prng, int wprng, ecc_key *key)
19+
static int ecc_sign_hash_ex(const unsigned char *in, unsigned long inlen,
20+
unsigned char *out, unsigned long *outlen,
21+
prng_state *prng, int wprng, ecc_key *key, int sigformat)
3822
{
3923
ecc_key pubkey;
40-
void *e, *p;
24+
void *r, *s, *e, *p;
4125
int err;
26+
unsigned long pbits, pbytes, i, shift_right;
27+
unsigned char ch, buf[MAXBLOCKSIZE];
4228

43-
LTC_ARGCHK(in != NULL);
44-
LTC_ARGCHK(r != NULL);
45-
LTC_ARGCHK(s != NULL);
46-
LTC_ARGCHK(key != NULL);
29+
LTC_ARGCHK(in != NULL);
30+
LTC_ARGCHK(out != NULL);
31+
LTC_ARGCHK(outlen != NULL);
32+
LTC_ARGCHK(key != NULL);
4733

4834
/* is this a private key? */
4935
if (key->type != PK_PRIVATE) {
@@ -59,13 +45,30 @@ int ecc_sign_hash_raw(const unsigned char *in, unsigned long inlen,
5945
return err;
6046
}
6147

62-
/* get the hash and load it as a bignum into 'e' */
6348
/* init the bignums */
64-
if ((err = mp_init_multi(&p, &e, NULL)) != CRYPT_OK) {
49+
if ((err = mp_init_multi(&r, &s, &p, &e, NULL)) != CRYPT_OK) {
6550
return err;
6651
}
67-
if ((err = mp_read_radix(p, (char *)key->dp->order, 16)) != CRYPT_OK) { goto errnokey; }
68-
if ((err = mp_read_unsigned_bin(e, (unsigned char *)in, (int)inlen)) != CRYPT_OK) { goto errnokey; }
52+
if ((err = mp_read_radix(p, (char *)key->dp->order, 16)) != CRYPT_OK) { goto errnokey; }
53+
54+
/* get the hash and load it as a bignum into 'e' */
55+
pbits = mp_count_bits(p);
56+
pbytes = (pbits+7) >> 3;
57+
if (pbits > inlen*8) {
58+
if ((err = mp_read_unsigned_bin(e, (unsigned char *)in, inlen)) != CRYPT_OK) { goto errnokey; }
59+
}
60+
else if (pbits % 8 == 0) {
61+
if ((err = mp_read_unsigned_bin(e, (unsigned char *)in, pbytes)) != CRYPT_OK) { goto errnokey; }
62+
}
63+
else {
64+
shift_right = 8 - pbits % 8;
65+
for (i=0, ch=0; i<pbytes; i++) {
66+
buf[i] = ch;
67+
ch = (in[i] << (8-shift_right));
68+
buf[i] = buf[i] ^ (in[i] >> shift_right);
69+
}
70+
if ((err = mp_read_unsigned_bin(e, (unsigned char *)buf, pbytes)) != CRYPT_OK) { goto errnokey; }
71+
}
6972

7073
/* make up a key and export the public copy */
7174
for (;;) {
@@ -74,31 +77,47 @@ int ecc_sign_hash_raw(const unsigned char *in, unsigned long inlen,
7477
}
7578

7679
/* find r = x1 mod n */
77-
if ((err = mp_mod(pubkey.pubkey.x, p, r)) != CRYPT_OK) { goto error; }
80+
if ((err = mp_mod(pubkey.pubkey.x, p, r)) != CRYPT_OK) { goto error; }
7881

7982
if (mp_iszero(r) == LTC_MP_YES) {
8083
ecc_free(&pubkey);
8184
} else {
82-
/* find s = (e + xr)/k */
83-
if ((err = mp_invmod(pubkey.k, p, pubkey.k)) != CRYPT_OK) { goto error; } /* k = 1/k */
84-
if ((err = mp_mulmod(key->k, r, p, s)) != CRYPT_OK) { goto error; } /* s = xr */
85-
if ((err = mp_add(e, s, s)) != CRYPT_OK) { goto error; } /* s = e + xr */
86-
if ((err = mp_mod(s, p, s)) != CRYPT_OK) { goto error; } /* s = e + xr */
87-
if ((err = mp_mulmod(s, pubkey.k, p, s)) != CRYPT_OK) { goto error; } /* s = (e + xr)/k */
88-
ecc_free(&pubkey);
89-
if (mp_iszero(s) == LTC_MP_NO) {
90-
break;
91-
}
85+
/* find s = (e + xr)/k */
86+
if ((err = mp_invmod(pubkey.k, p, pubkey.k)) != CRYPT_OK) { goto error; } /* k = 1/k */
87+
if ((err = mp_mulmod(key->k, r, p, s)) != CRYPT_OK) { goto error; } /* s = xr */
88+
if ((err = mp_add(e, s, s)) != CRYPT_OK) { goto error; } /* s = e + xr */
89+
if ((err = mp_mod(s, p, s)) != CRYPT_OK) { goto error; } /* s = e + xr */
90+
if ((err = mp_mulmod(s, pubkey.k, p, s)) != CRYPT_OK) { goto error; } /* s = (e + xr)/k */
91+
ecc_free(&pubkey);
92+
if (mp_iszero(s) == LTC_MP_NO) {
93+
break;
94+
}
9295
}
9396
}
9497

95-
err = CRYPT_OK;
98+
if (sigformat == 1) {
99+
/* RFC7518 format */
100+
if (*outlen < 2*pbytes) { err = CRYPT_MEM; goto errnokey; }
101+
zeromem(out, 2*pbytes);
102+
i = mp_unsigned_bin_size(r);
103+
if ((err = mp_to_unsigned_bin(r, out + (pbytes - i))) != CRYPT_OK) { goto errnokey; }
104+
i = mp_unsigned_bin_size(s);
105+
if ((err = mp_to_unsigned_bin(s, out + (2*pbytes - i))) != CRYPT_OK) { goto errnokey; }
106+
*outlen = 2*pbytes;
107+
err = CRYPT_OK;
108+
}
109+
else {
110+
/* store as ASN.1 SEQUENCE { r, s -- integer } */
111+
err = der_encode_sequence_multi(out, outlen,
112+
LTC_ASN1_INTEGER, 1UL, r,
113+
LTC_ASN1_INTEGER, 1UL, s,
114+
LTC_ASN1_EOL, 0UL, NULL);
115+
}
96116
goto errnokey;
97-
98117
error:
99118
ecc_free(&pubkey);
100119
errnokey:
101-
mp_clear_multi(p, e, NULL);
120+
mp_clear_multi(r, s, p, e, NULL);
102121
return err;
103122
}
104123

@@ -117,35 +136,29 @@ int ecc_sign_hash(const unsigned char *in, unsigned long inlen,
117136
unsigned char *out, unsigned long *outlen,
118137
prng_state *prng, int wprng, ecc_key *key)
119138
{
120-
void *r, *s;
121-
int err;
122-
123-
LTC_ARGCHK(in != NULL);
124-
LTC_ARGCHK(out != NULL);
125-
LTC_ARGCHK(outlen != NULL);
126-
LTC_ARGCHK(key != NULL);
127-
128-
if (mp_init_multi(&r, &s, NULL) != CRYPT_OK) {
129-
return CRYPT_MEM;
130-
}
131-
132-
if ((err = ecc_sign_hash_raw(in, inlen, r, s, prng, wprng, key)) != CRYPT_OK) {
133-
goto error;
134-
}
135-
136-
/* store as SEQUENCE { r, s -- integer } */
137-
err = der_encode_sequence_multi(out, outlen,
138-
LTC_ASN1_INTEGER, 1UL, r,
139-
LTC_ASN1_INTEGER, 1UL, s,
140-
LTC_ASN1_EOL, 0UL, NULL);
139+
return ecc_sign_hash_ex(in, inlen, out, outlen, prng, wprng, key, 0);
140+
}
141141

142-
error:
143-
mp_clear_multi(r, s, NULL);
144-
return err;
142+
/**
143+
Sign a message digest in RFC7518 format
144+
@param in The message digest to sign
145+
@param inlen The length of the digest
146+
@param out [out] The destination for the signature
147+
@param outlen [in/out] The max size and resulting size of the signature
148+
@param prng An active PRNG state
149+
@param wprng The index of the PRNG you wish to use
150+
@param key A private ECC key
151+
@return CRYPT_OK if successful
152+
*/
153+
int ecc_sign_hash_rfc7518(const unsigned char *in, unsigned long inlen,
154+
unsigned char *out, unsigned long *outlen,
155+
prng_state *prng, int wprng, ecc_key *key)
156+
{
157+
return ecc_sign_hash_ex(in, inlen, out, outlen, prng, wprng, key, 1);
145158
}
146159

147160
#endif
161+
148162
/* ref: $Format:%D$ */
149163
/* git commit: $Format:%H$ */
150164
/* commit time: $Format:%ai$ */
151-

0 commit comments

Comments
 (0)