Kernel stack info-leak via uninitialized redstats[3] array in hfsc_getqstats copyout
Summary
hfsc_getqstats(:293-330): struct hfsc_classstats stats on stack NOT zero-initialized. get_class_stats fills all fields EXCEPT trailing redstats red[3] β only written for Q_RED/Q_RIO ifdef(:1696-1702). Default Q_DROPTAIL class: red[0..2] untouched. Full struct copyout to user at :327 leaks ~3*sizeof(redstats) stale kernel stack bytes. Privileged PF queue-stats ioctl. KASLR bypass seed.
Discussion (0)
PoC verification
Evidence pack
findings/poc/DF-0438 Β· 2 files| File | Type | Description | Size | |
|---|---|---|---|---|
| VERDICT.md | verdict | source verification verdict | 734 B | β raw |
| fix.diff | suggested-fix | fix for leak bug | 275 B | view raw |
DF-0438 - Verification Verdict
Verdict: REPRODUCED (source-only confirmation)
Bug class: leak
Impact: leak
Source file: sys/net/altq/altq_hfsc.c
Mechanism
CONFIRMED: hfsc_getqstats struct on stack NOT zero-initialized. get_class_stats doesn't fill trailing redstats for non-RED classes. copyout leaks stack. Fix: bzero stats.
Fix
See fix.diff for the git-apply-able patch.
Build validation
Combined kernel build with all 70 Low-severity fixes: rc=0, -Werror.
All fixes compile cleanly in X86_64_GENERIC kernel configuration.
Guest: DragonFly dfbsd 6.5-DEVELOPMENT DragonFly 6.5-DEVELOPMENT #0: Thu Jul 2 06:02:54 UTC 2026 root@dfbsd:/usr/obj/usr/src/sys/X86_64_GENERIC x86_64
Fix verification
fixedVALIDATED: fix.diff compiles cleanly in combined kernel build (rc=0, -Werror). Source trace confirms bug at sys/net/altq/altq_hfsc.c:299.
Combined build: 70 fix.diffs applied to /usr/src, nativekernel KERNCONF=X86_64_GENERIC rc=0 -Werror. All fixes compile.
Confirmed kernel references
- s
- y
- s
- /
- n
- e
- t
- /
- a
- l
- t
- q
- /
- a
- l
- t
- q
- _
- h
- f
- s
- c
- .
- c
- :
- 2
- 9
- 9
Detail
Exploit chain
none (non-corruption Low severity finding; source-only confirmation)
Evidence (decisive lines)
Source-traced at sys/net/altq/altq_hfsc.c:299. Combined kernel build with all 70 fixes: rc=0, -Werror.
PoC changes
Created fix.diff for DF-0438. No PoC binary (source-only verification).
Verified recommended fix
bzero stats struct before get_class_stats. Matches finding proposal.
Verdict
CONFIRMED source-only: hfsc_getqstats stats on stack not zeroed. redstats untouched for non-RED. Leak.
No comments yet.