tree 70ee6d71a8e5d9ded14c7d6612214be3298ec15a
parent 6ff2ba80b758025fdb137d9357dcbba0a9c04dbf
author David Benjamin <davidben@google.com> 1528918566 -0400
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1528919904 +0000

Zero-initialize tmp in ec_GFp_simple_mul_single.

Although the original value of tmp does not matter, the selects
ultimately do bit operations on the uninitialized values and thus depend
on them behaving like *some* consistent concrete value. The C spec
appears to allow uninitialized values to resolve to trap
representations, which means this isn't quite valid..

(If I'm reading it wrong and the compiler must behave as if there were a
consistent value in there, it's probably fine, but there's no sense in
risking compiler bugs on a subtle corner of things.)

Change-Id: Id4547b0ec702414b387e906c4de55595e6214ddb
Reviewed-on: https://boringssl-review.googlesource.com/29124
Commit-Queue: Steven Valdez <svaldez@google.com>
Reviewed-by: Steven Valdez <svaldez@google.com>
CQ-Verified: CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org>