Remove brace from bad cherry-pick of DSA reallocation fix
authorRod Vagg <rod@vagg.org>
Mon, 29 Oct 2018 09:43:53 +0000 (20:43 +1100)
committerRod Vagg <rod@vagg.org>
Tue, 30 Oct 2018 02:52:55 +0000 (13:52 +1100)
Commit 56fb454 backported the DSA reallocation fix to 1.1.0, however a
code block that has multiple statements in 1.1.1+ only has a `goto` in
1.1.0 so introduces a brace that causes a compile failure.

CLA:trivial

Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from https://github.com/openssl/openssl/pull/7516)

crypto/ec/ec_mult.c

index ff882cce20e26003d189a438b23439e99d045bcb..8350082eb42e35c06cf3b58eca6b3a0556fab28f 100644 (file)
@@ -178,7 +178,7 @@ static int ec_mul_consttime(const EC_GROUP *group, EC_POINT *r,
     cardinality_bits = BN_num_bits(cardinality);
     group_top = bn_get_top(cardinality);
     if ((bn_wexpand(k, group_top + 2) == NULL)
-        || (bn_wexpand(lambda, group_top + 2) == NULL)) {
+        || (bn_wexpand(lambda, group_top + 2) == NULL))
         goto err;
 
     if (!BN_copy(k, scalar))