On Wed, Feb 17, 2021 at 10:40 AM John Fastabend <john.fastab...@gmail.com> wrote: > > @@ -802,9 +809,10 @@ int sk_psock_tls_strp_read(struct sk_psock *psock, > > struct sk_buff *skb) > > * TLS context. > > */ > > skb->sk = psock->sk; > > - tcp_skb_bpf_redirect_clear(skb); > > + skb_dst_drop(skb); > > + skb_bpf_redirect_clear(skb); > > Do we really need the skb_dst_drop() I thought we would have already dropped > this here > but I've not had time to check yet.
Yes, I got some serious complaints from dst_release() when I didn't add skb_dst_drop(). Thanks.
