Re: [PATCH] hidraw: Fix crash on HIDIOCGFEATURE with a destroyed device

2018-04-09 Thread Jiri Kosina
On Fri, 6 Apr 2018, Rodrigo Rivas Costa wrote: > Doing `ioctl(HIDIOCGFEATURE)` in a tight loop on a hidraw device > and then disconnecting the device, or unloading the driver, can > cause a NULL pointer dereference. > > When a hidraw device is destroyed it sets 0 to `dev->exist`. > Most functions

[PATCH] hidraw: Fix crash on HIDIOCGFEATURE with a destroyed device

2018-04-05 Thread Rodrigo Rivas Costa
Doing `ioctl(HIDIOCGFEATURE)` in a tight loop on a hidraw device and then disconnecting the device, or unloading the driver, can cause a NULL pointer dereference. When a hidraw device is destroyed it sets 0 to `dev->exist`. Most functions check 'dev->exist' before doing its work, but `hidraw_get_r