Skip to content

Commit db1a75b

Browse files
Jan Engelhardtdavem330
authored andcommitted
netfilter: xt_TCPMSS: collapse tcpmss_reverse_mtu{4,6} into one function
Signed-off-by: Jan Engelhardt <jengelh@medozas.de> Signed-off-by: Patrick McHardy <kaber@trash.net> Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent 72961ec commit db1a75b

File tree

1 file changed

+13
-29
lines changed

1 file changed

+13
-29
lines changed

net/netfilter/xt_TCPMSS.c

Lines changed: 13 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -147,17 +147,21 @@ tcpmss_mangle_packet(struct sk_buff *skb,
147147
return TCPOLEN_MSS;
148148
}
149149

150-
static u_int32_t tcpmss_reverse_mtu4(const struct iphdr *iph)
150+
static u_int32_t tcpmss_reverse_mtu(const struct sk_buff *skb,
151+
unsigned int family)
151152
{
152-
struct flowi fl = {
153-
.fl4_dst = iph->saddr,
154-
};
153+
struct flowi fl = {};
155154
const struct nf_afinfo *ai;
156155
struct rtable *rt = NULL;
157156
u_int32_t mtu = ~0U;
158157

158+
if (family == PF_INET)
159+
fl.fl4_dst = ip_hdr(skb)->saddr;
160+
else
161+
fl.fl6_dst = ipv6_hdr(skb)->saddr;
162+
159163
rcu_read_lock();
160-
ai = nf_get_afinfo(AF_INET);
164+
ai = nf_get_afinfo(family);
161165
if (ai != NULL)
162166
ai->route((struct dst_entry **)&rt, &fl);
163167
rcu_read_unlock();
@@ -178,7 +182,8 @@ tcpmss_tg4(struct sk_buff *skb, const struct net_device *in,
178182
__be16 newlen;
179183
int ret;
180184

181-
ret = tcpmss_mangle_packet(skb, targinfo, tcpmss_reverse_mtu4(iph),
185+
ret = tcpmss_mangle_packet(skb, targinfo,
186+
tcpmss_reverse_mtu(skb, PF_INET),
182187
iph->ihl * 4,
183188
sizeof(*iph) + sizeof(struct tcphdr));
184189
if (ret < 0)
@@ -193,28 +198,6 @@ tcpmss_tg4(struct sk_buff *skb, const struct net_device *in,
193198
}
194199

195200
#if defined(CONFIG_IP6_NF_IPTABLES) || defined(CONFIG_IP6_NF_IPTABLES_MODULE)
196-
static u_int32_t tcpmss_reverse_mtu6(const struct ipv6hdr *iph)
197-
{
198-
struct flowi fl = {
199-
.fl6_dst = iph->saddr,
200-
};
201-
const struct nf_afinfo *ai;
202-
struct rtable *rt = NULL;
203-
u_int32_t mtu = ~0U;
204-
205-
rcu_read_lock();
206-
ai = nf_get_afinfo(AF_INET6);
207-
if (ai != NULL)
208-
ai->route((struct dst_entry **)&rt, &fl);
209-
rcu_read_unlock();
210-
211-
if (rt != NULL) {
212-
mtu = dst_mtu(&rt->u.dst);
213-
dst_release(&rt->u.dst);
214-
}
215-
return mtu;
216-
}
217-
218201
static unsigned int
219202
tcpmss_tg6(struct sk_buff *skb, const struct net_device *in,
220203
const struct net_device *out, unsigned int hooknum,
@@ -229,7 +212,8 @@ tcpmss_tg6(struct sk_buff *skb, const struct net_device *in,
229212
tcphoff = ipv6_skip_exthdr(skb, sizeof(*ipv6h), &nexthdr);
230213
if (tcphoff < 0)
231214
return NF_DROP;
232-
ret = tcpmss_mangle_packet(skb, targinfo, tcpmss_reverse_mtu6(ipv6h),
215+
ret = tcpmss_mangle_packet(skb, targinfo,
216+
tcpmss_reverse_mtu(skb, PF_INET6),
233217
tcphoff,
234218
sizeof(*ipv6h) + sizeof(struct tcphdr));
235219
if (ret < 0)

0 commit comments

Comments
 (0)