Unbounded ucNumEntries-1 read in init_over_drive_limits reads patom_record_v2[-1] when ucNumEntries==0
Summary
vega10_processpptables.c:269-273 if ucRevId==1 patom_record_v2[ucNumEntries-1].ucACGEnable. ucNumEntries UCHAR; 0-1=-1 -> patom_record_v2[-1] reads 13B before entries[]. Called from vega10_pp_tables_initialize at 1165 BEFORE init_powerplay_extended_tables at 1170 which would reject ucNumEntries==0. Trigger: pp_table sysfs usGfxclkDependencyTableOffset ucRevId=1 ucNumEntries=0. Fix: check ucNumEntries>0.
Discussion (0)
PoC verification
Evidence pack
findings/poc/DF-1589 Β· 4 files| File | Type | Description | Size | |
|---|---|---|---|---|
| fix.diff | suggested-fix | git-apply-able fix for the cited bug | 556 B | view raw |
| VERDICT.md | verdict | source-confirmation analysis | 736 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 |
DF-1589 VERDICT
Verdict: REPRODUCED (source-confirmed)
Impact: Low (driver-level NULL deref / OOB / leak / DoS β hardware-gated)
Mechanism: vega10_processpptables.c:269-273 if ucRevId==1 patom_record_v2[ucNumEntries-1].ucACGEnable. ucNumEntries UCHAR; 0-1=-1 -> patom_record_v2[-1] reads 13B before entries[]. Called from vega10_pp_tables_i
Citation: sys/dev/drm/amd/powerplay/hwmgr/vega10_processpptables.c:269-273
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
fixedfix.diff compiled in batch kernel build rc=0 -Werror
fix.diff compiled in batch kernel build rc=0 -Werror
Confirmed kernel references
β
Detail
Exploit chain
none (Low severity)
Evidence (decisive lines)
Source-confirmed: ucNumEntries-1 underflow to [-1] OOB read (vega10_processpptables.c:269-273)
Verified recommended fix
Source-confirmed: ucNumEntries-1 underflow to [-1] OOB read (vega10_processpptables.c:269-273)
Verdict
Source-confirmed: ucNumEntries-1 underflow to [-1] OOB read (vega10_processpptables.c:269-273)
No comments yet.