[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20080731131056.eecc4826.akpm@linux-foundation.org>
Date: Thu, 31 Jul 2008 13:10:56 -0700
From: Andrew Morton <akpm@...ux-foundation.org>
To: Dmitry Torokhov <dmitry.torokhov@...il.com>
Cc: torvalds@...ux-foundation.org, rjw@...k.pl, bzolnier@...il.com,
sfr@...b.auug.org.au, linux-next@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-input@...r.kernel.org
Subject: Re: linux-next: Tree for July 30
On Thu, 31 Jul 2008 15:42:07 -0400
Dmitry Torokhov <dmitry.torokhov@...il.com> wrote:
> - return bits_to_user(bits, len, _IOC_SIZE(cmd), p, compat_mode);
> +
> + if ((_IOC_NR(cmd) & EV_MAX) == EV_KEY && buf_len == 0x1ff) {
> + printk(KERN_WARNING
> + "evdev.c(EVIOCGBIT): Detected suspicious "
> + "buffer size 0x1ff, limiting output to 64 "
> + "bytes. Make sure you are not using "
> + "EVIOCGBIT(EV_KEY, KEY_MAX)\n");
> + buf_len = 64;
> + }
If that works then great. But I think the printk could be improved.
Please provide sufficient information so that users (not programmers)
can go off and fix things up without needing to email kernel developers.
One suitable approach would be
printk("see http://userweb.kernel.org/~dtor/read-this.txt")
--
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/
Powered by blists - more mailing lists