diff options
| author | Sami Kerola | 2019-07-27 19:55:01 +0200 |
|---|---|---|
| committer | Sami Kerola | 2019-07-27 20:00:34 +0200 |
| commit | b5f376d11d9f9e25be32920705bd5ee54684ca36 (patch) | |
| tree | 3d5a956b9fbec0fcd6039fff81a710bf8c7bfccc | |
| parent | lscpu: (man) add note about cache sizes (diff) | |
| download | kernel-qcow2-util-linux-b5f376d11d9f9e25be32920705bd5ee54684ca36.tar.gz kernel-qcow2-util-linux-b5f376d11d9f9e25be32920705bd5ee54684ca36.tar.xz kernel-qcow2-util-linux-b5f376d11d9f9e25be32920705bd5ee54684ca36.zip | |
lscpu: prefer memcpy() to manual pointer arithmetic
With pointer arithmetic clang address sanitizer gives following error this
change addresses. Notice the following happens only when running as root.
sys-utils/lscpu-dmi.c:83:14: runtime error: load of misaligned address
0x55a1d62f3d1d for type 'const uint16_t' (aka 'const unsigned short'), which
requires 2 byte alignment
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
| -rw-r--r-- | sys-utils/lscpu-dmi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys-utils/lscpu-dmi.c b/sys-utils/lscpu-dmi.c index 29bd2e4fc..82d8ed443 100644 --- a/sys-utils/lscpu-dmi.c +++ b/sys-utils/lscpu-dmi.c @@ -80,7 +80,7 @@ static void to_dmi_header(struct dmi_header *h, uint8_t *data) { h->type = data[0]; h->length = data[1]; - h->handle = WORD(data + 2); + memcpy(&h->handle, data + 2, sizeof(h->handle)); h->data = data; } |