On 10/2/20 5:10 PM, Gustavo A. R. Silva wrote:
> diff --git a/include/uapi/linux/cdrom.h b/include/uapi/linux/cdrom.h
> index 2817230148fd..6c34f6e2f1f7 100644
> --- a/include/uapi/linux/cdrom.h
> +++ b/include/uapi/linux/cdrom.h
> @@ -289,7 +289,10 @@ struct cdrom_generic_command
>       unsigned char           data_direction;
>       int                     quiet;
>       int                     timeout;
> -     void                    __user *reserved[1];    /* unused, actually */
> +     union {
> +             void            __user *reserved[1];    /* unused, actually */
> +             void            __user *unused;
> +     };

What's the point of this union, why not just turn it into

        void *                  __user *unused;

?

-- 
Jens Axboe

Reply via email to