some const fixes and cleanup
[openssl.git] / crypto / evp / evp_pkey.c
index 1772647b164a81640a63f123658d1f013f8f3e14..7260460744d646f90e44f976baa84ebaf800abd9 100644 (file)
 #include "cryptlib.h"
 #include <openssl/x509.h>
 #include <openssl/rand.h>
+#include <openssl/rsa.h>
+#include <openssl/dsa.h>
+#include <openssl/bn.h>
 
 #ifndef OPENSSL_NO_DSA
 static int dsa_pkey2pkcs8(PKCS8_PRIV_KEY_INFO *p8inf, EVP_PKEY *pkey);
 #endif
-#ifndef OPENSSL_NO_ECDSA
-static int ecdsa_pkey2pkcs8(PKCS8_PRIV_KEY_INFO *p8inf, EVP_PKEY *pkey);
+#ifndef OPENSSL_NO_EC
+static int eckey_pkey2pkcs8(PKCS8_PRIV_KEY_INFO *p8inf, EVP_PKEY *pkey);
 #endif
 
 /* Extract a private key from a PKCS8 structure */
@@ -79,19 +82,21 @@ EVP_PKEY *EVP_PKCS82PKEY (PKCS8_PRIV_KEY_INFO *p8)
 #endif
 #ifndef OPENSSL_NO_DSA
        DSA *dsa = NULL;
+       ASN1_TYPE *t1, *t2;
+       ASN1_INTEGER *privkey;
+       STACK_OF(ASN1_TYPE) *ndsa = NULL;
 #endif
-#ifndef OPENSSL_NO_ECDSA
-       ECDSA    *ecdsa = NULL;
+#ifndef OPENSSL_NO_EC
+       EC_KEY *eckey = NULL;
+       const unsigned char *p_tmp;
 #endif
-#if !defined(OPENSSL_NO_DSA) || !defined(OPENSSL_NO_ECDSA)
-       ASN1_INTEGER *privkey;
-       ASN1_TYPE    *t1, *t2, *param = NULL;
-       STACK_OF(ASN1_TYPE) *n_stack = NULL;
+#if !defined(OPENSSL_NO_DSA) || !defined(OPENSSL_NO_EC)
+       ASN1_TYPE    *param = NULL;     
        BN_CTX *ctx = NULL;
        int plen;
 #endif
        X509_ALGOR *a;
-       unsigned char *p;
+       const unsigned char *p;
        const unsigned char *cp;
        int pkeylen;
        int  nid;
@@ -124,172 +129,194 @@ EVP_PKEY *EVP_PKCS82PKEY (PKCS8_PRIV_KEY_INFO *p8)
                EVP_PKEY_assign_RSA (pkey, rsa);
                break;
 #endif
-#if !defined(OPENSSL_NO_DSA) || !defined(OPENSSL_NO_ECDSA)
-               case NID_ecdsa_with_SHA1:
+#ifndef OPENSSL_NO_DSA
                case NID_dsa:
-               /* PKCS#8 DSA/ECDSA is weird: you just get a private key integer
+               /* PKCS#8 DSA is weird: you just get a private key integer
                 * and parameters in the AlgorithmIdentifier the pubkey must
                 * be recalculated.
                 */
        
-               /* Check for broken DSA/ECDSA PKCS#8, UGH! */
-               if(*p == (V_ASN1_SEQUENCE|V_ASN1_CONSTRUCTED)) 
-               {
-                       if(!(n_stack = ASN1_seq_unpack_ASN1_TYPE(p, pkeylen, 
+               /* Check for broken DSA PKCS#8, UGH! */
+               if(*p == (V_ASN1_SEQUENCE|V_ASN1_CONSTRUCTED)) {
+                   if(!(ndsa = ASN1_seq_unpack_ASN1_TYPE(p, pkeylen, 
                                                          d2i_ASN1_TYPE,
-                                                         ASN1_TYPE_free))) 
-                       {
-                               EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
-                               goto err;
-                       }
-                       if(sk_ASN1_TYPE_num(n_stack) != 2 ) 
-                       {
-                               EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
-                               goto err;
-                       }
+                                                         ASN1_TYPE_free))) {
+                       EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
+                       goto dsaerr;
+                   }
+                   if(sk_ASN1_TYPE_num(ndsa) != 2 ) {
+                       EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
+                       goto dsaerr;
+                   }
                    /* Handle Two broken types:
                     * SEQUENCE {parameters, priv_key}
                     * SEQUENCE {pub_key, priv_key}
                     */
 
-                   t1 = sk_ASN1_TYPE_value(n_stack, 0);
-                   t2 = sk_ASN1_TYPE_value(n_stack, 1);
-                   if(t1->type == V_ASN1_SEQUENCE) 
-                   {
+                   t1 = sk_ASN1_TYPE_value(ndsa, 0);
+                   t2 = sk_ASN1_TYPE_value(ndsa, 1);
+                   if(t1->type == V_ASN1_SEQUENCE) {
                        p8->broken = PKCS8_EMBEDDED_PARAM;
                        param = t1;
-                   } 
-                   else if(a->parameter->type == V_ASN1_SEQUENCE) 
-                   {
+                   } else if(a->parameter->type == V_ASN1_SEQUENCE) {
                        p8->broken = PKCS8_NS_DB;
                        param = a->parameter;
-                   } 
-                   else 
-                   {
+                   } else {
                        EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
-                       goto err;
+                       goto dsaerr;
                    }
 
                    if(t2->type != V_ASN1_INTEGER) {
                        EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
-                       goto err;
+                       goto dsaerr;
                    }
                    privkey = t2->value.integer;
-               } 
-               else 
-               {
-                       if (!(privkey=d2i_ASN1_INTEGER (NULL, &p, pkeylen))) 
-                       {
+               } else {
+                       if (!(privkey=d2i_ASN1_INTEGER (NULL, &p, pkeylen))) {
                                EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
-                               goto err;
+                               goto dsaerr;
                        }
                        param = p8->pkeyalg->parameter;
                }
-               if (!param || (param->type != V_ASN1_SEQUENCE)) 
-               {
+               if (!param || (param->type != V_ASN1_SEQUENCE)) {
                        EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
-                       goto err;
+                       goto dsaerr;
                }
                cp = p = param->value.sequence->data;
                plen = param->value.sequence->length;
-               if (!(ctx = BN_CTX_new())) 
-               {
+               if (!(dsa = d2i_DSAparams (NULL, &cp, plen))) {
+                       EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
+                       goto dsaerr;
+               }
+               /* We have parameters now set private key */
+               if (!(dsa->priv_key = ASN1_INTEGER_to_BN(privkey, NULL))) {
+                       EVPerr(EVP_F_EVP_PKCS82PKEY,EVP_R_BN_DECODE_ERROR);
+                       goto dsaerr;
+               }
+               /* Calculate public key (ouch!) */
+               if (!(dsa->pub_key = BN_new())) {
                        EVPerr(EVP_F_EVP_PKCS82PKEY,ERR_R_MALLOC_FAILURE);
-                       goto err;
+                       goto dsaerr;
                }
-               if (nid == NID_dsa)
+               if (!(ctx = BN_CTX_new())) {
+                       EVPerr(EVP_F_EVP_PKCS82PKEY,ERR_R_MALLOC_FAILURE);
+                       goto dsaerr;
+               }
+                       
+               if (!BN_mod_exp(dsa->pub_key, dsa->g,
+                                                dsa->priv_key, dsa->p, ctx)) {
+                       
+                       EVPerr(EVP_F_EVP_PKCS82PKEY,EVP_R_BN_PUBKEY_ERROR);
+                       goto dsaerr;
+               }
+
+               EVP_PKEY_assign_DSA(pkey, dsa);
+               BN_CTX_free (ctx);
+               if(ndsa) sk_ASN1_TYPE_pop_free(ndsa, ASN1_TYPE_free);
+               else ASN1_INTEGER_free(privkey);
+               break;
+               dsaerr:
+               BN_CTX_free (ctx);
+               sk_ASN1_TYPE_pop_free(ndsa, ASN1_TYPE_free);
+               DSA_free(dsa);
+               EVP_PKEY_free(pkey);
+               return NULL;
+               break;
+#endif
+#ifndef OPENSSL_NO_EC
+               case NID_X9_62_id_ecPublicKey:
+               p_tmp = p;
+               /* extract the ec parameters */
+               param = p8->pkeyalg->parameter;
+
+               if (!param || ((param->type != V_ASN1_SEQUENCE) &&
+                   (param->type != V_ASN1_OBJECT)))
                {
-#ifndef OPENSSL_NO_DSA
-                       if (!(dsa = d2i_DSAparams (NULL, &cp, plen))) 
-                       {
-                               EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
-                               goto err;
-                       }
-                       /* We have parameters now set private key */
-                       if (!(dsa->priv_key = ASN1_INTEGER_to_BN(privkey, NULL))) 
-                       {
-                               EVPerr(EVP_F_EVP_PKCS82PKEY,EVP_R_BN_DECODE_ERROR);
-                               goto err;
-                       }
-                       /* Calculate public key (ouch!) */
-                       if (!(dsa->pub_key = BN_new())) 
-                       {
-                               EVPerr(EVP_F_EVP_PKCS82PKEY,ERR_R_MALLOC_FAILURE);
-                               goto err;
-                       }
-                       if (!BN_mod_exp(dsa->pub_key, dsa->g,
-                                                dsa->priv_key, dsa->p, ctx)) 
-                       {
-                               EVPerr(EVP_F_EVP_PKCS82PKEY,EVP_R_BN_PUBKEY_ERROR);
-                               goto err;
-                       }
+                       EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
+                       goto ecerr;
+               }
 
-                       EVP_PKEY_assign_DSA(pkey, dsa);
-                       BN_CTX_free(ctx);
-                       if(n_stack) sk_ASN1_TYPE_pop_free(n_stack, ASN1_TYPE_free);
-                       else ASN1_INTEGER_free(privkey);
-#else
-                       EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_UNSUPPORTED_PRIVATE_KEY_ALGORITHM);
-                       goto err;
-#endif 
-               } 
-               else /* nid == NID_ecdsa_with_SHA1 */
+               if (param->type == V_ASN1_SEQUENCE)
                {
-#ifndef OPENSSL_NO_ECDSA
-                       if ((ecdsa = d2i_ECDSAParameters(NULL, &cp, plen)) == NULL)
+                       cp = p = param->value.sequence->data;
+                       plen = param->value.sequence->length;
+
+                       if (!(eckey = d2i_ECParameters(NULL, &cp, plen)))
                        {
-                               EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
-                               goto err;
+                               EVPerr(EVP_F_EVP_PKCS82PKEY,
+                                       EVP_R_DECODE_ERROR);
+                               goto ecerr;
                        }
-                       if ((ecdsa->priv_key = ASN1_INTEGER_to_BN(privkey, NULL)) == NULL)
+               }
+               else
+               {
+                       cp = p = param->value.object->data;
+                       plen = param->value.object->length;
+
+                       /* type == V_ASN1_OBJECT => the parameters are given
+                        * by an asn1 OID
+                        */
+                       if ((eckey = EC_KEY_new()) == NULL)
                        {
-                               EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
-                               goto err;
+                               EVPerr(EVP_F_EVP_PKCS82PKEY,
+                                       ERR_R_MALLOC_FAILURE);
+                               goto ecerr;
                        }
-                       if ((ecdsa->pub_key = EC_POINT_new(ecdsa->group)) == NULL)
+                       if ((eckey->group = EC_GROUP_new_by_nid(
+                             OBJ_obj2nid(a->parameter->value.object))) == NULL)
+                               goto ecerr;
+                       EC_GROUP_set_asn1_flag(eckey->group, 
+                                               OPENSSL_EC_NAMED_CURVE);
+               }
+
+               /* We have parameters now set private key */
+               if (!d2i_ECPrivateKey(&eckey, &p_tmp, pkeylen))
+               {
+                       EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_DECODE_ERROR);
+                       goto ecerr;
+               }
+
+               /* calculate public key (if necessary) */
+               if (!eckey->pub_key)
+               {
+                       /* the public key was not included in the SEC1 private
+                        * key => calculate the public key */
+                       eckey->pub_key = EC_POINT_new(eckey->group);
+                       if (!eckey->pub_key)
                        {
                                EVPerr(EVP_F_EVP_PKCS82PKEY, ERR_R_EC_LIB);
-                               goto err;
+                               goto ecerr;
                        }
-                       if (!EC_POINT_copy(ecdsa->pub_key, EC_GROUP_get0_generator(ecdsa->group)))
+                       if (!EC_POINT_copy(eckey->pub_key, 
+                               EC_GROUP_get0_generator(eckey->group)))
                        {
                                EVPerr(EVP_F_EVP_PKCS82PKEY, ERR_R_EC_LIB);
-                               goto err;
+                               goto ecerr;
                        }
-                       if (!EC_POINT_mul(ecdsa->group, ecdsa->pub_key, ecdsa->priv_key,
-                                         NULL, NULL, ctx))
+                       if (!EC_POINT_mul(eckey->group, eckey->pub_key, 
+                               eckey->priv_key, NULL, NULL, ctx))
                        {
                                EVPerr(EVP_F_EVP_PKCS82PKEY, ERR_R_EC_LIB);
-                               goto err;
+                               goto ecerr;
                        }
-                       
-                       EVP_PKEY_assign_ECDSA(pkey, ecdsa);
-                       BN_CTX_free(ctx);
-                       if (n_stack) sk_ASN1_TYPE_pop_free(n_stack, ASN1_TYPE_free);
-                       else
-                               ASN1_INTEGER_free(privkey);
-#else
-                       EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_UNSUPPORTED_PRIVATE_KEY_ALGORITHM);
-                       goto err;
-#endif
                }
+
+               EVP_PKEY_assign_EC_KEY(pkey, eckey);
+               if (ctx)
+                       BN_CTX_free(ctx);
                break;
-err:
-               if (ctx)   BN_CTX_free(ctx);
-               sk_ASN1_TYPE_pop_free(n_stack, ASN1_TYPE_free);
-#ifndef OPENSSL_NO_DSA
-               if (dsa)   DSA_free(dsa);
-#endif
-#ifndef OPENSSL_NO_ECDSA
-               if (ecdsa) ECDSA_free(ecdsa);
-#endif
-               if (pkey)  EVP_PKEY_free(pkey);
+ecerr:
+               if (ctx)
+                       BN_CTX_free(ctx);
+               if (eckey)
+                       EC_KEY_free(eckey);
+               if (pkey)
+                       EVP_PKEY_free(pkey);
                return NULL;
-               break;
 #endif
                default:
                EVPerr(EVP_F_EVP_PKCS82PKEY, EVP_R_UNSUPPORTED_PRIVATE_KEY_ALGORITHM);
-               if (!a->algorithm) strcpy (obj_tmp, "NULL");
+               if (!a->algorithm) BUF_strlcpy (obj_tmp, "NULL", sizeof obj_tmp);
                else i2t_ASN1_OBJECT(obj_tmp, 80, a->algorithm);
                ERR_add_error_data(2, "TYPE=", obj_tmp);
                EVP_PKEY_free (pkey);
@@ -314,7 +341,11 @@ PKCS8_PRIV_KEY_INFO *EVP_PKEY2PKCS8_broken(EVP_PKEY *pkey, int broken)
                return NULL;
        }
        p8->broken = broken;
-       ASN1_INTEGER_set (p8->version, 0);
+       if (!ASN1_INTEGER_set(p8->version, 0)) {
+               EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+               PKCS8_PRIV_KEY_INFO_free (p8);
+               return NULL;
+       }
        if (!(p8->pkeyalg->parameter = ASN1_TYPE_new ())) {
                EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
                PKCS8_PRIV_KEY_INFO_free (p8);
@@ -329,7 +360,7 @@ PKCS8_PRIV_KEY_INFO *EVP_PKEY2PKCS8_broken(EVP_PKEY *pkey, int broken)
 
                p8->pkeyalg->algorithm = OBJ_nid2obj(NID_rsaEncryption);
                p8->pkeyalg->parameter->type = V_ASN1_NULL;
-               if (!ASN1_pack_string ((char *)pkey, i2d_PrivateKey,
+               if (!ASN1_pack_string_of (EVP_PKEY,pkey, i2d_PrivateKey,
                                         &p8->pkey->value.octet_string)) {
                        EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
                        PKCS8_PRIV_KEY_INFO_free (p8);
@@ -346,9 +377,9 @@ PKCS8_PRIV_KEY_INFO *EVP_PKEY2PKCS8_broken(EVP_PKEY *pkey, int broken)
 
                break;
 #endif
-#ifndef OPENSSL_NO_ECDSA
-               case EVP_PKEY_ECDSA:
-               if (!ecdsa_pkey2pkcs8(p8, pkey))
+#ifndef OPENSSL_NO_EC
+               case EVP_PKEY_EC:
+               if (!eckey_pkey2pkcs8(p8, pkey))
                {
                        PKCS8_PRIV_KEY_INFO_free(p8);
                        return(NULL);
@@ -361,7 +392,7 @@ PKCS8_PRIV_KEY_INFO *EVP_PKEY2PKCS8_broken(EVP_PKEY *pkey, int broken)
                return NULL;
        }
        RAND_add(p8->pkey->value.octet_string->data,
-                p8->pkey->value.octet_string->length, 0);
+                p8->pkey->value.octet_string->length, 0.0);
        return p8;
 }
 
@@ -383,37 +414,41 @@ PKCS8_PRIV_KEY_INFO *PKCS8_set_broken(PKCS8_PRIV_KEY_INFO *p8, int broken)
                default:
                EVPerr(EVP_F_EVP_PKCS8_SET_BROKEN,EVP_R_PKCS8_UNKNOWN_BROKEN_TYPE);
                return NULL;
-               break;
-               
        }
 }
 
 #ifndef OPENSSL_NO_DSA
 static int dsa_pkey2pkcs8(PKCS8_PRIV_KEY_INFO *p8, EVP_PKEY *pkey)
 {
-       ASN1_STRING *params;
-       ASN1_INTEGER *prkey;
-       ASN1_TYPE *ttmp;
-       STACK_OF(ASN1_TYPE) *ndsa;
-       unsigned char *p, *q;
+       ASN1_STRING *params = NULL;
+       ASN1_INTEGER *prkey = NULL;
+       ASN1_TYPE *ttmp = NULL;
+       STACK_OF(ASN1_TYPE) *ndsa = NULL;
+       unsigned char *p = NULL, *q;
        int len;
 
        p8->pkeyalg->algorithm = OBJ_nid2obj(NID_dsa);
        len = i2d_DSAparams (pkey->pkey.dsa, NULL);
        if (!(p = OPENSSL_malloc(len))) {
                EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
-               PKCS8_PRIV_KEY_INFO_free (p8);
-               return 0;
+               goto err;
        }
        q = p;
        i2d_DSAparams (pkey->pkey.dsa, &q);
-       params = ASN1_STRING_new();
-       ASN1_STRING_set(params, p, len);
+       if (!(params = ASN1_STRING_new())) {
+               EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+               goto err;
+       }
+       if (!ASN1_STRING_set(params, p, len)) {
+               EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+               goto err;
+       }
        OPENSSL_free(p);
+       p = NULL;
        /* Get private key into integer */
        if (!(prkey = BN_to_ASN1_INTEGER (pkey->pkey.dsa->priv_key, NULL))) {
                EVPerr(EVP_F_EVP_PKEY2PKCS8,EVP_R_ENCODE_ERROR);
-               return 0;
+               goto err;
        }
 
        switch(p8->broken) {
@@ -421,15 +456,16 @@ static int dsa_pkey2pkcs8(PKCS8_PRIV_KEY_INFO *p8, EVP_PKEY *pkey)
                case PKCS8_OK:
                case PKCS8_NO_OCTET:
 
-               if (!ASN1_pack_string((char *)prkey, i2d_ASN1_INTEGER,
+               if (!ASN1_pack_string_of(ASN1_INTEGER,prkey, i2d_ASN1_INTEGER,
                                         &p8->pkey->value.octet_string)) {
                        EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
-                       M_ASN1_INTEGER_free (prkey);
-                       return 0;
+                       goto err;
                }
 
                M_ASN1_INTEGER_free (prkey);
+               prkey = NULL;
                p8->pkeyalg->parameter->value.sequence = params;
+               params = NULL;
                p8->pkeyalg->parameter->type = V_ASN1_SEQUENCE;
 
                break;
@@ -437,32 +473,51 @@ static int dsa_pkey2pkcs8(PKCS8_PRIV_KEY_INFO *p8, EVP_PKEY *pkey)
                case PKCS8_NS_DB:
 
                p8->pkeyalg->parameter->value.sequence = params;
+               params = NULL;
                p8->pkeyalg->parameter->type = V_ASN1_SEQUENCE;
-               ndsa = sk_ASN1_TYPE_new_null();
-               ttmp = ASN1_TYPE_new();
-               if (!(ttmp->value.integer = BN_to_ASN1_INTEGER (pkey->pkey.dsa->pub_key, NULL))) {
+               if (!(ndsa = sk_ASN1_TYPE_new_null())) {
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+                       goto err;
+               }
+               if (!(ttmp = ASN1_TYPE_new())) {
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+                       goto err;
+               }
+               if (!(ttmp->value.integer =
+                       BN_to_ASN1_INTEGER(pkey->pkey.dsa->pub_key, NULL))) {
                        EVPerr(EVP_F_EVP_PKEY2PKCS8,EVP_R_ENCODE_ERROR);
-                       PKCS8_PRIV_KEY_INFO_free(p8);
-                       return 0;
+                       goto err;
                }
                ttmp->type = V_ASN1_INTEGER;
-               sk_ASN1_TYPE_push(ndsa, ttmp);
+               if (!sk_ASN1_TYPE_push(ndsa, ttmp)) {
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+                       goto err;
+               }
 
-               ttmp = ASN1_TYPE_new();
+               if (!(ttmp = ASN1_TYPE_new())) {
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+                       goto err;
+               }
                ttmp->value.integer = prkey;
+               prkey = NULL;
                ttmp->type = V_ASN1_INTEGER;
-               sk_ASN1_TYPE_push(ndsa, ttmp);
+               if (!sk_ASN1_TYPE_push(ndsa, ttmp)) {
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+                       goto err;
+               }
+               ttmp = NULL;
 
-               p8->pkey->value.octet_string = ASN1_OCTET_STRING_new();
+               if (!(p8->pkey->value.octet_string = ASN1_OCTET_STRING_new())) {
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+                       goto err;
+               }
 
                if (!ASN1_seq_pack_ASN1_TYPE(ndsa, i2d_ASN1_TYPE,
                                         &p8->pkey->value.octet_string->data,
                                         &p8->pkey->value.octet_string->length)) {
 
                        EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
-                       sk_ASN1_TYPE_pop_free(ndsa, ASN1_TYPE_free);
-                       M_ASN1_INTEGER_free(prkey);
-                       return 0;
+                       goto err;
                }
                sk_ASN1_TYPE_pop_free(ndsa, ASN1_TYPE_free);
                break;
@@ -470,238 +525,187 @@ static int dsa_pkey2pkcs8(PKCS8_PRIV_KEY_INFO *p8, EVP_PKEY *pkey)
                case PKCS8_EMBEDDED_PARAM:
 
                p8->pkeyalg->parameter->type = V_ASN1_NULL;
-               ndsa = sk_ASN1_TYPE_new_null();
-               ttmp = ASN1_TYPE_new();
+               if (!(ndsa = sk_ASN1_TYPE_new_null())) {
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+                       goto err;
+               }
+               if (!(ttmp = ASN1_TYPE_new())) {
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+                       goto err;
+               }
                ttmp->value.sequence = params;
+               params = NULL;
                ttmp->type = V_ASN1_SEQUENCE;
-               sk_ASN1_TYPE_push(ndsa, ttmp);
+               if (!sk_ASN1_TYPE_push(ndsa, ttmp)) {
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+                       goto err;
+               }
 
-               ttmp = ASN1_TYPE_new();
+               if (!(ttmp = ASN1_TYPE_new())) {
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+                       goto err;
+               }
                ttmp->value.integer = prkey;
+               prkey = NULL;
                ttmp->type = V_ASN1_INTEGER;
-               sk_ASN1_TYPE_push(ndsa, ttmp);
+               if (!sk_ASN1_TYPE_push(ndsa, ttmp)) {
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+                       goto err;
+               }
+               ttmp = NULL;
 
-               p8->pkey->value.octet_string = ASN1_OCTET_STRING_new();
+               if (!(p8->pkey->value.octet_string = ASN1_OCTET_STRING_new())) {
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
+                       goto err;
+               }
 
                if (!ASN1_seq_pack_ASN1_TYPE(ndsa, i2d_ASN1_TYPE,
                                         &p8->pkey->value.octet_string->data,
                                         &p8->pkey->value.octet_string->length)) {
 
                        EVPerr(EVP_F_EVP_PKEY2PKCS8,ERR_R_MALLOC_FAILURE);
-                       sk_ASN1_TYPE_pop_free(ndsa, ASN1_TYPE_free);
-                       M_ASN1_INTEGER_free (prkey);
-                       return 0;
+                       goto err;
                }
                sk_ASN1_TYPE_pop_free(ndsa, ASN1_TYPE_free);
                break;
        }
        return 1;
+err:
+       if (p != NULL) OPENSSL_free(p);
+       if (params != NULL) ASN1_STRING_free(params);
+       if (prkey != NULL) M_ASN1_INTEGER_free(prkey);
+       if (ttmp != NULL) ASN1_TYPE_free(ttmp);
+       if (ndsa != NULL) sk_ASN1_TYPE_pop_free(ndsa, ASN1_TYPE_free);
+       return 0;
 }
 #endif
 
-#ifndef OPENSSL_NO_ECDSA
-static int ecdsa_pkey2pkcs8(PKCS8_PRIV_KEY_INFO *p8, EVP_PKEY *pkey)
+#ifndef OPENSSL_NO_EC
+static int eckey_pkey2pkcs8(PKCS8_PRIV_KEY_INFO *p8, EVP_PKEY *pkey)
 {
-       ASN1_STRING       *params=NULL;
-       ASN1_INTEGER      *prkey=NULL;
-       ASN1_TYPE         *ttmp=NULL;
-       STACK_OF(ASN1_TYPE) *necdsa=NULL;
-       unsigned char     *p=NULL, *q=NULL;
-       int len=0;
-       EC_POINT          *point=NULL;
-
-       if (pkey->pkey.ecdsa == NULL || pkey->pkey.ecdsa->group == NULL)
-       {
-               EVPerr(EVP_F_ECDSA_PKEY2PKCS8, EVP_R_MISSING_PARAMETERS);
-               return 0;
-       }
-       p8->pkeyalg->algorithm = OBJ_nid2obj(NID_ecdsa_with_SHA1);
-       len = i2d_ECDSAParameters(pkey->pkey.ecdsa, NULL);
-       if ((p = OPENSSL_malloc(len)) == NULL)
+       EC_KEY          *eckey;
+       unsigned char   *p, *pp;
+       int             nid, i, ret = 0;
+       unsigned int    tmp_flags;
+
+       if (pkey->pkey.eckey == NULL || pkey->pkey.eckey->group == NULL)
        {
-               EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
+               EVPerr(EVP_F_EC_KEY_PKEY2PKCS8, EVP_R_MISSING_PARAMETERS);
                return 0;
        }
-       q = p;
-       if (!i2d_ECDSAParameters(pkey->pkey.ecdsa, &q))
+       eckey = pkey->pkey.eckey;
+
+       /* set the ec parameters OID */
+       if (p8->pkeyalg->algorithm)
+               ASN1_OBJECT_free(p8->pkeyalg->algorithm);
+
+       p8->pkeyalg->algorithm = OBJ_nid2obj(NID_X9_62_id_ecPublicKey);
+
+       /* set the ec parameters */
+
+       if (p8->pkeyalg->parameter)
        {
-               EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_ECDSA_LIB);
-               OPENSSL_free(p);
-               return 0;
+               ASN1_TYPE_free(p8->pkeyalg->parameter);
+               p8->pkeyalg->parameter = NULL;
        }
-       if ((params = ASN1_STRING_new()) == NULL)
+
+       if ((p8->pkeyalg->parameter = ASN1_TYPE_new()) == NULL)
        {
-               EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
-               OPENSSL_free(p);
+               EVPerr(EVP_F_EC_KEY_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
                return 0;
-               
        }
-       if (!ASN1_STRING_set(params, p, len))
+       
+       if (EC_GROUP_get_asn1_flag(eckey->group)
+                     && (nid = EC_GROUP_get_nid(eckey->group)))
        {
-               EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_ASN1_LIB);
-               OPENSSL_free(p);
-               return 0;
+               /* we have a 'named curve' => just set the OID */
+               p8->pkeyalg->parameter->type = V_ASN1_OBJECT;
+               p8->pkeyalg->parameter->value.object = OBJ_nid2obj(nid);
        }
-       OPENSSL_free(p);
-       if ((prkey = BN_to_ASN1_INTEGER(pkey->pkey.ecdsa->priv_key, NULL)) == NULL)
+       else    /* explicit parameters */
        {
-               EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_ASN1_LIB);
-               return 0;
-       }
-
-       switch(p8->broken) {
-
-               case PKCS8_OK:
-               case PKCS8_NO_OCTET:
-
-               if (!ASN1_pack_string((char *)prkey, i2d_ASN1_INTEGER,
-                                        &p8->pkey->value.octet_string)) 
-               {
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
-                       M_ASN1_INTEGER_free(prkey);
-                       return 0;
-               }
-
-               ASN1_INTEGER_free(prkey);
-               p8->pkeyalg->parameter->value.sequence = params;
-               p8->pkeyalg->parameter->type = V_ASN1_SEQUENCE;
-
-               break;
-
-               case PKCS8_NS_DB:
-
-               p8->pkeyalg->parameter->value.sequence = params;
-               p8->pkeyalg->parameter->type = V_ASN1_SEQUENCE;
-               necdsa = sk_ASN1_TYPE_new_null();
-               if (necdsa == NULL || (ttmp = ASN1_TYPE_new()) == NULL)
+               if ((i = i2d_ECParameters(eckey, NULL)) == 0)
                {
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
-                       sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
+                       EVPerr(EVP_F_EC_KEY_PKEY2PKCS8, ERR_R_EC_LIB);
                        return 0;
                }
-
-               if ((point = EC_GROUP_get0_generator(pkey->pkey.ecdsa->group)) == NULL)
+               if ((p = (unsigned char *) OPENSSL_malloc(i)) == NULL)
                {
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_EC_LIB);
+                       EVPerr(EVP_F_EC_KEY_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
                        return 0;
-               }
-               len = EC_POINT_point2oct(pkey->pkey.ecdsa->group, point, ECDSA_get_conversion_form(pkey->pkey.ecdsa),
-                                        NULL, 0, NULL);
-               p = OPENSSL_malloc(len);
-               if (!len || !p || !EC_POINT_point2oct(pkey->pkey.ecdsa->group, point,
-                       ECDSA_get_conversion_form(pkey->pkey.ecdsa), p, len, NULL))
+               }       
+               pp = p;
+               if (!i2d_ECParameters(eckey, &pp))
                {
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_EC_LIB);
+                       EVPerr(EVP_F_EC_KEY_PKEY2PKCS8, ERR_R_EC_LIB);
                        OPENSSL_free(p);
                        return 0;
                }
-               if ((ttmp->value.octet_string = ASN1_OCTET_STRING_new()) == NULL)
-               {
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
-                       return 0;
-               }
-               if (!ASN1_OCTET_STRING_set(ttmp->value.octet_string, p, len))
+               p8->pkeyalg->parameter->type = V_ASN1_SEQUENCE;
+               if ((p8->pkeyalg->parameter->value.sequence 
+                       = ASN1_STRING_new()) == NULL)
                {
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, EVP_R_ASN1_LIB);
+                       EVPerr(EVP_F_EC_KEY_PKEY2PKCS8, ERR_R_ASN1_LIB);
+                       OPENSSL_free(p);
                        return 0;
                }
+               ASN1_STRING_set(p8->pkeyalg->parameter->value.sequence, p, i);
                OPENSSL_free(p);
-               
-               ttmp->type = V_ASN1_OCTET_STRING;
-               if (!sk_ASN1_TYPE_push(necdsa, ttmp))
-               {
-                       sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
-                       ASN1_INTEGER_free(prkey);
-                       return 0;
-               }
-
-               if ((ttmp = ASN1_TYPE_new()) == NULL)
-               {
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
-                       return 0;
-               }
-               ttmp->value.integer = prkey;
-               ttmp->type = V_ASN1_INTEGER;
-               if (!sk_ASN1_TYPE_push(necdsa, ttmp))
-               {
-                       sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
-                       ASN1_INTEGER_free(prkey);
-                       return 0;
-               }
-
-               if ((p8->pkey->value.octet_string = ASN1_OCTET_STRING_new()) == NULL)
-               {       
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
-                       sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
-                       return 0;
-               }
-
-               if (!ASN1_seq_pack_ASN1_TYPE(necdsa, i2d_ASN1_TYPE,
-                                        &p8->pkey->value.octet_string->data,
-                                        &p8->pkey->value.octet_string->length)) 
-               {
-
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
-                       sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
-                       return 0;
-               }
-               sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
-               break;
+       }
 
-               case PKCS8_EMBEDDED_PARAM:
+       /* set the private key */
 
-               p8->pkeyalg->parameter->type = V_ASN1_NULL;
-               necdsa = sk_ASN1_TYPE_new_null();
-               if ((ttmp = ASN1_TYPE_new()) == NULL)
-               {
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
-                       sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
-                       ASN1_INTEGER_free(prkey);
-                       return 0;
-               }
-               ttmp->value.sequence = params;
-               ttmp->type = V_ASN1_SEQUENCE;
-               if (!sk_ASN1_TYPE_push(necdsa, ttmp))
-               {
-                       sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
-                       ASN1_INTEGER_free(prkey);
-                       return 0;
-               }
+       /* do not include the parameters in the SEC1 private key
+        * see PKCS#11 12.11 */
+       tmp_flags  = pkey->pkey.eckey->enc_flag;
+       pkey->pkey.eckey->enc_flag |= EC_PKEY_NO_PARAMETERS;
+       i = i2d_ECPrivateKey(pkey->pkey.eckey, NULL);
+       if (!i)
+       {
+               pkey->pkey.eckey->enc_flag = tmp_flags;
+               EVPerr(EVP_F_EC_KEY_PKEY2PKCS8, ERR_R_EC_LIB);
+               return 0;
+       }
+       p = (unsigned char *) OPENSSL_malloc(i);
+       if (!p)
+       {
+               pkey->pkey.eckey->enc_flag = tmp_flags;
+               EVPerr(EVP_F_EC_KEY_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
+               return 0;
+       }
+       pp = p;
+       if (!i2d_ECPrivateKey(pkey->pkey.eckey, &pp))
+       {
+               pkey->pkey.eckey->enc_flag = tmp_flags;
+               EVPerr(EVP_F_EC_KEY_PKEY2PKCS8, ERR_R_EC_LIB);
+               OPENSSL_free(p);
+               return 0;
+       }
+       /* restore old encoding flags */
+       pkey->pkey.eckey->enc_flag = tmp_flags;
 
-               if ((ttmp = ASN1_TYPE_new()) == NULL)
-               {
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
-                       sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
-                       ASN1_INTEGER_free(prkey);
-                       return 0;
-               }
-               ttmp->value.integer = prkey;
-               ttmp->type = V_ASN1_INTEGER;
-               if (!sk_ASN1_TYPE_push(necdsa, ttmp))
-               {
-                       sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
-                       ASN1_INTEGER_free(prkey);
-                       return 0;
-               }
+       switch(p8->broken) {
 
-               if ((p8->pkey->value.octet_string = ASN1_OCTET_STRING_new()) == NULL)
-               {
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
-                       sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
-                       return 0;
-               }
+               case PKCS8_OK:
+               p8->pkey->value.octet_string = ASN1_OCTET_STRING_new();
+               if (!p8->pkey->value.octet_string ||
+                   !M_ASN1_OCTET_STRING_set(p8->pkey->value.octet_string,
+                   (const void *)p, i))
 
-               if (!ASN1_seq_pack_ASN1_TYPE(necdsa, i2d_ASN1_TYPE,
-                                        &p8->pkey->value.octet_string->data,
-                                        &p8->pkey->value.octet_string->length)) 
                {
-                       EVPerr(EVP_F_ECDSA_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
-                       sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
-                       return 0;
+                       EVPerr(EVP_F_EC_KEY_PKEY2PKCS8, ERR_R_MALLOC_FAILURE);
                }
-               sk_ASN1_TYPE_pop_free(necdsa, ASN1_TYPE_free);
+               else
+                       ret = 1;
                break;
+               case PKCS8_NO_OCTET:            /* RSA specific */
+               case PKCS8_NS_DB:               /* DSA specific */
+               case PKCS8_EMBEDDED_PARAM:      /* DSA specific */
+               default:
+                       EVPerr(EVP_F_EVP_PKEY2PKCS8,EVP_R_ENCODE_ERROR);
        }
-       return 1;
+       OPENSSL_cleanse(p, (size_t)i);
+       OPENSSL_free(p);
+       return ret;
 }
 #endif