DF-3670 / verdict.json
{ "finding_id": "DF-3670", "status": "inconclusive", "reproduced": 0, "impact": "none", "confidence": "certain", "verdict": "Source-level certain (upstream divergence proven by diff): ixgbe_calc_eeprom_checksum_generic (ixgbe_common.c:2211-2262) walks NVM sections using NVM-resident pointer (2233) and length (2242) with a u16 loop index (2214, loop at 2250) and no bound against hw->eeprom.word_size (itself EEC.SIZE-derived, 1277-1280). FreeBSDs copy of the same shared code added pointer/length range checks and u32 indexing; DragonFlys 2017 snapshot lacks them. Crafted/corrupt NVM -> checksum computed over wrong words: attach-time validate (if_ix.c:615) wrongly fails (NIC-disable DoS) or wrongly passes; update path (ixgbe_common.c:2320-2346) writes a wrong checksum into NVM (persistent corruption). Hardware-gated: guest has no ixgbe device (env.txt). fix.diff (upstream bounds) compile-validated -Werror-clean on md5-identical guest source.", "exploit_chain": "", "evidence": "VERDICT.md (walk-through + upstream diff excerpt); build_patched.log; env.txt", "kernel_refs": [ "sys/dev/netif/ix/ixgbe_common.c:2211", "sys/dev/netif/ix/ixgbe_common.c:2233", "sys/dev/netif/ix/ixgbe_common.c:2250", "sys/dev/netif/ix/ixgbe_common.c:1277", "sys/dev/netif/ix/ixgbe_common.c:2320", "sys/dev/netif/ix/if_ix.c:615" ], "poc_changes": "New pass-2 finding, no seed. fix.diff authored from upstream FreeBSDs bound checks, minimized to local style.", "attempts": 1, "guest_uname": "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", "runtime_sec": 60, "guest_dirty": 0, "build_cmd": "cd /usr/src && patch -p1 < fix.diff && cd sys/dev/netif/ix && make obj && make", "run_cmd": "n/a (needs ixgbe NIC with crafted NVM)", "code_hash": "fb2f27f9401e883665c74821f4d06a933980b11b6cc120142bc7097e56284ba6", "notes": "Same trust class as DF-3642 (e1000 flash erase). Divergence diff stored at /tmp/opencode/fx_ixgbe_common.c during the run; excerpt embedded in VERDICT.md.", "recommended_fix": "Import upstream bounds: reject pointer >= word_size and length >= word_size - pointer before the walk (fix.diff).", "fix_status": "not_testable", "fix_baseline_reproduced": 0, "fix_patched_reproduced": 0, "fix_verdict": "Compile-validated only; runtime validation needs hardware.", "fix_evidence": "build_baseline.log / build_patched.log (-Werror clean with fix)." } |