[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4735D341.8090606@gmail.com>
Date: Sat, 10 Nov 2007 16:50:25 +0100
From: Jiri Slaby <jirislaby@...il.com>
To: Frank Seidel <fseidel@...e.de>
CC: linux-kernel@...r.kernel.org
Subject: Re: [RFC 4/13] Char: nozomi, tty index cleanup
On 11/10/2007 04:41 PM, Frank Seidel wrote:
> On Samstag 10 November 2007 00:45:30, you (Jiri Slaby) wrote:
>> nozomi, tty index cleanup
>>
>> - don't store unneeded copy of tty->index into port structure, tty->index
>> is available everywhere
>> - mod tty->index by MAX_PORT where expected (otherwise array index out of
>> bounds)
> The last point i can't retrace, as all those parts were already IMHO.
> NTTY_TTY_MINORS is just an alias for MAX_PORT, but of course its much
> better readable and also shorter. But i doubt this prevents to an array
> index to get out of bounds ;-)
Aha, this is comment from old patch (before you made the same fixes in c version
of the driver), where I added % in some places, e.g. index of port in dc
structure...
regards,
--
Jiri Slaby (jirislaby@...il.com)
Faculty of Informatics, Masaryk University
-
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