pinctrl: Relax user input size in pinmux_select()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Sun, 4 Jun 2023 13:12:15 +0000 (16:12 +0300)
committerLinus Walleij <linus.walleij@linaro.org>
Fri, 9 Jun 2023 07:20:56 +0000 (09:20 +0200)
This is debugfs and there is no much sense to strict the user from
sending as much data as they can. The memdup_user_nul() will anyway
fail if there is not enough memory.

Relax the user input size by removing an artificial limitaion.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20230604131215.78847-2-andriy.shevchenko@linux.intel.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinmux.c

index 2d2f3bd..82c750a 100644 (file)
@@ -677,7 +677,6 @@ void pinmux_show_setting(struct seq_file *s,
 DEFINE_SHOW_ATTRIBUTE(pinmux_functions);
 DEFINE_SHOW_ATTRIBUTE(pinmux_pins);
 
-#define PINMUX_SELECT_MAX 128
 static ssize_t pinmux_select(struct file *file, const char __user *user_buf,
                                   size_t len, loff_t *ppos)
 {
@@ -689,9 +688,6 @@ static ssize_t pinmux_select(struct file *file, const char __user *user_buf,
        unsigned int num_groups;
        int fsel, gsel, ret;
 
-       if (len > PINMUX_SELECT_MAX)
-               return -ENOMEM;
-
        buf = memdup_user_nul(user_buf, len);
        if (IS_ERR(buf))
                return PTR_ERR(buf);