On 31.05.2016 20:39, David Miller wrote: > From: Alan Davey <alan.da...@metaswitch.com> > Date: Mon, 23 May 2016 15:23:45 +0100 > >> One of the bugs documented in the raw(7) man page is as follows: When the >> IP_HDRINCL option is set, datagrams will not be fragmented and are limited to >> the interface MTU. >> >> This patch fixes the bug by removing the check for "length > >> rt->dst.dev->mtu" >> in raw_send_hdrinc() (net/ipv4/raw.c). Datagrams are no longer limited to >> the >> interface MTU size if the IP_HDRINCL option is set, but are fragmented, if >> necessary, in the same way as all other datagrams. >> >> Signed-off-by: Alan Davey <alan.da...@metaswitch.com> > > This is not a bug, it's a feature and it's how RAW ipv4 sockets have behaved > for two decades. > > If the user wants to use hdr inclusion, he can send multiple frames and set > the fragmentation bits appropriately.
Actually this is difficult, as only the kernel is in control of the IP_ID generator and it should never leak its state to user space. The probability of IP_ID collisions would increase drastically because of this, even imagine different programs doing so concurrently. Basically those checks are pretty much unnecessary anyway, if we set raw sockets by default into inet->pmtudisc IPPMTUDISC_DO mode, so a user can easily control the fragmentation on a raw socket themselves based on already provided infrastructure. Bye, Hannes
