diff options
| author | Ryan Salsamendi <rsalsamendi@hotmail.com> | 2017-06-09 12:01:39 -0700 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2017-06-14 19:51:12 +0200 |
| commit | f0b071202377e3e4c5550ddaa28a727556745fa7 (patch) | |
| tree | c0c239caf4cba7beb00ceea61a401eb9d1f7b220 /src/lib/hexstrtobin.c | |
| parent | cde2bdf496408f7083448d5805114360ba7bef8c (diff) | |
arch/x86: Fix undefined behavior
Fixes report found by undefined behavior sanitizer. Dereferencing a
pointer that is not aligned to the size of access is undefined behavior.
Switch to memcpy() for unaligned write to EBDA_LOWMEM. Change other
write16()s in setup_ebda() to memcpy() for consistency.
Change-Id: I79814bd47a14ec59d84068b11d094dc2531995d9
Signed-off-by: Ryan Salsamendi <rsalsamendi@hotmail.com>
Reviewed-on: https://review.coreboot.org/20132
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com>
Diffstat (limited to 'src/lib/hexstrtobin.c')
0 files changed, 0 insertions, 0 deletions