Add validation for unused parameter values in the gap between
VXLAN_PARAM_WITH_LOCAL_ADDR4 and VXLAN_PARAM_WITH_LOCAL_ADDR6 to prevent
panics.
PR: 297151
Reported by: Robert Morris <rtm@lcs.mit.edu>
MFC after: 3 days
Differential D58552
if_vxlan(4): fix panic by validating unused drvspec values Authored by pouria on Thu, Jul 30, 1:19 PM. Tags None Referenced Files
Details Add validation for unused parameter values in the gap between PR: 297151 See PR297151
Diff Detail
Event TimelineComment Actions I also can fix this by adding extra validation check in vxlan_ioctl_drvspec() to avoid renumbering. MFC after: 3 days Comment Actions This breaks the userspace ABI. Old ifconfig binaries won't work properly with a new kernel after this change. Comment Actions I understand, that's why I said I can do it the other way. Comment Actions Yes, in general FreeBSD tries quite hard to avoid breaking the kernel<->userspace ABI. There are exceptions, but if it's easy to avoid this breakage, then we should. It's very useful to be able to upgrade the kernel without upgrading userspace, and one might want an ifconfig binary from a jail to continue working even if the host is upgraded.
| ||||||||||||||||||||||||||||||||||