On Sat, May 24, 2025 at 03:58:25PM +0800, Haoxiang Li wrote:
Add check for the return value of rcar_gen4_ptp_alloc()
to prevent potential null pointer dereference.
Was the null deref observed in the wild? Asking because I am
wondering if this is clean up instead of a Fixes ?
quoted hunk ↗ jump to hunk
Fixes: b0d3969d2b4d ("net: ethernet: rtsn: Add support for Renesas Ethernet-TSN")
Cc: stable@vger.kernel.org
Signed-off-by: Haoxiang Li <redacted>
---
drivers/net/ethernet/renesas/rtsn.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/net/ethernet/renesas/rtsn.c b/drivers/net/ethernet/renesas/rtsn.c
index 6b3f7fca8d15..f5df3374d279 100644
--- a/drivers/net/ethernet/renesas/rtsn.c
+++ b/drivers/net/ethernet/renesas/rtsn.c
@@ -1260,6 +1260,10 @@ static int rtsn_probe(struct platform_device *pdev)
priv->pdev = pdev;
priv->ndev = ndev;
priv->ptp_priv = rcar_gen4_ptp_alloc(pdev);
+ if (!priv->ptp_priv) {
+ ret = -ENOMEM;
+ goto error_free;
+ }
spin_lock_init(&priv->lock);
platform_set_drvdata(pdev, priv);--
2.25.1