linux-kernel - Re: [PATCH v3] dmaengine: moxart: handle irq_of_parse_and

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

Message-ID: <CAL7siYN5gUgv6kG72paLeoeqh2pic6ZySGf_r90oK_2UyPKjiA@mail.gmail.com>
Date: Sun, 18 Aug 2024 21:03:27 +0800
From: lock hey lee <lee.lockhey@...il.com>
To: Ma Ke <make24@...as.ac.cn>
Cc: dmaengine@...r.kernel.org, 
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, stable@...r.kernel.org
Subject: Re: [PATCH v3] dmaengine: moxart: handle irq_of_parse_and_map() errors

Hello Ma Ke,

On 2024/8/18 15:17:57 +0800, Ma Ke wrote:
>Zero and negative number is not a valid IRQ for in-kernel code and the
>irq_of_parse_and_map() function returns zero on error.  So this check for
>valid IRQs should only accept values > 0.
>
>Cc: stable@...r.kernel.org
>Fixes: 2d9e31b9412c ("dmaengine: moxart: remove NO_IRQ")
>Signed-off-by: Ma Ke <make24@...as.ac.cn>
>---
>Changes in v3:
>- added missed changelog v2.
>Changes in v2:
>- added Cc stable line;
>- added Fixes line.
>---
>  drivers/dma/moxart-dma.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/dma/moxart-dma.c b/drivers/dma/moxart-dma.c
>index 66dc6d31b603..16dd3c5aba4d 100644
>--- a/drivers/dma/moxart-dma.c
>+++ b/drivers/dma/moxart-dma.c
>@@ -568,7 +568,7 @@ static int moxart_probe(struct platform_device *pdev)
>               return -ENOMEM;
>
>       irq = irq_of_parse_and_map(node, 0);
>-      if (!irq) {
>+      if (irq <= 0) {

The ’irq‘ variable type here is ‘unsigned int, which will never be negative, :-)

>
>               dev_err(dev, "no IRQ resource\n");
>               return -EINVAL;
>       }
>--
>2.25.1
>
>

Thanks,
Luoxi