Re: [PATCH] input: fix error at the default input_get_keycode call

[Mauro Carvalho Chehab <mchehab@redhat.com>]

Hi Valdis,

Em 31-05-2010 16:01, Valdis.Kletnieks@vt.edu escreveu:
> On Thu, 20 May 2010 13:51:46 EDT, Valdis.Kletnieks@vt.edu said:
> 
> *ping*?  Still broken in a linux-next pull I did about an hour ago.  The
> patch fixed one oops, but it just died a little further down - I'm guessing
> it missed a case?

Sorry, -ETOOMUCHWORK

I've removed the offended patches from my linux-next tree for now. I'll do
more tests on it and provide a a fix for it soon. I've created a separate
branch for the input changes at:

	http://git.linuxtv.org/v4l-dvb.git?a=shortlog;h=refs/heads/staging/input

> 
>>> input_default_getkeycode_from_index() returns the scancode at kt_entry.scancode
>>> pointer. Fill it with the scancode address at the function call.
>>>
>>> Thanks-to: Vladis Kletnieks <Valdis.Kletnieks@vt.edu> for pointing the issue
>>>
>>> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
>>> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
>>>
>>> diff --git a/drivers/input/input.c b/drivers/input/input.c
>>> index 3b63fad..7851d8e 100644
>>
>> Still dies, but in input_set_keycode() instead...
>>
>> [   35.294528] BUG: unable to handle kernel NULL pointer dereference at (null
> )
>> [   35.295005] IP: [<(null)>] (null)
>> [   35.296935] PGD 11da3c067 PUD 11d4ad067 PMD 0 
>> [   35.296935] Oops: 0010 [#1] PREEMPT SMP 
>> [   35.299667] last sysfs file: /sys/devices/pci0000:00/0000:00:1a.7/usb1/idVendor
>> [   35.300328] CPU 0 
>> [   35.300328] Modules linked in:
>> [   35.300328] 
>> [   35.300328] Pid: 2481, comm: keymap Not tainted 2.6.34-mmotm0519 #1 0X564R/Latitude E6500                  
>> [   35.300328] RIP: 0010:[<0000000000000000>]  [<(null)>] (null)
>> [   35.300328] RSP: 0018:ffff88011d4d5cb0  EFLAGS: 00010046
>> [   35.310163] RAX: 0000000000000000 RBX: ffff88011c03e000 RCX: 0000000000000081
>> [   35.310163] RDX: ffff88011d4d5cc4 RSI: ffff88011d4d5cc8 RDI: ffff88011c03e000
>> [   35.310163] RBP: ffff88011d4d5d28 R08: ffff88011e9b28e8 R09: 0000000000000001
>> [   35.310163] R10: ffffffff81e0b160 R11: 0000000000000004 R12: 00000000000000a4
>> [   35.310163] R13: ffff88011c03e830 R14: 0000000000000286 R15: ffff88011d4d5cc8
>> [   35.310163] FS:  00007f4b86283700(0000) GS:ffff880002600000(0000) knlGS:0000000000000000
>> [   35.319397] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
>> [   35.319397] CR2: 0000000000000000 CR3: 000000011d575000 CR4: 00000000000406f0
>> [   35.319397] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
>> [   35.319397] DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400
>> [   35.319397] Process keymap (pid: 2481, threadinfo ffff88011d4d4000, task ffff88011e9b28c0)
>> [   35.319397] Stack:
>> [   35.319397]  ffffffff813bf3d1 ffff88011d4d5cf8 0000008100000246 00000081000000a4
>> [   35.319397] <0> 0000000000000004 0000000000000000 ffff88011d4d5cc4 ffff88011cf11200
>> [   35.319397] <0> ffff88011c179000 ffff88011d4d5d28 0000000000000081 00007fff9ee21fa0
>> [   35.319397] Call Trace:
>> [   35.319397]  [<ffffffff813bf3d1>] ? input_set_keycode+0xad/0x12c
>> [   35.319397]  [<ffffffff813c231d>] evdev_do_ioctl+0x22b/0x79b
>> [   35.337913]  [<ffffffff815a4b04>] ? __mutex_lock_common+0x564/0x580
>> [   35.337913]  [<ffffffff813c28ca>] ? evdev_ioctl_handler+0x3d/0x80
>> [   35.341507]  [<ffffffff813c28ca>] ? evdev_ioctl_handler+0x3d/0x80
>> [   35.341507]  [<ffffffff813c28f0>] evdev_ioctl_handler+0x63/0x80
>> [   35.344034]  [<ffffffff813c292a>] evdev_ioctl+0xb/0xd
>> [   35.344034]  [<ffffffff810ea6cd>] vfs_ioctl+0x2d/0xa1
>> [   35.344034]  [<ffffffff810eac4c>] do_vfs_ioctl+0x494/0x4cd
>> [   35.344034]  [<ffffffff810eacdc>] sys_ioctl+0x57/0x95
>> [   35.344034]  [<ffffffff8100266b>] system_call_fastpath+0x16/0x1b
>> [   35.344034] Code:  Bad RIP value.
>> [   35.344034] RIP  [<(null)>] (null)
>> [   35.344034]  RSP <ffff88011d4d5cb0>
>> [   35.344034] CR2: 0000000000000000
>> [   35.357018] ---[ end trace 394fa5aa8a77b6f3 ]---