| Message ID | 20260427040011.2107748-1-qingfang.deng@linux.dev |
|---|---|
| State | New |
| Headers | show |
| Series | [Openvpn-devel,ovpn-net-next] ovpn: reset MAC header before passing skb up | expand |
Hi Qingfang, thanks for the patch! On 27/04/2026 06:00, Qingfang Deng wrote: > After decapsulating a packet, the skb->mac_header still points to the > outer transport header. Call skb_reset_mac_header() in > ovpn_netdev_write() to ensure the MAC header points to the beginning of > the inner IP packet. May you elaborate on what this is exactly fixing? Did you encounter a bug triggered by this missing line? I am asking because I wonder what is "expected" as MAC header for a packet not having one at all (packets delivered to the ovpn interface are L3 only, as per the interface type itself). Thanks! Regards, > > Reported-by: Minqiang Chen <ptpt52@gmail.com> > Fixes: 8534731dbf2d ("ovpn: implement packet processing") > Signed-off-by: Qingfang Deng <qingfang.deng@linux.dev> > --- > drivers/net/ovpn/io.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/ovpn/io.c b/drivers/net/ovpn/io.c > index db43a1f8a07a..d92bb87be2b2 100644 > --- a/drivers/net/ovpn/io.c > +++ b/drivers/net/ovpn/io.c > @@ -85,6 +85,7 @@ static void ovpn_netdev_write(struct ovpn_peer *peer, struct sk_buff *skb) > skb_scrub_packet(skb, true); > > /* network header reset in ovpn_decrypt_post() */ > + skb_reset_mac_header(skb); > skb_reset_transport_header(skb); > skb_reset_inner_headers(skb); >
diff --git a/drivers/net/ovpn/io.c b/drivers/net/ovpn/io.c index db43a1f8a07a..d92bb87be2b2 100644 --- a/drivers/net/ovpn/io.c +++ b/drivers/net/ovpn/io.c @@ -85,6 +85,7 @@ static void ovpn_netdev_write(struct ovpn_peer *peer, struct sk_buff *skb) skb_scrub_packet(skb, true); /* network header reset in ovpn_decrypt_post() */ + skb_reset_mac_header(skb); skb_reset_transport_header(skb); skb_reset_inner_headers(skb);
After decapsulating a packet, the skb->mac_header still points to the outer transport header. Call skb_reset_mac_header() in ovpn_netdev_write() to ensure the MAC header points to the beginning of the inner IP packet. Reported-by: Minqiang Chen <ptpt52@gmail.com> Fixes: 8534731dbf2d ("ovpn: implement packet processing") Signed-off-by: Qingfang Deng <qingfang.deng@linux.dev> --- drivers/net/ovpn/io.c | 1 + 1 file changed, 1 insertion(+)