tree 992632518f5d942d9e8adbb340dbd84587083eca
parent f8d05579b4c529a9132ece8e2ab7ae70cee9321e
author Adam Langley <alangley@gmail.com> 1514518741 -0800
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1514909491 +0000

Fix |ASN1_INTEGER_set| when setting zero.

|ASN1_INTEGER_set| and |BN_to_ASN1_INTEGER| disagree about how to encode
zero. OpenSSL master has aligned around the behaviour of the latter
(i.e. a single zero byte) so fix |ASN1_INTEGER_set| to do that. (This is
also the form that DER requires.)

At the same time, fix undefined behaviour when negative a |long| whose
value is |LONG_MIN|.

Change-Id: I1198de35e61a286ac6472e99152f3d22fda59044
Reviewed-on: https://boringssl-review.googlesource.com/24485
Commit-Queue: Adam Langley <agl@google.com>
CQ-Verified: CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org>
Reviewed-by: David Benjamin <davidben@google.com>