tree 8bbc8afbf8f33d6da2e8079c7606d7c89308e9a5
parent 081e3f34a2b324edce50b7a5df9b2e283781af7b
author Brian Smith <brian@briansmith.org> 1457492980 -1000
committer David Benjamin <davidben@google.com> 1457550276 +0000

Avoid potential uninitialized memory read in crypto/ec/p256-x86_64.c.

If the function returns early due to an error, then the coordinates of the
result will have their |top| value set to a value beyond what has actually
been been written. Fix that, and make it easier to avoid such issues in the
future by refactoring the code.

As a bonus, avoid a false positive MSVC 64-bit opt build "potentially
uninitialized value used" warning.

Change-Id: I8c48deb63163a27f739c8797962414f8ca2588cd
Reviewed-on: https://boringssl-review.googlesource.com/6579
Reviewed-by: David Benjamin <davidben@google.com>