tree 56d10a1040d22b970a34ee420012916b9d1a23f2
parent c93be52c9ef59801704a80bc39ddcfe3bd4277ed
author David Benjamin <davidben@google.com> 1552609541 -0500
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1552936711 +0000

Rewrite BN_CTX.

While allocating near INT_MAX BIGNUMs or stack frames would never happen, we
should properly handle overflow here. Rewrite it to just be a STACK_OF(BIGNUM)
plus a stack of indices. Also simplify the error-handling. If we make the
errors truly sticky (rather than just sticky per frame), we don't need to keep
track of err_stack and friends.

Thanks to mlbrown for reporting the integer overflows in the original
implementation.

Bug: chromium:942269
Change-Id: Ie9c9baea3eeb82d65d88b1cb1388861f5cd84fe5
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/35328
Commit-Queue: Adam Langley <agl@google.com>
Reviewed-by: Adam Langley <agl@google.com>