diff options
| author | Evan Benn <evanbenn@chromium.org> | 2022-10-05 11:47:53 +1100 |
|---|---|---|
| committer | Felix Singer <felixsinger@posteo.net> | 2022-10-08 20:29:47 +0000 |
| commit | 634da83f403a86702670514447c8e76466cbcd75 (patch) | |
| tree | fe5f1170fec347d3a452817b8754fab18ec728a4 /dmi.c | |
| parent | 7c40ba24fe73f820370d99a35ff071d667ef7c74 (diff) | |
| download | flashrom-634da83f403a86702670514447c8e76466cbcd75.tar.gz flashrom-634da83f403a86702670514447c8e76466cbcd75.tar.bz2 flashrom-634da83f403a86702670514447c8e76466cbcd75.zip | |
s25f.c: Fix undefined behaviour on shift
dev_id, a uint8_t, was shifted left by 24 bits. After promotion to int,
this results in shifting into the sign bit, which is undefined
behaviour. Cast to uint32_t to prevent the promotion to signed int.
BUG=None
BRANCH=None
TEST=None
Change-Id: I88188ef2ba2af919eeae9ba08916374d31d8b989
Signed-off-by: Evan Benn <evanbenn@chromium.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/68127
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Diffstat (limited to 'dmi.c')
0 files changed, 0 insertions, 0 deletions