From: Kyungmin Park <kmpark@infradead.org>
To: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Andi Kleen <andi@firstfloor.org>,
linux-kernel@vger.kernel.org, linux-input@vger.kernel.org
Subject: Re: [PATCH] Input notifier support
Date: Sun, 1 Mar 2009 12:16:24 +0900 [thread overview]
Message-ID: <9c9fda240902281916m221e1996gbfccd1d53b2ffa75@mail.gmail.com> (raw)
In-Reply-To: <20090228190644.ZZRA012@mailhub.coreip.homeip.net>
Hi,
On Sun, Mar 1, 2009 at 12:07 PM, Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
> On Sun, Mar 01, 2009 at 11:52:56AM +0900, Kyungmin Park wrote:
>> On Sun, Mar 1, 2009 at 9:42 AM, Dmitry Torokhov
>> <dmitry.torokhov@gmail.com> wrote:
>> > On Sun, Mar 01, 2009 at 12:34:38AM +0100, Andi Kleen wrote:
>> >> Kyungmin Park <kmpark@infradead.org> writes:
>> >>
>> >> > Some hardware doesn't connected with key button and led. In this case key should be connected with led by software. Of course each application can control it however it's too big burden to application programmer.
>> >> >
>> >> > So add input notifier and then use it at other frameworks such as led.
>> >> > Of course, other input device can use this one.
>> >> >
>> >> > Any commnets are welcome.
>> >>
>> >> It looks like the perfect interface for a password stealing root kit.
>> >>
>> >> Yes there are probably other ways to do this, but still this seems to
>> >> make it very easy.
>> >>
>> >
>> > We already have good interface for that. That's why you want to limit
>> > access to /dev/input/eventX ;) and not make it world-readable.
>> >
>>
>> Hi Dmitry,
>>
>> Could you tell me know which interface you're talking?
>> I agreed it's not good idea to pass key code value to drivers. it
>> caused security issues.
>>
>
> The interface to steal the root password. One can simply open all input
> devices and read all events (if the process has sufficient rights).
>
"I don't think we need to do the notifier route. It is very easy to add
another input handler in style of rfkill-input or apm-power that would
listen to events from all ungrabbed input devices and act accordingly.
Your LED trigger example should work very well if implemented as another
input handler."
Ah, I mean the this one, sorry for confusing you. You mean add led
trigger using input handler.
Then my question how to know which input device is connected trigger
As know it's input device is allocated by dynamically then how do we
know which input event is used and set?
Thank you,
Kyungmin Park
next prev parent reply other threads:[~2009-03-01 3:16 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-25 4:47 [PATCH] Input notifier support Kyungmin Park
2009-02-25 8:03 ` Trilok Soni
2009-02-25 9:55 ` Kyungmin Park
2009-02-26 7:01 ` Trilok Soni
2009-02-27 2:11 ` Kyungmin Park
2009-02-27 12:35 ` Trilok Soni
2009-02-28 22:30 ` Dmitry Torokhov
2009-02-28 23:34 ` Andi Kleen
2009-03-01 0:42 ` Dmitry Torokhov
2009-03-01 2:52 ` Kyungmin Park
2009-03-01 3:07 ` Dmitry Torokhov
2009-03-01 3:16 ` Kyungmin Park [this message]
2009-03-01 6:29 ` Dmitry Torokhov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=9c9fda240902281916m221e1996gbfccd1d53b2ffa75@mail.gmail.com \
--to=kmpark@infradead.org \
--cc=andi@firstfloor.org \
--cc=dmitry.torokhov@gmail.com \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).