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  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, 10 Sep 2017 12:33:56 +0100
From:   Tomasz Kramkowski <tk@...-tk.com>
To:     Ben Hutchings <ben@...adent.org.uk>
Cc:     linux-kernel@...r.kernel.org, stable@...r.kernel.org,
        akpm@...ux-foundation.org, Jiri Kosina <jkosina@...e.cz>,
        Benjamin Tissoires <benjamin.tissoires@...hat.com>,
        Valtteri Heikkilä <rnd@....fi>
Subject: Re: [PATCH 3.16 153/233] HID: usbhid: add quirk for innomedia INNEX
 GENESIS/ATARI adapter

On Sat, Sep 09, 2017 at 10:47:14PM +0100, Ben Hutchings wrote:
> 3.16.48-rc1 review patch.  If anyone has any objections, please let me know.
> 
> ------------------
> 
> From: Tomasz Kramkowski <tk@...-tk.com>
> 
> commit 9547837bdccb4af127528b36a73377150658b4ac upstream.
> 
> The (1292:4745) Innomedia INNEX GENESIS/ATARI adapter needs
> HID_QUIRK_MULTI_INPUT to split the device up into two controllers
> instead of inputs from both being merged into one.
> 
> Signed-off-by: Tomasz Kramkowski <tk@...-tk.com>
> Acked-By: Benjamin Tissoires <benjamin.tissoires@...hat.com>
> Signed-off-by: Jiri Kosina <jkosina@...e.cz>
> [bwh: Backported to 3.16: adjust context]
> Signed-off-by: Ben Hutchings <ben@...adent.org.uk>
> ---- snip ----

Hi,

Unfortunately this quirk will not make the device mentioned in the commit
work on its own.

This fix was an additional fix on top of another more fundamental fix
which made the device, and others like it, operational. This fix was
included in commit:

Valtteri Heikkilä (1):
      HID: reject input outside logical range only if null state is set
         [3f3752705dbd50b66b66ad7b4d54fe33d2f746ed]

This patch does not appear in your shortlog for this submission and it
doesn't appear when I search linux-stable as of v3.16.47.

Additionally, the following patch might also be helpful in ensuring
the correct operation of the aforementioned device (and other devices
like it):

Tomasz Kramkowski (1):
      HID: clamp input to logical range if no null state
         [c3883fe06488a483658ba5d849b70e49bee15e7c]

However, if the above patches are not appropriate for backporting
to stable then I have absolutely nothing against my patch not being
backported either. My patch alone will not make the device operate in
a useful manner.

Thank you for considering my patch for the 3.16 stable tree and have a
nice day,

-- 
Tomasz Kramkowski

Powered by blists - more mailing lists