'uinput' Oops upon select() or poll() on 2.6.7

From: Sau Dan Lee
Date: Wed Jun 16 2004 - 06:36:16 EST



Re-reporting this bug found 3 weeks ago, because it's still in 2.6.7.
(It was found and reported for 2.6.7-rc1.)

For details, please check the bugzilla entry

http://bugzilla.kernel.org/show_bug.cgi?id=2786


Dmitry Torokhov has already offered a patch on 2004-05-28 12:52.

Vojtech Pavlik said on 2004-05-28 13:59 that he has fixed it in his
input tree.


But the bug is STILL in 2.6.7. I don't understand. What are the -rc*
supposed for, if bugs do not get fixed?



from Dmitry Torokhov 2004-05-28 12:52:

--- 2.6.6/drivers/input/misc/uinput.c 2004-05-11 14:57:25.793252800 -0500
+++ linux-2.6.6/drivers/input/misc/uinput.c 2004-05-28 14:42:36.922273600 -0500
@@ -279,10 +279,11 @@
{
struct uinput_device *udev = file->private_data;

- poll_wait(file, &udev->waitq, wait);
-
- if (udev->head != udev->tail)
- return POLLIN | POLLRDNORM;
+ if (test_bit(UIST_CREATED, &udev->state)) {
+ poll_wait(file, &udev->waitq, wait);
+ if (udev->head != udev->tail)
+ return POLLIN | POLLRDNORM;
+ }

return 0;
}


--
Sau Dan LEE ???(Big5) ~{@nJX6X~}(HZ)

E-mail: danlee@xxxxxxxxxxxxxxxxxxxxxxxxxx
Home page: http://www.informatik.uni-freiburg.de/~danlee

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/