[coreboot-gerrit] [S] Change in coreboot[main]: commonlib/bsd/lz4_wrapper.c: Fix misaligned access

[Angel Pons (Code Review)]

Attention is currently required from: Arthur Heymans, Julius Werner, Maximilian Brune, Nico Huber, Philipp Hug, Ron Minnich.

Angel Pons has posted comments on this change by Maximilian Brune. ( https://review.coreboot.org/c/coreboot/+/81910?usp=email )

Change subject: commonlib/bsd/lz4_wrapper.c: Fix misaligned access
......................................................................

Patch Set 5: Code-Review+1

(1 comment)

File src/commonlib/bsd/lz4_wrapper.c:

https://review.coreboot.org/c/coreboot/+/81910/comment/748ac5cc_339ff531?usp... :
PS5, Line 14: static uint16_t LZ4_readLE16(const void *src)
            : {
            : 	return *((const uint8_t *)src + 1) << 8
            : 	      | *(const uint8_t *)src;
            : }
            : 
            : static uint32_t LZ4_readLE32(const void *src)
            : {
            : 	return *((const uint8_t *)src + 3) << 24
            : 	     | *((const uint8_t *)src + 2) << 16
            : 	     | *((const uint8_t *)src + 1) << 8
            : 	     |  *(const uint8_t *)src;
            : }
> +1 to Angel's suggested edit

Given Nico's tests, I don't think there's any need to overcomplicate this.