diff options
author | Karina Yankevich <[email protected]> | 2024-01-21 20:29:55 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <[email protected]> | 2024-01-27 16:44:00 -0800 |
commit | d6429a3555fb29f380c5841a12f5ac3f7444af03 (patch) | |
tree | 8032a319d6d49a245975c2563152cae6fec4b531 /tools/perf/scripts/python/gecko.py | |
parent | 7626c52b6b461b44a46cea4d51b4cf749657f5a8 (diff) |
usb: storage: sddr55: fix sloppy typing in sddr55_{read|write}_data()
In sddr55_{read|write}_data(), the address variables are needlessly typed
as *unsigned long* -- which is 32-bit type on the 32-bit arches and 64-bit
type on the 64-bit arches; those variables' value should fit into just 3
command bytes and consists of 10-bit block # (or at least the max block #
seems to be 1023) and 4-/5-bit page # within a block, so 32-bit *unsigned*
*int* type should be more than enough...
Found by Linux Verification Center (linuxtesting.org) with the Svace static
analysis tool.
[Sergey: rewrote the patch subject/description]
Signed-off-by: Karina Yankevich <[email protected]>
Signed-off-by: Sergey Shtylyov <[email protected]>
Reviewed-by: Alan Stern <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Greg Kroah-Hartman <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/gecko.py')
0 files changed, 0 insertions, 0 deletions