Hi Dmitry On Thu, Sep 13, 2012 at 6:57 PM, Dmitry Torokhov <[email protected]> wrote: > Hi David, > > On Thu, Sep 13, 2012 at 06:18:17PM +0200, David Herrmann wrote: >> Hi Dmitry >> >> On Thu, Sep 13, 2012 at 6:17 PM, David Herrmann >> <[email protected]> wrote: >> > The "input_table" array is very small and we cannot be sure that the file >> > the user opens has a minor-ID below 256 (8 << 5). Hence, simply check that >> > the minor isn't out-of-bounds. If it is, return -ENODEV. >> > >> > Signed-off-by: David Herrmann <[email protected]> >> > Cc: <[email protected]> >> >> This should be "Cc: <[email protected]>". Sorry for the typo. > > register_chrdev() reserves exactly 256 minors so it is not possible for > input_open_file() to be called with minor that will get us out of > bounds.
Ah, magic numbers. I love them! But thanks, good to know, as I am currently working on integrating dynamic-minors. Sorry for the noise. Regards David -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
