diff options
| author | Adam Langley <agl@google.com> | 2016-02-29 08:01:05 -0800 |
|---|---|---|
| committer | The Android Automerger <android-build@google.com> | 2016-05-27 11:30:00 -0700 |
| commit | 74750e1fb24149043a533497f79c577b704d6e30 (patch) | |
| tree | 2f563fe617a1af6a4cc77be3d62ae7465ed54e62 /src | |
| parent | 591be84e89682622957c8f103ca4be3a5ed0f800 (diff) | |
| download | external_boringssl-74750e1fb24149043a533497f79c577b704d6e30.zip external_boringssl-74750e1fb24149043a533497f79c577b704d6e30.tar.gz external_boringssl-74750e1fb24149043a533497f79c577b704d6e30.tar.bz2 | |
Fix encoding bug in i2c_ASN1_INTEGER
(Imported from upstream's 3661bb4e7934668bd99ca777ea8b30eedfafa871.)
Fix bug where i2c_ASN1_INTEGER mishandles zero if it is marked as
negative.
Thanks to Huzaifa Sidhpurwala <huzaifas@redhat.com> and Hanno Böck
<hanno@hboeck.de> for reporting this issue.
BUG=590615
(cherry-picked from c4eec0c16b02c97a62a95b6a08656c3a9ddb6baa)
Bug: 28175332
Change-Id: I8959e8ae01510a5924862a3f353be23130eee554
Reviewed-on: https://boringssl-review.googlesource.com/7199
Reviewed-by: David Benjamin <davidben@google.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/crypto/asn1/a_int.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/crypto/asn1/a_int.c b/src/crypto/asn1/a_int.c index 2ecccc5..16b8950 100644 --- a/src/crypto/asn1/a_int.c +++ b/src/crypto/asn1/a_int.c @@ -125,6 +125,8 @@ int i2c_ASN1_INTEGER(ASN1_INTEGER *a, unsigned char **pp) { ret=a->length; i=a->data[0]; + if (ret == 1 && i == 0) + neg=0; if (!neg && (i > 127)) { pad=1; pb=0; @@ -158,7 +160,7 @@ int i2c_ASN1_INTEGER(ASN1_INTEGER *a, unsigned char **pp) p += a->length - 1; i = a->length; /* Copy zeros to destination as long as source is zero */ - while(!*n) { + while(!*n && i > 1) { *(p--) = 0; n--; i--; |