2023-03-24 12:19:57

by Ian Abbott

[permalink] [raw]
Subject: Re: comedi

On 24/03/2023 11:51, Gary Rookard wrote:
> From ff0a68124526ea1ac003fbfdd4de47e347881dfe Mon Sep 17 00:00:00 2001
> From: Gary Rookard <[email protected]>
> Date: Thu, 23 Mar 2023 16:10:12 -0400
> Subject: [PATCH] Staging: comedi: comedi_fops: fixed missing blank line
> coding
>  style issues
>
> Fixed/added 3 blank lines to resolve  issues, comedi_fops is checkpatch
> clean.
>
> Signed-off-by: Gary Rookard <[email protected]>
> ---
>  drivers/staging/comedi/comedi_fops.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/drivers/staging/comedi/comedi_fops.c
> b/drivers/staging/comedi/comedi_fops.c
> index e85a99b68f31..3f70e5dfac39 100644
> --- a/drivers/staging/comedi/comedi_fops.c
> +++ b/drivers/staging/comedi/comedi_fops.c
> @@ -2169,6 +2169,7 @@ static long comedi_unlocked_ioctl(struct file
> *file, unsigned int cmd,
>                 break;
>         case COMEDI_CHANINFO: {
>                 struct comedi_chaninfo it;
> +
>                 if (copy_from_user(&it, (void __user *)arg, sizeof(it)))
>                         rc = -EFAULT;
>                 else
> @@ -2177,6 +2178,7 @@ static long comedi_unlocked_ioctl(struct file
> *file, unsigned int cmd,
>         }
>         case COMEDI_RANGEINFO: {
>                 struct comedi_rangeinfo it;
> +
>                 if (copy_from_user(&it, (void __user *)arg, sizeof(it)))
>                         rc = -EFAULT;
>                 else
> @@ -2249,6 +2251,7 @@ static long comedi_unlocked_ioctl(struct file
> *file, unsigned int cmd,
>         }
>         case COMEDI_INSN: {
>                 struct comedi_insn insn;
> +
>                 if (copy_from_user(&insn, (void __user *)arg,
> sizeof(insn)))
>                         rc = -EFAULT;
>                 else
> --
> 2.34.1
>
> Sent from Mailspring

This is already resolved by commit 76cd0c7cc20f ("Staging: comedi: Added
blank lines to fix coding style issue") in the 5.10 kernel (from the
year 2020):

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=76cd0c7cc20f3ca71f1be38bfc0c9d3c0807e7ca

--
-=( Ian Abbott <[email protected]> || MEV Ltd. is a company )=-
-=( registered in England & Wales. Regd. number: 02862268. )=-
-=( Regd. addr.: S11 & 12 Building 67, Europa Business Park, )=-
-=( Bird Hall Lane, STOCKPORT, SK3 0XA, UK. || http://www.mev.co.uk )=-