d9f49974e3
This is a reland of https://boringssl-review.googlesource.com/2330. I believe I've now cleared the fallout. Android's attestion format uses some ludicrously large tag numbers: https://developer.android.com/training/articles/security-key-attestation.html#certificate_schema Add support for these in CBS/CBB. The public API does not change for callers who were using the CBS_ASN1_* constants, but it is no longer the case that tag representations match their DER encodings for small tag numbers. When passing tags into CBS/CBB, use CBS_ASN1_* constants. When working with DER byte arrays (most commonly test vectors), use the numbers themselves. Bug: 214 Update-Note: The in-memory representation of CBS/CBB tags changes. Additionally, we now support tag numbers above 30. I believe I've now actually cleared the fallout of the former. There is one test in Chromium and the same test in the internal repository that needs fixing. Change-Id: I49b9d30df01f023c646d31156360ff69c91626a3 Reviewed-on: https://boringssl-review.googlesource.com/24404 Commit-Queue: Adam Langley <agl@google.com> Reviewed-by: Adam Langley <agl@google.com> CQ-Verified: CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> |
||
|---|---|---|
| .. | ||
| asn1_compat.c | ||
| ber.c | ||
| bytestring_test.cc | ||
| cbb.c | ||
| cbs.c | ||
| CMakeLists.txt | ||
| internal.h | ||