linux-kernel - Re: [PATCH 00/12] HID: hid-lg, hid-lg4ff: Mostly cleanup patches

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sun, 29 Mar 2015 11:01:00 +0200
From:	Elias Vanderstuyft <elias.vds@...il.com>
To:	Michal Malý <madcatxster@...oid-pointer.net>
Cc:	Jiri Kosina <jkosina@...e.cz>,
	"open list:HID CORE LAYER" <linux-input@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Simon Wood <simon@...gewell.org>
Subject: Re: [PATCH 00/12] HID: hid-lg, hid-lg4ff: Mostly cleanup patches

On Sat, Mar 21, 2015 at 12:47 PM, Michal Malý
<madcatxster@...oid-pointer.net> wrote:
> Hello everyone,
>
> this is a mostly boring series that deals with a few inconsistencies in the
> code that have accumulated over the years. Besides that it patches up a handful
> of problems such a return values not being checked etc.
>
> The only significant change comes in patch 0007 which introduces a spinlock to
> handle concurrent access to the HID report that is used by the driver to send
> data to the wheel. I would appreciate some comments on this one, particularly
> on the way it handles deinitialization.
>
> Michal Malý (12):
>   HID: hid-lg4ff: (Cleanup) Remove double underscore prefix from numeric
>     types.
>   HID: hid-lg4ff: (Cleanup) Remove "hid_" prefix from some functions
>     names.
>   HID: hid-lg4ff: (Cleanup) Replace DEVICE_ATTR_RW with DEVICE_ATTR to
>     have all internal functions prefixed with "lg4ff_"
>   HID: hid-lg4ff: (Cleanup) Remove unused variable from the
>     "lg4ff_device_entry" struct.
>   HID: hid-lg4ff: Update a warning message for a case where device is
>     incorrectly flagged to be handled by hid-lg4ff in hid-lg.
>   HID: hid-lg: Check return values from lg[N]ff_init()
>   HID: hid-lg4ff: Protect concurrent access to the output HID report
>     values with a spinlock.
>   HID: hid-lg4ff: Store pointer to the output HID report struct in the
>     device entry struct.
>   HID: hid-lg4ff: Constify those members of lg4ff_device_entry struct
>     whose value is not supposed to change.
>   HID: hid-lg4ff: Allow the driver to continue without sysfs interface.
>   HID: hid-lg4ff: Update respective sysfs interface documentation
>   HID: hid-lg: Only one of LG_FF flags can be set for a given device.
>
>  .../ABI/testing/sysfs-driver-hid-logitech-lg4ff    |   8 +-
>  drivers/hid/hid-lg.c                               |  21 +-
>  drivers/hid/hid-lg4ff.c                            | 459 ++++++++++++++-------
>  drivers/hid/hid-lg4ff.h                            |   4 +-
>  4 files changed, 319 insertions(+), 173 deletions(-)
>
> --
> 2.3.3
>

Tested on kernel 3.18.6 (I had to apply some uptream patches first).
I tried to crash the kernel by simultaneously running many instances
of FF applications (which made use of effects with nonzero attack
times, thus the ff-memless timer's timeout got modified often), and
then unplugging the device while in use.
Instead of crashing, the kernel behaved like it should, no panics or
lockups were found.

Tested with to following devices:
- Logitech MOMO (Black) : PID 0xCA03
- Logitech Formula Vibration : PID 0xCA04

Tested-by: Elias Vanderstuyft <elias.vds@...il.com>

Regards,
Elias
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/