linux-kernel - Re: [PATCH v8 03/20] gpio: uapi: define GPIO_MAX_NAME

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 15 Sep 2020 13:12:23 +0300
From:   Andy Shevchenko <andy.shevchenko@...il.com>
To:     Kent Gibson <warthog618@...il.com>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        "open list:GPIO SUBSYSTEM" <linux-gpio@...r.kernel.org>,
        Bartosz Golaszewski <bgolaszewski@...libre.com>,
        Linus Walleij <linus.walleij@...aro.org>
Subject: Re: [PATCH v8 03/20] gpio: uapi: define GPIO_MAX_NAME_SIZE for array sizes

On Wed, Sep 9, 2020 at 1:29 PM Kent Gibson <warthog618@...il.com> wrote:
>
> Replace constant array sizes with a macro constant to clarify the source
> of array sizes, provide a place to document any constraints on the size,
> and to simplify array sizing in userspace if constructing structs
> from their composite fields.
>

Reviewed-by: Andy Shevchenko <andy.shevchenko@...il.com>


> Signed-off-by: Kent Gibson <warthog618@...il.com>
> ---
>  include/uapi/linux/gpio.h | 17 +++++++++++------
>  1 file changed, 11 insertions(+), 6 deletions(-)
>
> diff --git a/include/uapi/linux/gpio.h b/include/uapi/linux/gpio.h
> index 9c27cecf406f..285cc10355b2 100644
> --- a/include/uapi/linux/gpio.h
> +++ b/include/uapi/linux/gpio.h
> @@ -14,6 +14,11 @@
>  #include <linux/ioctl.h>
>  #include <linux/types.h>
>
> +/*
> + * The maximum size of name and label arrays.
> + */
> +#define GPIO_MAX_NAME_SIZE 32
> +
>  /**
>   * struct gpiochip_info - Information about a certain GPIO chip
>   * @name: the Linux kernel name of this GPIO chip
> @@ -22,8 +27,8 @@
>   * @lines: number of GPIO lines on this chip
>   */
>  struct gpiochip_info {
> -       char name[32];
> -       char label[32];
> +       char name[GPIO_MAX_NAME_SIZE];
> +       char label[GPIO_MAX_NAME_SIZE];
>         __u32 lines;
>  };
>
> @@ -52,8 +57,8 @@ struct gpiochip_info {
>  struct gpioline_info {
>         __u32 line_offset;
>         __u32 flags;
> -       char name[32];
> -       char consumer[32];
> +       char name[GPIO_MAX_NAME_SIZE];
> +       char consumer[GPIO_MAX_NAME_SIZE];
>  };
>
>  /* Maximum number of requested handles */
> @@ -123,7 +128,7 @@ struct gpiohandle_request {
>         __u32 lineoffsets[GPIOHANDLES_MAX];
>         __u32 flags;
>         __u8 default_values[GPIOHANDLES_MAX];
> -       char consumer_label[32];
> +       char consumer_label[GPIO_MAX_NAME_SIZE];
>         __u32 lines;
>         int fd;
>  };
> @@ -182,7 +187,7 @@ struct gpioevent_request {
>         __u32 lineoffset;
>         __u32 handleflags;
>         __u32 eventflags;
> -       char consumer_label[32];
> +       char consumer_label[GPIO_MAX_NAME_SIZE];
>         int fd;
>  };
>
> --
> 2.28.0
>


-- 
With Best Regards,
Andy Shevchenko