On Wed, Jan 03, 2018 at 10:48:03PM +0000, James Chapman wrote:
> This patch series removes all code to support a configurable offset in
> transmitted l2tp packets. Code to handle this is incomplete and buggy
> and has been this way for years. If anyone tried to configure an
> offset, it would be ignored for L2TPv2 tunnels, or for L2TPv3 tunnels,
> could result in L2TPv3 packets being transmitted which are not
> compliant with L2TPv3 RFC3931. This patch series removes the support
> for configurable offsets.
>
Thanks for the detailed cover letter!
There are a few more places talking about offsets:
* Description of SESSION_CREATE in include/uapi/linux/l2tp.h.
* Description of L2TPv3 packet format above l2tp_recv_common() in
net/l2tp/l2tp_core.c.
I can submit a followup patch to remove them.
Reviewed-by: Guillaume Nault <g.na...@alphalink.fr>
Tested-by: Guillaume Nault <g.na...@alphalink.fr>
