linux-kernel - Re: [PATCH v2] PCI: dwc: keystone: Fix potential NULL dereference

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

Message-ID: <63b89455-3e30-421a-a082-00d39e836e20@intel.com>
Date: Thu, 25 Apr 2024 15:00:14 +0200
From: Alexander Lobakin <aleksander.lobakin@...el.com>
To: Aleksandr Mishin <amishin@...rgos.ru>
CC: Rob Herring <robh@...nel.org>, Lorenzo Pieralisi <lpieralisi@...nel.org>,
	Krzysztof Wilczyński <kw@...ux.com>, Bjorn Helgaas
	<bhelgaas@...gle.com>, Manivannan Sadhasivam
	<manivannan.sadhasivam@...aro.org>, Uwe Kleine-König
	<u.kleine-koenig@...gutronix.de>, Serge Semin <fancer.lancer@...il.com>,
	Niklas Cassel <cassel@...nel.org>, Yoshihiro Shimoda
	<yoshihiro.shimoda.uh@...esas.com>, Damien Le Moal <dlemoal@...nel.org>,
	Siddharth Vadapalli <s-vadapalli@...com>, <linux-pci@...r.kernel.org>,
	<linux-kernel@...r.kernel.org>, <lvc-project@...uxtesting.org>
Subject: Re: [PATCH v2] PCI: dwc: keystone: Fix potential NULL dereference

From: Aleksandr Mishin <amishin@...rgos.ru>
Date: Thu, 25 Apr 2024 12:21:35 +0300

> In ks_pcie_setup_rc_app_regs() resource_list_first_type() may return
> NULL which is later dereferenced. Fix this bug by adding NULL check.
> 
> Found by Linux Verification Center (linuxtesting.org) with SVACE.

Please stop spamming with "potential fixes" made mechanically from
static analyzer reports without looking into the code flow. These
patches are mostly incorrect and may hurt.
Either have a stable repro and then fix the real bug or don't touch
anything at all.

> 
> Fixes: 0f71c60ffd26 ("PCI: dwc: Remove storing of PCI resources")
> Signed-off-by: Aleksandr Mishin <amishin@...rgos.ru>

Thanks,
Olek