changeset 867:d2d624c951ca

- Increase buffer size, fixes converting 521bit ECC key - Fix assertion that key size is exactly curve size
author Matt Johnston <matt@ucc.asn.au>
date Wed, 27 Nov 2013 21:30:05 +0800
parents 39d872718d4b
children 463e2b70be14
files keyimport.c
diffstat 1 files changed, 7 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/keyimport.c	Mon Nov 25 23:30:01 2013 +0800
+++ b/keyimport.c	Wed Nov 27 21:30:05 2013 +0800
@@ -1033,6 +1033,7 @@
 		int curve_oid_len = 0;
 		const void* curve_oid = NULL;
 		unsigned long pubkey_size = 2*curve_size+1;
+		unsigned int k_size;
 
 		/* version. less than 10 bytes */
 		buf_incrwritepos(seq_buf,
@@ -1040,11 +1041,12 @@
 		buf_putbyte(seq_buf, 1);
 
 		/* privateKey */
-		dropbear_assert(mp_unsigned_bin_size((*eck)->k) == curve_size);
+		k_size = mp_unsigned_bin_size((*eck)->k);
+		dropbear_assert(k_size <= curve_size);
 		buf_incrwritepos(seq_buf,
-			ber_write_id_len(buf_getwriteptr(seq_buf, 10), 4, curve_size, 0));
-	    mp_to_unsigned_bin((*eck)->k, buf_getwriteptr(seq_buf, curve_size));
-		buf_incrwritepos(seq_buf, curve_size);
+			ber_write_id_len(buf_getwriteptr(seq_buf, 10), 4, k_size, 0));
+	    mp_to_unsigned_bin((*eck)->k, buf_getwriteptr(seq_buf, k_size));
+		buf_incrwritepos(seq_buf, k_size);
 
 		/* SECGCurveNames */
 		switch (key->type)
@@ -1085,7 +1087,7 @@
 
 		buf_setpos(seq_buf, 0);
 			
-		outblob = (unsigned char*)m_malloc(200);
+		outblob = (unsigned char*)m_malloc(1000);
 
 		pos = 0;
 		pos += ber_write_id_len(outblob+pos, 16, seq_buf->len, ASN1_CONSTRUCTED);