netfilter: ipvs: Remove useless ipvsh param of frag_safe_skb_hp
The param of frag_safe_skb_hp, ipvsh, isn't used now. So remove it and update the callers' codes too. Signed-off-by: Gao Feng <gfree.wind@vip.163.com> Acked-by: Simon Horman <horms+renesas@verge.net.au> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
This commit is contained in:
parent
2c9e8637ea
commit
6b3d933000
|
@ -69,8 +69,7 @@ struct ip_vs_iphdr {
|
||||||
};
|
};
|
||||||
|
|
||||||
static inline void *frag_safe_skb_hp(const struct sk_buff *skb, int offset,
|
static inline void *frag_safe_skb_hp(const struct sk_buff *skb, int offset,
|
||||||
int len, void *buffer,
|
int len, void *buffer)
|
||||||
const struct ip_vs_iphdr *ipvsh)
|
|
||||||
{
|
{
|
||||||
return skb_header_pointer(skb, offset, len, buffer);
|
return skb_header_pointer(skb, offset, len, buffer);
|
||||||
}
|
}
|
||||||
|
|
|
@ -322,7 +322,7 @@ ip_vs_conn_fill_param_proto(struct netns_ipvs *ipvs,
|
||||||
{
|
{
|
||||||
__be16 _ports[2], *pptr;
|
__be16 _ports[2], *pptr;
|
||||||
|
|
||||||
pptr = frag_safe_skb_hp(skb, iph->len, sizeof(_ports), _ports, iph);
|
pptr = frag_safe_skb_hp(skb, iph->len, sizeof(_ports), _ports);
|
||||||
if (pptr == NULL)
|
if (pptr == NULL)
|
||||||
return 1;
|
return 1;
|
||||||
|
|
||||||
|
|
|
@ -433,7 +433,7 @@ ip_vs_schedule(struct ip_vs_service *svc, struct sk_buff *skb,
|
||||||
/*
|
/*
|
||||||
* IPv6 frags, only the first hit here.
|
* IPv6 frags, only the first hit here.
|
||||||
*/
|
*/
|
||||||
pptr = frag_safe_skb_hp(skb, iph->len, sizeof(_ports), _ports, iph);
|
pptr = frag_safe_skb_hp(skb, iph->len, sizeof(_ports), _ports);
|
||||||
if (pptr == NULL)
|
if (pptr == NULL)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
|
@ -566,7 +566,7 @@ int ip_vs_leave(struct ip_vs_service *svc, struct sk_buff *skb,
|
||||||
struct netns_ipvs *ipvs = svc->ipvs;
|
struct netns_ipvs *ipvs = svc->ipvs;
|
||||||
struct net *net = ipvs->net;
|
struct net *net = ipvs->net;
|
||||||
|
|
||||||
pptr = frag_safe_skb_hp(skb, iph->len, sizeof(_ports), _ports, iph);
|
pptr = frag_safe_skb_hp(skb, iph->len, sizeof(_ports), _ports);
|
||||||
if (!pptr)
|
if (!pptr)
|
||||||
return NF_DROP;
|
return NF_DROP;
|
||||||
dport = likely(!ip_vs_iph_inverse(iph)) ? pptr[1] : pptr[0];
|
dport = likely(!ip_vs_iph_inverse(iph)) ? pptr[1] : pptr[0];
|
||||||
|
@ -982,7 +982,7 @@ static int ip_vs_out_icmp_v6(struct netns_ipvs *ipvs, struct sk_buff *skb,
|
||||||
unsigned int offset;
|
unsigned int offset;
|
||||||
|
|
||||||
*related = 1;
|
*related = 1;
|
||||||
ic = frag_safe_skb_hp(skb, ipvsh->len, sizeof(_icmph), &_icmph, ipvsh);
|
ic = frag_safe_skb_hp(skb, ipvsh->len, sizeof(_icmph), &_icmph);
|
||||||
if (ic == NULL)
|
if (ic == NULL)
|
||||||
return NF_DROP;
|
return NF_DROP;
|
||||||
|
|
||||||
|
@ -1214,7 +1214,7 @@ static struct ip_vs_conn *__ip_vs_rs_conn_out(unsigned int hooknum,
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
pptr = frag_safe_skb_hp(skb, iph->len,
|
pptr = frag_safe_skb_hp(skb, iph->len,
|
||||||
sizeof(_ports), _ports, iph);
|
sizeof(_ports), _ports);
|
||||||
if (!pptr)
|
if (!pptr)
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
|
@ -1407,7 +1407,7 @@ ip_vs_out(struct netns_ipvs *ipvs, unsigned int hooknum, struct sk_buff *skb, in
|
||||||
__be16 _ports[2], *pptr;
|
__be16 _ports[2], *pptr;
|
||||||
|
|
||||||
pptr = frag_safe_skb_hp(skb, iph.len,
|
pptr = frag_safe_skb_hp(skb, iph.len,
|
||||||
sizeof(_ports), _ports, &iph);
|
sizeof(_ports), _ports);
|
||||||
if (pptr == NULL)
|
if (pptr == NULL)
|
||||||
return NF_ACCEPT; /* Not for me */
|
return NF_ACCEPT; /* Not for me */
|
||||||
if (ip_vs_has_real_service(ipvs, af, iph.protocol, &iph.saddr,
|
if (ip_vs_has_real_service(ipvs, af, iph.protocol, &iph.saddr,
|
||||||
|
@ -1741,7 +1741,7 @@ static int ip_vs_in_icmp_v6(struct netns_ipvs *ipvs, struct sk_buff *skb,
|
||||||
|
|
||||||
*related = 1;
|
*related = 1;
|
||||||
|
|
||||||
ic = frag_safe_skb_hp(skb, iph->len, sizeof(_icmph), &_icmph, iph);
|
ic = frag_safe_skb_hp(skb, iph->len, sizeof(_icmph), &_icmph);
|
||||||
if (ic == NULL)
|
if (ic == NULL)
|
||||||
return NF_DROP;
|
return NF_DROP;
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue