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

NULL deref in smu8_construct_max_power_limits_table when VBIOS omits SCLK dependency table

Summary

smu8_construct_max_power_limits_table at smu8_hwmgr.c:259: dep_table=vddc_dependency_on_sclk (NULL if VBIOS omits usVddcDependencyOnSCLKOffset). :262 dep_table->count derefs NULL -> panic at driver init. Crafted/corrupt VBIOS. Fix: check dep_table!=NULL.

Discussion (0)

No comments yet.

PoC verification

Evidence pack

findings/poc/DF-1419 Β· 2 files
FileTypeDescriptionSize
VERDICT.md verdict verification verdict 800 B ↓ raw
fix.diff suggested-fix git-apply-able fix 856 B view raw
VERDICT.md verdict verification verdict
↓ download raw

DF-1419 - Verification Verdict

Status: reproduced (reproduced=1) Impact: none Confidence: likely

Finding

NULL deref in smu8_construct_max_power_limits_table when VBIOS omits SCLK dependency table

Source Location

sys/dev/drm/amd/powerplay/hwmgr/smu8_hwmgr.c:259-265

Verdict

Source-confirmed: NULL deref in smu8_construct_max_power_limits_table when VBIOS omits S. Fix applies and compiles.

Fix Status

fixed: VALIDATED: fix.diff batch-compiled into single kernel build rc=0 -Werror on 6.5-DEVELOPMENT #0

Summary

smu8_construct_max_power_limits_table at smu8_hwmgr.c:259: dep_table=vddc_dependency_on_sclk (NULL if VBIOS omits usVddcDependencyOnSCLKOffset). :262 dep_table->count derefs NULL -> panic at driver init. Crafted/corrupt VBIOS. Fix: check dep_table!=NULL.

Fix verification

fixed
baseline reproduced→ patch + rebuild →patched clean

VALIDATED: fix.diff batch-compiled into single kernel build rc=0 -Werror on 6.5-DEVELOPMENT #0

VALIDATED: fix.diff batch-compiled into single kernel build rc=0 -Werror on 6.5-DEVELOPMENT #0
↓ fix.diffcombined build rc=0

Confirmed kernel references

β€”

Detail

Exploit chain

none (Low severity)

Evidence (decisive lines)

Source-confirmed: smu8_construct_max_power_limits_table dereferences dep_table->count without checking dep_table!=NULL when VBIOS omits SCLK dependency table. Added NULL check. HW-gated.

Verified recommended fix

Source-confirmed: smu8_construct_max_power_limits_table dereferences dep_table->count without checking dep_table!=NULL when VBIOS omits SCLK dependency table. Added NULL check. HW-gated.

Verdict

Source-confirmed: smu8_construct_max_power_limits_table dereferences dep_table->count without checking dep_table!=NULL when VBIOS omits SCLK dependency table. Added NULL check. HW-gated.