netdev - Re: [PATCH net] net: phy: marvell10g: fix null pointer 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] [day] [month] [year] [list]

Date:   Tue, 11 Aug 2020 15:43:31 -0700 (PDT)
From:   David Miller <davem@...emloft.net>
To:     marek.behun@....cz
Cc:     netdev@...r.kernel.org, maxime.chevallier@...tlin.com,
        andrew@...n.ch, baruch@...s.co.il, rmk+kernel@...linux.org.uk
Subject: Re: [PATCH net] net: phy: marvell10g: fix null pointer dereference

From: Marek Behún <marek.behun@....cz>
Date: Mon, 10 Aug 2020 17:01:58 +0200

> Commit c3e302edca24 ("net: phy: marvell10g: fix temperature sensor on 2110")
> added a check for PHY ID via phydev->drv->phy_id in a function which is
> called by devres at a time when phydev->drv is already set to null by
> phy_remove function.
> 
> This null pointer dereference can be triggered via SFP subsystem with a
> SFP module containing this Marvell PHY. When the SFP interface is put
> down, the SFP subsystem removes the PHY.
> 
> Fixes: c3e302edca24 ("net: phy: marvell10g: fix temperature sensor on 2110")
> Signed-off-by: Marek Behún <marek.behun@....cz>

Applied and queued up for -stable, thank you.