diff options
| author | David Benjamin <davidben@chromium.org> | 2014-11-03 01:59:32 +0300 |
|---|---|---|
| committer | Adam Langley <agl@google.com> | 2014-11-06 04:34:33 +0300 |
| commit | 9ecafa5c78846cff77dd569403bcf3534bf20961 (patch) | |
| tree | 6cce43baaf12fd989400c6a3c9119a519fbbeefc /crypto/ec/wnaf.c | |
| parent | f2b32a2de2c0bb4f93e009ea409cfe6f1139f44e (diff) | |
Shush some dead assignments.
Appease clang scan-build a bit. I'm not sure it's actually worth silencing all
of them because some of them look like preserving invariants between local
variables, but some are clearly pointless or can be restructured slightly.
Change-Id: I0bc81e2589bb402ff3ef0182d7a8921e31b85052
Reviewed-on: https://boringssl-review.googlesource.com/2205
Reviewed-by: Adam Langley <agl@google.com>
Diffstat (limited to 'crypto/ec/wnaf.c')
| -rw-r--r-- | crypto/ec/wnaf.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/crypto/ec/wnaf.c b/crypto/ec/wnaf.c index 2fed4a57..cac7234c 100644 --- a/crypto/ec/wnaf.c +++ b/crypto/ec/wnaf.c @@ -443,8 +443,8 @@ int ec_wNAF_mul(const EC_GROUP *group, EC_POINT *r, const BIGNUM *scalar, * as the wNAF belonging to the generator, * so wNAF splitting will not buy us anything. */ - numblocks = 1; - totalnum = num + 1; /* don't use wNAF splitting */ + numblocks = 1; /* don't use wNAF splitting */ + totalnum = num + numblocks; wNAF[num] = tmp_wNAF; wNAF[num + 1] = NULL; wNAF_len[num] = tmp_len; |