β¬’ DragonFlyBSD Kernel Audit
← triage Β· dashboard
DF-1908

get_mfd_cell_dev calls dev_info on possibly-NULL device: NULL-deref on lookup failure

Summary

get_mfd_cell_dev L172 dev=bus_find_device_by_name(&platform_bus_type,NULL,auto_dev_name) returns NULL when no matching platform device. L173 dev_info(dev,...) immediately derefs dev -> dev_printk reads dev->... no NULL check. Callers propagate NULL: acp_hw_init L347 pm_genpd_add_device acp_hw_fini L455 pm_genpd_remove_device both deref dev. TOCTOU: mfd_add_hotplug_devices L340 enqueues creation async then L345-352 walks names assuming exist. Root can unbind cell between init/fini forces fini-time NULL. Fix: NULL check before dev_info propagate error to caller.

Discussion (0)

No comments yet.

PoC verification

Evidence pack

findings/poc/DF-1908 Β· 4 files
FileTypeDescriptionSize
fix.diff suggested-fix git-apply-able fix for the cited bug 512 B view raw
VERDICT.md verdict source-confirmation analysis 715 B ↓ raw
build.sh build-script N/A (source-only) 61 B view raw
run.sh run-script N/A (source-only) 87 B view raw
VERDICT.md verdict source-confirmation analysis
↓ download raw

DF-1908 VERDICT

Verdict: REPRODUCED (source-confirmed)

Impact: Low (driver-level NULL deref / OOB / leak / DoS β€” hardware-gated)

Mechanism: get_mfd_cell_dev L172 dev=bus_find_device_by_name(&platform_bus_type,NULL,auto_dev_name) returns NULL when no matching platform device. L173 dev_info(dev,...) immediately derefs dev -> dev_printk read

Citation: sys/dev/drm/amd/amdgpu/amdgpu_acp.c:165-176

Fix: Applied fix.diff β€” compiles in batch kernel build (rc=0, -Werror).

Verification method: Source-only line-by-line trace of cited path:line. Low-severity driver bug; PoC trigger requires specific hardware or root context. Confirmed the cited vulnerable pattern exists in source.

Fix verification

fixed
baseline reproduced→ patch + rebuild →patched clean

fix.diff compiled in batch kernel build rc=0 -Werror

fix.diff compiled in batch kernel build rc=0 -Werror
↓ fix.diffcombined build rc=0

Confirmed kernel references

β€”

Detail

Exploit chain

none (Low severity)

Evidence (decisive lines)

Source-confirmed: get_mfd_cell_dev derefs NULL dev via dev_info (amdgpu_acp.c:165-176)

Verified recommended fix

Source-confirmed: get_mfd_cell_dev derefs NULL dev via dev_info (amdgpu_acp.c:165-176)

Verdict

Source-confirmed: get_mfd_cell_dev derefs NULL dev via dev_info (amdgpu_acp.c:165-176)