NULL deref in rfcomm_session_complete: credit NULL check under #ifdef DIAGNOSTIC only, no guard on production
Summary
rfcomm_session_complete(:446-454): loops count times dequeuing credit from rs_credits. NULL check on credit(:448-453) wrapped in #ifdef DIAGNOSTIC β no guard on production builds. If L2CAP over-reports count vs outstanding credits, STAILQ_FIRST returns NULL, line :454 derefs credit->rc_dlc -> panic. Not directly remote-triggerable (requires L2CAP completion accounting bug) but defense-in-depth gap.
Discussion (0)
PoC verification
Evidence pack
findings/poc/DF-0456 Β· 2 files| File | Type | Description | Size | |
|---|---|---|---|---|
| VERDICT.md | verdict | source verification verdict | 743 B | β raw |
| fix.diff | suggested-fix | fix for null-deref bug | 428 B | view raw |
DF-0456 - Verification Verdict
Verdict: REPRODUCED (source-only confirmation)
Bug class: null-deref
Impact: dos
Source file: sys/netbt/rfcomm_session.c
Mechanism
CONFIRMED: rfcomm_session_complete NULL check on credit wrapped in #ifdef DIAGNOSTIC. No guard on production builds. STAILQ_FIRST returns NULL -> crash. Fix: remove #ifdef.
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/netbt/rfcomm_session.c:448.
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
- b
- t
- /
- r
- f
- c
- o
- m
- m
- _
- s
- e
- s
- s
- i
- o
- n
- .
- c
- :
- 4
- 4
- 8
Detail
Exploit chain
none (non-corruption Low severity finding; source-only confirmation)
Evidence (decisive lines)
Source-traced at sys/netbt/rfcomm_session.c:448. Combined kernel build with all 70 fixes: rc=0, -Werror.
PoC changes
Created fix.diff for DF-0456. No PoC binary (source-only verification).
Verified recommended fix
Remove #ifdef DIAGNOSTIC around NULL check. Matches finding proposal.
Verdict
CONFIRMED source-only: NULL check on credit wrapped in #ifdef DIAGNOSTIC. No guard in production.
No comments yet.