Commit cc39f925 authored by Andy Polyakov's avatar Andy Polyakov Committed by Richard Levitte
Browse files

ec/ec_mult.c: get BN_CTX_start,end sequence right. 



Triggered by Coverity analysis.

Reviewed-by: default avatarRich Salz <rsalz@openssl.org>

(cherry picked from commit 7d859d1c)

Reviewed-by: default avatarBernd Edlinger <bernd.edlinger@hotmail.de>
Reviewed-by: default avatarPaul Dale <paul.dale@oracle.com>
Reviewed-by: default avatarRichard Levitte <levitte@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/6549)
parent 77b6b171

crypto/ec/ec_mult.c

+3 −2
Original line number Diff line number Diff line
@@ -144,7 +144,9 @@ static int ec_mul_consttime(const EC_GROUP *group, EC_POINT *r, 
    int ret = 0;



    if (ctx == NULL && (ctx = new_ctx = BN_CTX_secure_new()) == NULL)

        goto err;

        return 0;



    BN_CTX_start(ctx);



    order_bits = BN_num_bits(group->order);
@@ -162,7 +164,6 @@ static int ec_mul_consttime(const EC_GROUP *group, EC_POINT *r, 


    EC_POINT_BN_set_flags(s, BN_FLG_CONSTTIME);



    BN_CTX_start(ctx);

    lambda = BN_CTX_get(ctx);

    k = BN_CTX_get(ctx);

    if (k == NULL)