Hello, On Fri, 26 Jun 2015, Alex Gartrell wrote: > Previously there was a trivial panic > > unshare -n /bin/bash <<EOF > ip addr add dev lo face::1/128 > ipvsadm -A -t [face::1]:15213 > ipvsadm -a -t [face::1]:15213 -r b00c::1 > echo boom | nc face::1 15213 > EOF > > This patch allows us to replicate the net logic above and simply capture > the skb_dst(skb)->dev and use that for the purpose of the invocation. > > Signed-off-by: Alex Gartrell <agartrell@xxxxxx> Looks good to me, thanks! Simon, patch looks suitable for ipvs (net) tree. Acked-by: Julian Anastasov <ja@xxxxxx> > --- > net/netfilter/ipvs/ip_vs_xmit.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/net/netfilter/ipvs/ip_vs_xmit.c b/net/netfilter/ipvs/ip_vs_xmit.c > index bf66a86..b99d806 100644 > --- a/net/netfilter/ipvs/ip_vs_xmit.c > +++ b/net/netfilter/ipvs/ip_vs_xmit.c > @@ -505,6 +505,13 @@ err_put: > return -1; > > err_unreach: > + /* The ip6_link_failure function requires the dev field to be set > + * in order to get the net (further for the sake of fwmark > + * reflection). > + */ > + if (!skb->dev) > + skb->dev = skb_dst(skb)->dev; > + > dst_link_failure(skb); > return -1; > } > -- > Alex Gartrell <agartrell@xxxxxx> Regards -- Julian Anastasov <ja@xxxxxx> -- To unsubscribe from this list: send the line "unsubscribe lvs-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
