Switch ossl_ecdsa_verify() to timingsafe_memcmp()
authortb <tb@openbsd.org>
Mon, 3 Jul 2023 09:59:20 +0000 (09:59 +0000)
committertb <tb@openbsd.org>
Mon, 3 Jul 2023 09:59:20 +0000 (09:59 +0000)
Requested by jsing

lib/libcrypto/ecdsa/ecs_ossl.c

index 7e03c23..83be5fd 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: ecs_ossl.c,v 1.49 2023/07/03 09:55:42 tb Exp $ */
+/* $OpenBSD: ecs_ossl.c,v 1.50 2023/07/03 09:59:20 tb Exp $ */
 /*
  * Written by Nils Larsch for the OpenSSL project
  */
@@ -440,7 +440,7 @@ ossl_ecdsa_verify(int type, const unsigned char *dgst, int dgst_len,
        /* Ensure signature uses DER and doesn't have trailing garbage */
        if ((derlen = i2d_ECDSA_SIG(s, &der)) != sig_len)
                goto err;
-       if (memcmp(sigbuf, der, derlen))
+       if (timingsafe_memcmp(sigbuf, der, derlen))
                goto err;
 
        ret = ECDSA_do_verify(dgst, dgst_len, s, eckey);