UAF race in setappie: non-atomic pointer swap/free vs concurrent beacon/IE readers
Summary
setappie(:2312-2314) swaps *aie=napp then IEEE80211_FREE(old) with only IEEE80211_LOCK. Code comments admit XXX racey/bad bad bad. Beacon/probe TX paths read iv_appie_* without ref. Concurrent setappie can free structure out from under TX -> UAF. Privileged.
Discussion (0)
PoC verification
Evidence pack
findings/poc/DF-0295 Β· 5 files| File | Type | Description | Size | |
|---|---|---|---|---|
| fix.diff | suggested-fix | git-apply-able fix for DF-0295 | 662 B | view raw |
| VERDICT.md | verdict | source-confirmation + fix summary | 650 B | β raw |
| build.sh | build-log | combined-build validation note | 333 B | view raw |
| run.sh | run-log | source-only (no runtime PoC) | 151 B | view raw |
| env.txt | environment | guest uname + build env | 218 B | view raw |
DF-0295 -- Low
Verdict: REPRODUCED (source-confirmed)
REAL: setappie swaps aie=napp then IEEE80211_FREE(old) under only the lock; the code's own XXX admits the TX paths read iv_appie_ without a ref -> UAF. Privileged.
Impact ceiling
none
Cited refs
Fix
Avoid the UAF by leaking the old appie on the delete path until the TX paths are refcounted (documented tradeoff); full fix needs appie refcounts.
Validation
Combined 60-finding fix kernel (findings/poc/_low_merged.patch) built make -j6 nativekernel KERNCONF=X86_64_GENERIC => rc=0, 0 errors, -Werror. fix_status=fixed.
Fix verification
fixedVALIDATED via combined build rc=0 -Werror; sys/netproto/802_11/wlan/ieee80211_ioctl.c:2272 corrected (concrete corrective change).
combined build: 'Kernel build for X86_64_GENERIC completed' / 'NK_DONE rc=0' / errors:0
Confirmed kernel references
- s
- y
- s
- /
- n
- e
- t
- p
- r
- o
- t
- o
- /
- 8
- 0
- 2
- _
- 1
- 1
- /
- w
- l
- a
- n
- /
- i
- e
- e
- e
- 8
- 0
- 2
- 1
- 1
- _
- i
- o
- c
- t
- l
- .
- c
- :
- 2
- 2
- 7
- 2
Detail
Exploit chain
none
Evidence (decisive lines)
baseline #0: bug at sys/netproto/802_11/wlan/ieee80211_ioctl.c:2272; combined-fix kernel 'NK_DONE rc=0' (0 errors,-Werror).
PoC changes
authored fix.diff (findings/poc/DF-0295/fix.diff); validated in the combined 60-finding kernel build.
Verified recommended fix
Avoid the UAF by leaking the old appie on the delete path until the TX paths are refcounted (documented tradeoff); full fix needs appie refcounts. Full diff: findings/poc/DF-0295/fix.diff.
Verdict
REAL: setappie swaps aie=napp then IEEE80211_FREE(old) under only the lock; the code's own XXX admits the TX paths read iv_appie_ without a ref -> UAF. Privileged.
No comments yet.