Re: comedi

[Ian Abbott]

On 24/03/2023 11:51, Gary Rookard wrote:
>  From ff0a68124526ea1ac003fbfdd4de47e347881dfe Mon Sep 17 00:00:00 2001
> From: Gary Rookard <garyrookard@xxxxxxxxxxxxx>
> 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 <garyrookard@xxxxxxxxxxxxx>
> ---
>   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 <abbotti@xxxxxxxxx> || 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. || www.mev.co.uk )=-