linux-kernel - Re: comedi

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>] [day] [month] [year] [list]

Message-ID: <f3e94902-3e0c-5857-bf44-0e2c42de30a6@mev.co.uk>
Date:   Fri, 24 Mar 2023 12:10:12 +0000
From:   Ian Abbott <abbotti@....co.uk>
To:     Gary Rookard <garyrookard@...uxmail.org>,
        "greg@...ah.com" <greg@...ah.com>
Cc:     "hsweeten@...ionengravers.com" <hsweeten@...ionengravers.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: comedi

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