Skip to content

Commit 07fd339

Browse files
ISCAS-Vulabrobherring
authored andcommitted
of: unittest: fix use-after-free in testdrv_probe()
The function testdrv_probe() retrieves the device_node from the PCI device, applies an overlay, and then immediately calls of_node_put(dn). This releases the reference held by the PCI core, potentially freeing the node if the reference count drops to zero. Later, the same freed pointer 'dn' is passed to of_platform_default_populate(), leading to a use-after-free. The reference to pdev->dev.of_node is owned by the device model and should not be released by the driver. Remove the erroneous of_node_put() to prevent premature freeing. Fixes: 26409dd ("of: unittest: Add pci_dt_testdrv pci driver") Cc: stable@vger.kernel.org Signed-off-by: Wentao Liang <vulab@iscas.ac.cn> Link: https://patch.msgid.link/20260409034859.429071-1-vulab@iscas.ac.cn Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
1 parent faecdd4 commit 07fd339

1 file changed

Lines changed: 0 additions & 1 deletion

File tree

drivers/of/unittest.c

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4317,7 +4317,6 @@ static int testdrv_probe(struct pci_dev *pdev, const struct pci_device_id *id)
43174317

43184318
size = info->dtbo_end - info->dtbo_begin;
43194319
ret = of_overlay_fdt_apply(info->dtbo_begin, size, &ovcs_id, dn);
4320-
of_node_put(dn);
43214320
if (ret)
43224321
return ret;
43234322

0 commit comments

Comments
 (0)