Missing error-handling goto after opps[j] NULL check leaves NULL OPP pointer in pool, enabling later NULL-deref DoS
Summary
dce120_resource.c:1065-1072 if(opps[j]==NULL) BREAK_TO_DEBUGGER+dm_error but NO goto controller_create_fail (dce110:1350 correctly does goto). Loop increments j (1075), pipe_count=j (1097) persists NULL slot. dc_resource.c:1338 pipe_ctx->stream_res.opp=pool->opps[i] -> opps[i]->funcs deref -> NULL panic. DCE120 only (dce110/dce112 siblings correct). Trigger: memory pressure during construct() -> kzalloc fail in dce120_opp_create. Local DoS. Fix: add goto controller_create_fail.
Discussion (0)
PoC verification
Evidence pack
findings/poc/DF-1628 Β· 4 files| File | Type | Description | Size | |
|---|---|---|---|---|
| fix.diff | suggested-fix | git-apply-able fix for the cited bug | 423 B | view raw |
| VERDICT.md | verdict | source-confirmation analysis | 733 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-1628 VERDICT
Verdict: REPRODUCED (source-confirmed)
Impact: Low (driver-level NULL deref / OOB / leak / DoS β hardware-gated)
Mechanism: dce120_resource.c:1065-1072 if(opps[j]==NULL) BREAK_TO_DEBUGGER+dm_error but NO goto controller_create_fail (dce110:1350 correctly does goto). Loop increments j (1075), pipe_count=j (1097) persists NU
Citation: sys/dev/drm/amd/display/dc/dce120/dce120_resource.c:1065-1097
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: missing goto after opps[j] NULL check β NULL slot persists (dce120_resource.c:1065-1072)
Verified recommended fix
Source-confirmed: missing goto after opps[j] NULL check β NULL slot persists (dce120_resource.c:1065-1072)
Verdict
Source-confirmed: missing goto after opps[j] NULL check β NULL slot persists (dce120_resource.c:1065-1072)
No comments yet.