linux-kernel - Re: [PATCH] Fix braille keyboard keysym generation

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] [day] [month] [year] [list]

Message-ID: <20091120230114.GN4986@const.famille.thibault.fr>
Date:	Sat, 21 Nov 2009 00:01:14 +0100
From:	Samuel Thibault <samuel.thibault@...-lyon.org>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	linux-kernel@...r.kernel.org, alan@...rguk.ukuu.org.uk
Subject: Re: [PATCH] Fix braille keyboard keysym generation

Andrew Morton, le Fri 20 Nov 2009 13:19:54 -0800, a écrit :
> >  	if (keycode >= NR_KEYS)
> >  		if (keycode >= KEY_BRL_DOT1 && keycode <= KEY_BRL_DOT8)
> > -			keysym = K(KT_BRL, keycode - KEY_BRL_DOT1 + 1);
> > +			keysym = U(K(KT_BRL, keycode - KEY_BRL_DOT1 + 1));
> >  		else
> >  			return;
> >  	else
> 
> Sorry, but I cannot work out (with a reasonable amount of effort) what
> the user-visible effects of this bug are likely to be.  So I am not in
> a position to decide which kernel(s) this patch should be merged into.

It will make braille keyboards actually work.  Apparently I had never
really tried it and so it has never worked up to now.

> Please be careful to include this aspect in the changelog when
> appropriate, thanks.

Right, sorry.  Braille keyboards announced as KEY_BRL_DOT* are still
pretty rare (that's why the bug completely went unnoticed from the
start), so it's maybe not worth including the patch in the stable
kernels, but making it for 2.6.32 would be a good thing.

Samuel
--
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/