linux-kernel - Re: [PATCH 0/1] HID: uclogic: Add HID_QUIRK_HIDINPUT

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]

Message-ID: <nycvar.YFH.7.76.2211141509340.6045@cbobk.fhfr.pm>
Date:   Mon, 14 Nov 2022 15:10:07 +0100 (CET)
From:   Jiri Kosina <jikos@...nel.org>
To:     Alexander Zhang <alex@...xyzhang.dev>
cc:     José Expósito <jose.exposito89@...il.com>,
        benjamin.tissoires@...hat.com, openglfreak@...glemail.com,
        linux-input@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/1] HID: uclogic: Add HID_QUIRK_HIDINPUT_FORCE quirk

On Thu, 10 Nov 2022, Alexander Zhang wrote:

> Hi José,
> 
> On 11/10/22 9:40 AM, José Expósito wrote:
> > Hi everyone,
> > 
> > About 3 months ago, Torge Matthies sent a patch fixing an issue with
> > his XP-Pen Star G640 [1].
> > 
> > His patch changes the IS_INPUT_APPLICATION() macro to accept
> > HID_DG_DIGITIZER. However, it looks like it is not being merged because
> > of the possible side effects that changing that macro could generate.
> > 
> > This patch aims to fix the same issue but using a more conservative
> > approach hoping that the bug can be fixed.
> > 
> > Torge, Alexander, could you test it and confirm that it fixes your
> > issues, please?
> 
> I tested the patch on commit 4bbf3422df78 and it fixes the issue.

I will add

	Tested-by: Alexander Zhang <alex@...xyzhang.dev>

to the commit; please speak up if you disagree.

Thanks,

-- 
Jiri Kosina
SUSE Labs