]> git.proxmox.com Git - mirror_ubuntu-focal-kernel.git/commit
i2c: robotfuzz-osif: fix control-request directions
authorJohan Hovold <johan@kernel.org>
Mon, 24 May 2021 09:09:12 +0000 (11:09 +0200)
committerStefan Bader <stefan.bader@canonical.com>
Fri, 13 Aug 2021 07:44:24 +0000 (09:44 +0200)
commit0477705b8b6e983c104efe1a3abba26337613533
treebaf9fa97f6894e131ec059b11be8de4fe3018dc2
parent85f22599f71d6ea21a8f020d9ec57f2f90ad0fd3
i2c: robotfuzz-osif: fix control-request directions

BugLink: https://bugs.launchpad.net/bugs/1936242
commit 4ca070ef0dd885616ef294d269a9bf8e3b258e1a upstream.

The direction of the pipe argument must match the request-type direction
bit or control requests may fail depending on the host-controller-driver
implementation.

Control transfers without a data stage are treated as OUT requests by
the USB stack and should be using usb_sndctrlpipe(). Failing to do so
will now trigger a warning.

Fix the OSIFI2C_SET_BIT_RATE and OSIFI2C_STOP requests which erroneously
used the osif_usb_read() helper and set the IN direction bit.

Reported-by: syzbot+9d7dadd15b8819d73f41@syzkaller.appspotmail.com
Fixes: 83e53a8f120f ("i2c: Add bus driver for for OSIF USB i2c device.")
Cc: stable@vger.kernel.org # 3.14
Signed-off-by: Johan Hovold <johan@kernel.org>
Signed-off-by: Wolfram Sang <wsa@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
drivers/i2c/busses/i2c-robotfuzz-osif.c