DF-3067 — PAD-only undo FIFO makes hammer_recover_stage1's seqno backscan (and stage2's extended-range scan) unbounded — mount never terminates (livelock DoS)

Kernel: DragonFly 6.5-DEVELOPMENT #0 (X86_64_GENERIC, INVARIANTS), sys/vfs/hammer File under audit: sys/vfs/hammer/hammer_recover.c (pass 2). New finding — not DF-0812/DF-2594 (redo_data_bytes) nor DF-3040 (vol0_undo_array index).

Root cause (hammer_recover.c)

• For every version ≥ 4 mount (stock images are version 7), stage1 runs a seqno backscan (:257-270): a for(;;) reverse FIFO walk whose only exits are an I/O/signature error or the first non-PAD record (to grab its hdr_seq). PAD records are explicitly exempt from the seqno discontinuity check (:266-269), and hammer_recover_scan_rev wraps the walk at the zone base (:789-790), making the FIFO a ring.
• An undo FIFO consisting solely of valid 8-byte PAD records ({0xC84E, 0x8040, 8} — self-consistent head==tail, no CRC required per _hammer_check_signature:930) defeats both exits: no error, no non-PAD. The loop has no byte counter and no lap limit → infinite loop at 100% CPU inside mount(2). The disabled #if 0 block at :312-335 (a "grossly beyond last_offset" bound for the sibling forward scan) shows the author considered bounding these walks — it is compiled out.
• Same unbounded shape in stage2's extended-range scan (:652-670): it runs until scan_offset == ext_offset where ext_offset = hmp->recover_stage2_offset is REDO_SYNC.redo_offset — an unvalidated u64 from the crafted record (:423-424); a garbage value that is not a reachable record boundary plus an all-PAD gap = same infinite loop (RW mounts).

Threat model

Mount of a crafted HAMMER image (root, or unprivileged with vfs.usermount=1 — same precondition family as DF-0797/0798/0812/2594/3040). A read-only mount suffices for the stage1 variant. The mount syscall never returns; the mounting thread spins in kernel mode and cannot be killed (the loop has no cancellation points), pinning a CPU and the vn device forever. Works on INVARIANTS and production kernels (no assertion involved).

Reproduce

# guest (root): sh build.sh   (cc padflood.c; truncate 12G + newfs_hammer base)
sh run.sh
# expected:
#   padflood floods all undo bigblocks with PAD-8s, sets vol_version=6,
#   first=next=UNDO|0x1000, then:
#   MOUNTING ... (30s timeout) MOUNT_RC=124
#   ps: mount_hammer still present (unkillable, state R)
#   dmesg: thousands of "rev scan_offset 3000xxxxxxxxxxxx" lines cycling
#          down through the 1GB FIFO and wrapping - the unbounded walk
#   guest otherwise alive (other ssh commands keep working)

Success criterion: MOUNT_RC=124 + unkillable mount process + cyclic rev scan_offset debug flood + guest still up = reproduced (impact: dos).

DF-3067 VERDICT — REPRODUCED (dos)

Status: reproduced / impact: dos / confidence: certain. Baseline verified on the stock INVARIANTS kernel (DragonFly 6.5-DEVELOPMENT #0, Thu Jul 2 06:02:54 UTC 2026); fix validated on the rebuilt kernel #1.

What was run

padflood backscan patches a real newfs_hammer image (12G sparse, undo FIFO 64 bigblocks, version 7→6): every byte of a 256KB undo ring (alloc_offset = UNDO|0x40000, undo bigblock 0 at phys 0x1088000) becomes a valid 8-byte PAD record {0xC84E,0x8040,8}, and vol0_blockmap[3].first_offset = next_offset = UNDO|0x1000. Mounting it (RO or RW) runs hammer_recover_stage1's version-4 seqno backscan (hammer_recover.c:257-270), which walks the FIFO backwards to find a non-PAD record to seed hdr_seq.

Why it hangs (path:line)

• The backscan for(;;) has exactly two exits: an I/O/signature error, or a non-PAD record (:266-269) — PAD records are exempt from the seqno discontinuity check that bounds every other walk in this file.
• hammer_recover_scan_rev wraps the walk at the zone base (:789-790: scan_offset == HAMMER_ENCODE_UNDO(0) → alloc_offset), making the FIFO a ring; PAD-8 records are self-consistent head==tail (8 bytes, _hammer_check_signature:901-924,930,948-961 — no CRC needed), so every position validates and neither exit ever fires.
• There is no byte counter and no lap limit in the loop (the sibling forward scan's "grossly ahead" bound is #if 0'd out at :312-335).

Baseline observations (stock kernel #0)

• The mount never returns; the mounting thread spins at 100% CPU in kernel mode; kill -9 pends forever (no cancellation points) — the process cannot be reaped, vn0 stays configured.
• With vfs.hammer.debug_general=0x80 the serial console trace (bootlog_walk_trace.log, 47,039 lines captured while still running) shows: first traced offset 3000000000001000 == the crafted first_offset; the walk descends to the zone base, wraps UP 3000000000000000 → 300000000003fff8 at iterations 513 and 33281 (hammer_recover.c:789-790); 32,769 distinct offsets = the entire ring revisited — lap ≥ 2 at capture time, still descending.
• Guest otherwise alive (fresh ssh sessions answered) — CPU burn + wedged mount + unrecoverable vn device; repeated mounts stack spinning threads.
• Works on INVARIANTS and production kernels (no assertion involved); a read-only mount suffices (stage1 runs before the RW/RO distinction matters).

Stage2 variant (same root cause, not separately run): the extended-range scan :652-670 terminates only when scan_offset == ext_offset where ext_offset = REDO_SYNC.redo_offset — an unvalidated u64 (:423-424); a garbage value plus an all-PAD gap = the same infinite loop on RW mounts.

Fix validation (fix.diff, kernel #1: Sat Sep 5 22:10:54 UTC 2026)

One-lap walked bound added to both backscan loops. Identical PoC image on the patched kernel: the walk does exactly one lap (trace ends back at 3000000000001008) then recovery failure during seqno backscan → Failed to recover HAMMER filesystem on mount, mount_hammer exits, guest up. Regressions: stock image mount/RW-write/sync/umount/remount cycle clean; hard-crash dirty recovery (reboot -q on mounted dirty fs) clean (recovery check seqno=0010014d … files intact).

Impact classification

Local DoS (livelock) from mount of a crafted image — root or vfs.usermount=1 (same precondition family as DF-0797/0798/0812/2594/3040); unkillable spinning kernel thread per mount attempt. Not memory corruption; no escalation path identified (the walk only reads FIFO buffers).

Fix status (batch validation)

Applied with the 36 sibling fix.diffs of this batch in findings/poc/_batch41/all_fixes.patch; single in-guest make -j6 nativekernel KERNCONF=X86_64_GENERIC completed RC=0 with zero errors under -Werror (full log: findings/poc/_batch41/fix_build8.log). Behavior A/B intentionally skipped for this Info-tier item.