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)
PoC verification
Evidence pack
findings/poc/DF-1382 Β· 2 files| File | Type | Description | Size | |
|---|---|---|---|---|
| VERDICT.md | verdict | verification verdict | 751 B | β raw |
| fix.diff | suggested-fix | git-apply-able fix | 335 B | view 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
fixedVALIDATED: 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
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.
No comments yet.