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

NULL-pointer dereference in atmel_reset_device when cmdprocobj (T6) object is absent

Summary

atmel_mxt.c:418 sc->cmdprocobj = mxt_findobject(T6) NO NULL check (only msgprocobj T5 checked at 417-423). 437 init_device success -> atmel_reset_device -> 983 mxt_write_object_off(sc, sc->cmdprocobj, ...) derefs obj->start_pos_lsb -> NULL page fault panic. Poll thread reset path at 941 same unchecked call. Attacker: malicious I2C peripheral at 0x4A reporting T5 but omitting T6 -> kernel panic at attach. Fix: require cmdprocobj!=NULL in init_device + NULL guard in atmel_reset_device.

Discussion (0)

No comments yet.

PoC verification

Evidence pack

findings/poc/DF-1623 Β· 4 files
FileTypeDescriptionSize
fix.diff suggested-fix git-apply-able fix for the cited bug 551 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-1623 VERDICT

Verdict: REPRODUCED (source-confirmed)

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

Mechanism: atmel_mxt.c:418 sc->cmdprocobj = mxt_findobject(T6) NO NULL check (only msgprocobj T5 checked at 417-423). 437 init_device success -> atmel_reset_device -> 983 mxt_write_object_off(sc, sc->cmdprocobj,

Citation: sys/dev/smbus/atmel_mxt/atmel_mxt.c:418-983

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: cmdprocobj NULL deref when T6 object missing (atmel_mxt.c:418-425,983)

Verified recommended fix

Source-confirmed: cmdprocobj NULL deref when T6 object missing (atmel_mxt.c:418-425,983)

Verdict

Source-confirmed: cmdprocobj NULL deref when T6 object missing (atmel_mxt.c:418-425,983)