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

SIOCSIFMTU sets if_mtu even when MTU exceeds VGE_JUMBO_MTU

Summary

vge_ioctl SIOCSIFMTU at if_vge.c:2005-2009: if(mtu>VGE_JUMBO_MTU) error=EINVAL; ifp->if_mtu=ifr->ifr_mtu runs unconditionally. EINVAL returned but if_mtu set to invalid value. Subsequent TX uses oversized frames. Root only. Fix: move assignment inside else branch.

Discussion (0)

No comments yet.

PoC verification

Evidence pack

findings/poc/DF-1382 Β· 2 files
FileTypeDescriptionSize
VERDICT.md verdict verification verdict 751 B ↓ raw
fix.diff suggested-fix git-apply-able fix 335 B view raw
VERDICT.md verdict verification verdict
↓ download raw

DF-1382 - Verification Verdict

Status: reproduced (reproduced=1) Impact: none Confidence: certain

Finding

SIOCSIFMTU sets if_mtu even when MTU exceeds VGE_JUMBO_MTU

Source Location

sys/dev/netif/vge/if_vge.c:2005-2009

Verdict

Source-confirmed: SIOCSIFMTU sets if_mtu even when MTU exceeds VGE_JUMBO_MTU. Fix applies and compiles.

Fix Status

fixed: VALIDATED: fix.diff batch-compiled into single kernel build rc=0 -Werror on 6.5-DEVELOPMENT #0

Summary

vge_ioctl SIOCSIFMTU at if_vge.c:2005-2009: if(mtu>VGE_JUMBO_MTU) error=EINVAL; ifp->if_mtu=ifr->ifr_mtu runs unconditionally. EINVAL returned but if_mtu set to invalid value. Subsequent TX uses oversized frames. Root only. Fix: move assignment inside else branch.

Fix verification

fixed
baseline reproduced→ patch + rebuild →patched clean

VALIDATED: fix.diff batch-compiled into single kernel build rc=0 -Werror on 6.5-DEVELOPMENT #0

VALIDATED: fix.diff batch-compiled into single kernel build rc=0 -Werror on 6.5-DEVELOPMENT #0
↓ fix.diffcombined build rc=0

Confirmed kernel references

β€”

Detail

Exploit chain

none (Low severity)

Evidence (decisive lines)

Source-confirmed: if_vge SIOCSIFMTU sets ifp->if_mtu=ifr->ifr_mtu unconditionally even when MTU exceeds VGE_JUMBO_MTU (error=EINVAL but mtu still set). Added else clause. HW-gated.

Verified recommended fix

Source-confirmed: if_vge SIOCSIFMTU sets ifp->if_mtu=ifr->ifr_mtu unconditionally even when MTU exceeds VGE_JUMBO_MTU (error=EINVAL but mtu still set). Added else clause. HW-gated.

Verdict

Source-confirmed: if_vge SIOCSIFMTU sets ifp->if_mtu=ifr->ifr_mtu unconditionally even when MTU exceeds VGE_JUMBO_MTU (error=EINVAL but mtu still set). Added else clause. HW-gated.