diff options
| author | Paul Selkirk <paul@psgd.org> | 2018-07-24 18:21:44 -0400 |
|---|---|---|
| committer | Paul Selkirk <paul@psgd.org> | 2018-07-24 18:21:44 -0400 |
| commit | 222ec2b0d1ab78b142ad00d5969975c64801eeab (patch) | |
| tree | b200f650d95bd16fdcda93b6329c2ffd998f8d3c /cryptech_backup | |
| parent | e1a2a7ff2e240c118b62fd372254e8f8097bd416 (diff) | |
A recent(?) version of arm-none-eabi-gcc decided to make storage for enums
the minimum size necessary, so hal_asn1_decode_lms_algorithm and
hal_asn1_decode_lmots_algorithm were writing 4 bytes of data into 1-byte
variables. Hilarity ensued. Yes, I already knew that conflating enum with
uint32_t was a bad idea, I was just being lazy.
For that matter, sizeof(size_t) isn't guaranteed either, although
arm-none-eabi-gcc treats it as 32 bits on this 32-bit target (for now), so
exercise proper data hygiene in hal_asn1_decode_size_t as well.
Diffstat (limited to 'cryptech_backup')
0 files changed, 0 insertions, 0 deletions