tree 77ede5c257333fd7980f3f110bec466fcccb2568
parent 49e9f67d8b7cbeb3953b5548ad1009d15947a523
author David Benjamin <davidben@google.com> 1598737395 -0400
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1600369757 +0000

Avoid unions in X509_NAME logic.

Cast between T* and ASN1_VALUE* directly, rather than messing with
unions. This is necessary to avoid UB in C++ and is a strict aliasing
violation in C too. (While C does allow type-punning in unions, pointers
to union fields have weird rules. I suspect most of our unions should be
reworked.)

Change-Id: Ibe274a7df5d5826f8c5f335255d196e9b7587105
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/42726
Commit-Queue: David Benjamin <davidben@google.com>
Reviewed-by: Adam Langley <agl@google.com>