Commit 3318493b authored by Jeff Mahoney's avatar Jeff Mahoney Committed by Paul Yang
Browse files

apps/speed: fix segfault while looking up algorithm name 



The backport of master commit 5c6a69f5 (apps/speed: fix possible OOB
access in some EC arrays) as 1.1.0 commit 4e079413 introduced a
regression.  The ecdh_choices array is iterated using an element count
but is NULL terminated.  This means that running 'openssl speed somealgo'
will result in a segfault when opt_found hits the NULL entry.

Fixes #8243

CLA: trivial

Signed-off-by: default avatarJeff Mahoney <jeffm@suse.com>

Reviewed-by: default avatarPaul Yang <yang.yang@baishancloud.com>
Reviewed-by: default avatarMatt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/8244)
parent f499873c

apps/speed.c

+0 −1
Original line number Diff line number Diff line
@@ -537,7 +537,6 @@ static const OPT_PAIR ecdh_choices[] = { 
    {"ecdhb409", R_EC_B409},

    {"ecdhb571", R_EC_B571},

    {"ecdhx25519", R_EC_X25519},

    {NULL}

};

# define EC_NUM       OSSL_NELEM(ecdh_choices)