Commit 93879f8e authored by Jonas Maebe's avatar Jonas Maebe Committed by Matt Caswell
Browse files

cryptodev_asym, zapparams: use OPENSSL_* allocation routines, handle errors 



zapparams modification based on tip from Matt Caswell

RT#3198

Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
Reviewed-by: default avatarMatt Caswell <matt@openssl.org>
parent 2c4a056f

crypto/engine/eng_cryptodev.c

+13 −6
Original line number Diff line number Diff line
@@ -1257,8 +1257,7 @@ static void zapparams(struct crypt_kop *kop) 
    int i;



    for (i = 0; i < kop->crk_iparams + kop->crk_oparams; i++) {

        if (kop->crk_param[i].crp_p)

            free(kop->crk_param[i].crp_p);

        OPENSSL_free(kop->crk_param[i].crp_p);

        kop->crk_param[i].crp_p = NULL;

        kop->crk_param[i].crp_nbits = 0;

    }
@@ -1271,16 +1270,24 @@ cryptodev_asym(struct crypt_kop *kop, int rlen, BIGNUM *r, int slen, 
    int fd, ret = -1;



    if ((fd = get_asym_dev_crypto()) < 0)

        return (ret);

        return ret;



    if (r) {

        kop->crk_param[kop->crk_iparams].crp_p = calloc(rlen, sizeof(char));

        kop->crk_param[kop->crk_iparams].crp_p = OPENSSL_zalloc(rlen);

        if (kop->crk_param[kop->crk_iparams].crp_p == NULL)

            return ret;

        kop->crk_param[kop->crk_iparams].crp_nbits = rlen * 8;

        kop->crk_oparams++;

    }

    if (s) {

        kop->crk_param[kop->crk_iparams + 1].crp_p =

            calloc(slen, sizeof(char));

            OPENSSL_zalloc(slen);

        /* No need to free the kop->crk_iparams parameter if it was allocated,

         * callers of this routine have to free allocated parameters through

         * zapparams both in case of success and failure

         */

        if (kop->crk_param[kop->crk_iparams+1].crp_p == NULL)

            return ret;

        kop->crk_param[kop->crk_iparams + 1].crp_nbits = slen * 8;

        kop->crk_oparams++;

    }
@@ -1293,7 +1300,7 @@ cryptodev_asym(struct crypt_kop *kop, int rlen, BIGNUM *r, int slen, 
        ret = 0;

    }



    return (ret);

    return ret;

}



static int