From f6b270b012b8c52d8140b40d73282d3114d53004 Mon Sep 17 00:00:00 2001 From: Neil Horman Date: Fri, 26 Apr 2024 12:38:38 -0400 Subject: [PATCH] Fix alignment errors in hashtable fuzzer we extract several values (uint16_t and uint64_t from the fuzzer buff passed in, but they weren't aligned on 2 and 8 byte boundaries. Adjust the fuzzer to memcpy data to the target variables to avoid unalignment issues Fixes #24272 --- fuzz/hashtable.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fuzz/hashtable.c b/fuzz/hashtable.c index 64a736d815193..35cf9c8f3ba74 100644 --- a/fuzz/hashtable.c +++ b/fuzz/hashtable.c @@ -142,7 +142,7 @@ int FuzzerTestOneInput(const uint8_t *buf, size_t len) * parse out our operation flags and key */ op_flags = buf[0]; - keyval = *((uint16_t *)&buf[1]); + memcpy(&keyval, &buf[1], sizeof(uint16_t)); /* * Initialize our key @@ -177,7 +177,7 @@ int FuzzerTestOneInput(const uint8_t *buf, size_t len) rc_prediction = 0; } - valptr->value = *(uint64_t *)&buf[3]; + memcpy(&valptr->value, &buf[3], sizeof(uint64_t)); /* * do the insert/replace */