summaryrefslogtreecommitdiff
path: root/include
diff options
context:
space:
mode:
authorMarek Vasut <[email protected]>2022-08-26 23:15:55 +0200
committerTom Rini <[email protected]>2022-08-31 12:21:31 -0400
commit1aa9a04ff687b8d55b0fb68ae2a688c8705665cc (patch)
tree91c54c3e0915ee095705e9d203d1528dff423bed /include
parent6eea9408ac1e2607c231f3049953261b327d7b14 (diff)
Revert "i2c: fix stack buffer overflow vulnerability in i2c md command"
This reverts commit 8f8c04bf1ebbd2f72f1643e7ad9617dafa6e5409. The commit is largely wrong and breaks most of i2c command functionality. The problem described in the aforementioned commit commit message is valid, however the commit itself does many more changes unrelated to fixing that one problem it describes. Those extra changes, namely the handling of i2c device address length as unsigned instead of signed integer, breaks the expectation that address length may be negative value. The negative value is used by DM to indicate that address length of device does not change. The actual bug documented in commit 8f8c04bf1ebbd2f72f1643e7ad9617dafa6e5409 can be fixed by extra sanitization in separate patch. Signed-off-by: Marek Vasut <[email protected]> Cc: Heiko Schocher <[email protected]> Cc: Nicolas Iooss <[email protected]> Cc: Simon Glass <[email protected]> Cc: Tim Harvey <[email protected]> Reviewed-by: Simon Glass <[email protected]>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions