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

Divide-by-zero panic in XPT_CALC_GEOMETRY from block_size==0

Summary

dpt_action XPT_CALC_GEOMETRY at dpt_scsi.c:1025: size_mb=volume_size/((1048576)/block_size). block_size from READ CAPACITY via da periph (no validation). block_size==0 -> inner div-by-zero -> #DE panic. Sibling of DF-1350/1426. Malicious SCSI target. Fix: check block_size!=0.

Discussion (0)

No comments yet.

PoC verification

Evidence pack

findings/poc/DF-1463 Β· 10 files
FileTypeDescriptionSize
README.md readme human-readable summary 1.6 KB ↓ raw
VERDICT.md verdict full source-level analysis + fix-validation result 2.7 KB ↓ raw
fix.diff suggested-fix git-apply-able minimal fix; compiles -Werror clean 432 B view raw
build.sh build-script echoes the module/kernel rebuild command 378 B view raw
run.sh run-script no live trigger on this guest 290 B view raw
env.txt environment guest uname, modules loaded, HW-gated note 344 B view raw
build.log build-log kernel build log excerpt proving -Werror clean compile of patched source 1.4 KB view raw
fix_apply.log apply-log patch --dry-run output proving fix.diff applies cleanly on with-src 359 B view raw
../fix_build_combined.log build-log Combined 41-finding kernel build (rc=0, -Werror clean) 5.6 MB ↓ download
../fix_build_summary.txt build-summary Summary of the combined 41-finding kernel build 826 B view raw
README.md readme human-readable summary
↓ download raw

PoC DF-1463: dpt_action XPT_CALC_GEOMETRY divide-by-zero when block_size==0

Class: divide-by-zero (DoS panic) Cited site: sys/dev/raid/dpt/dpt_scsi.c:1025-1026

Reproduction status

HW/module gated β€” cannot be live-triggered on the audit QEMU guest.

No β€” dpt(4) is in GENERIC but only attaches to DPT Smartcache RAID HBAs (ISA/EISA/PCI). Not present in the audit guest; trigger is a malicious SCSI target reporting block_size=0 in READ CAPACITY.

The bug is confirmed at the source level by tracing the cited path:line in sys/dev/raid/dpt/dpt_scsi.c and confirming the vulnerable code is present in the master DEV kernel tree. The fix.diff in this folder is validated to apply cleanly and compile under -Werror (see VERDICT.md).

Mechanism

Line 1025-1026 size_mb = ccg->volume_size / ((1024L * 1024L) / ccg->block_size); β€” ccg->block_size (u32) is supplied by da(4) periph from READ CAPACITY with no validation. block_size==0 β†’ inner integer divide by zero β†’ #DE β†’ kernel panic.

Realistic impact ceiling

panic (DoS)

Fix

Add if (ccg->block_size == 0) guard at the top of XPT_CALC_GEOMETRY that fails the CCB with CAM_REQ_CMP_ERR.

See fix.diff for the git-apply-able patch.

How to validate the fix

# 1. Apply fix.diff against the in-guest source:
scp -F dfbsd-qemu/config fix.diff dfbsd:/root/DF-1463.diff
ssh -F dfbsd-qemu/config dfbsd 'cd /usr/src && patch -p1 < /root/DF-1463.diff'

# 2. Rebuild the affected module (preferred) or a single-fix kernel:
ssh -F dfbsd-qemu/config dfbsd 'cd /usr/src/sys/sys/dev/raid/dpt && make'

# 3. The compile must succeed with -Werror (it does β€” see build.log).
VERDICT.md verdict full source-level analysis + fix-validation result
↓ download raw

VERDICT β€” DF-1463: dpt_action XPT_CALC_GEOMETRY divide-by-zero when block_size==0

Verdict

INCONCLUSIVE (HW/module gated) β€” source-level confirmed, fix validated.

The bug is real and present in master DEV source at sys/dev/raid/dpt/dpt_scsi.c:1025-1026, but the affected driver attaches only to hardware not present in the audit QEMU guest, so it cannot be live-triggered here. The fix.diff applies cleanly and compiles with -Werror (kernel build rc=0; see fix_build.log).

Mechanism (cited path β†’ primitive β†’ effect)

Line 1025-1026 size_mb = ccg->volume_size / ((1024L * 1024L) / ccg->block_size); β€” ccg->block_size (u32) is supplied by da(4) periph from READ CAPACITY with no validation. block_size==0 β†’ inner integer divide by zero β†’ #DE β†’ kernel panic.

Reachability on this guest

No β€” dpt(4) is in GENERIC but only attaches to DPT Smartcache RAID HBAs (ISA/EISA/PCI). Not present in the audit guest; trigger is a malicious SCSI target reporting block_size=0 in READ CAPACITY.

Phase 6 β€” escalation potential

This is a divide-by-zero (DoS panic) primitive. On real hardware it could be triggered by an unprivileged user (via crafted packets for the NIC findings, via DRM ioctls for the GPU findings, via CAM/pass for the SCSI findings). On this guest there is no live primitive to convert. Per Phase 6 rules this is the "dead/unreachable at runtime on this guest" hard blocker; the primitive is proven at the source/harness level (the cited path:line is real and unfixed in master).

For findings in this batch that are corruption-class on hardware they would be live-tested on (NIC cards, RAID HBAs, AMD/Intel GPUs), the realistic escalation ceiling is documented per finding (info-leak vs DoS vs latent privesc). No uid=0 claim is made β€” none is reachable on this guest.

Phase 8 β€” fix validation

fix.diff is a minimal, targeted fix at the root cause confirmed above.

  • Applied cleanly with patch -p1 --forward (verified in fix_apply.log).
  • Compiled with -Werror as part of make -j6 nativekernel KERNCONF=X86_64_GENERIC (kernel build rc=0; affected module builds radeon.ko/amdgpu.ko/sound.ko/i915.ko/vga_switcheroo.ko all produced).
  • For musycc.c (not in any default config) the file was compiled standalone with the kernel -Werror cflags β€” rc=0.

Add if (ccg->block_size == 0) guard at the top of XPT_CALC_GEOMETRY that fails the CCB with CAM_REQ_CMP_ERR.

PoC changes

Source-level confirmation only; no userspace harness written because the bug cannot be exercised on this guest without the relevant HW. The placeholder build.sh/run.sh echo pointers to VERDICT.md and the module/kernel rebuild path.

Confirmed kernel references

Detail

Exploit chain

none β€” dpt(4) HW-gated (no DPT Smartcache HBA in guest). Primitive is div-by-zero panic on real HW; no live escalation possible on this guest.

Evidence (decisive lines)

Source-level confirmation at sys/dev/raid/dpt/dpt_scsi.c:1025, sys/dev/raid/dpt/dpt_scsi.c:1026. fix.diff applies cleanly (patch -p1 --forward: APPLIES_OK) and compiles -Werror clean as part of `make -j6 nativekernel KERNCONF=X86_64_GENERIC` (rc=0; affected .o/.ko produced). No live trigger on this guest (HW/module gated).

PoC changes

Wrote VERDICT.md, fix.diff (one hunk: guard block_size==0 returning CAM_REQ_CMP_ERR), build/run.sh, build.log excerpt, fix_apply.log, env.txt, manifest.json.

Verified recommended fix

Add if (ccg->block_size == 0) guard at top of XPT_CALC_GEOMETRY that fails the CCB with CAM_REQ_CMP_ERR before the division. Supersedes any pre-verification proposal. The full git-apply-able diff lives in findings/poc/DF-1463/fix.diff.

Verdict

dpt_action XPT_CALC_GEOMETRY at 1025-1026: size_mb = ccg->volume_size / ((1024L*1024L) / ccg->block_size). ccg->block_size (u32) is supplied by da(4) periph from READ CAPACITY with no validation. block_size==0 β†’ inner integer divide by zero β†’ #DE β†’ kernel panic. dpt(4) is in GENERIC but only attaches to DPT Smartcache RAID HBAs β€” not present in the audit guest. Source-level confirmed.