netfilter: nf_conntrack: add nf_ct_kill()
Encapsulate the common if (del_timer(&ct->timeout)) ct->timeout.function((unsigned long)ct) sequence in a new function. Signed-off-by: Patrick McHardy <kaber@trash.net> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
31d8519c9c
commit
51091764f2
|
@ -223,6 +223,8 @@ static inline void nf_ct_refresh(struct nf_conn *ct,
|
|||
__nf_ct_refresh_acct(ct, 0, skb, extra_jiffies, 0);
|
||||
}
|
||||
|
||||
extern void nf_ct_kill(struct nf_conn *ct);
|
||||
|
||||
/* These are for NAT. Icky. */
|
||||
/* Update TCP window tracking data when NAT mangles the packet */
|
||||
extern void nf_conntrack_tcp_update(const struct sk_buff *skb,
|
||||
|
|
|
@ -87,9 +87,8 @@ static int icmp_packet(struct nf_conn *ct,
|
|||
means this will only run once even if count hits zero twice
|
||||
(theoretically possible with SMP) */
|
||||
if (CTINFO2DIR(ctinfo) == IP_CT_DIR_REPLY) {
|
||||
if (atomic_dec_and_test(&ct->proto.icmp.count)
|
||||
&& del_timer(&ct->timeout))
|
||||
ct->timeout.function((unsigned long)ct);
|
||||
if (atomic_dec_and_test(&ct->proto.icmp.count))
|
||||
nf_ct_kill(ct);
|
||||
} else {
|
||||
atomic_inc(&ct->proto.icmp.count);
|
||||
nf_conntrack_event_cache(IPCT_PROTOINFO_VOLATILE, skb);
|
||||
|
|
|
@ -89,9 +89,8 @@ static int icmpv6_packet(struct nf_conn *ct,
|
|||
means this will only run once even if count hits zero twice
|
||||
(theoretically possible with SMP) */
|
||||
if (CTINFO2DIR(ctinfo) == IP_CT_DIR_REPLY) {
|
||||
if (atomic_dec_and_test(&ct->proto.icmp.count)
|
||||
&& del_timer(&ct->timeout))
|
||||
ct->timeout.function((unsigned long)ct);
|
||||
if (atomic_dec_and_test(&ct->proto.icmp.count))
|
||||
nf_ct_kill(ct);
|
||||
} else {
|
||||
atomic_inc(&ct->proto.icmp.count);
|
||||
nf_conntrack_event_cache(IPCT_PROTOINFO_VOLATILE, skb);
|
||||
|
|
|
@ -848,6 +848,13 @@ acct:
|
|||
}
|
||||
EXPORT_SYMBOL_GPL(__nf_ct_refresh_acct);
|
||||
|
||||
void nf_ct_kill(struct nf_conn *ct)
|
||||
{
|
||||
if (del_timer(&ct->timeout))
|
||||
ct->timeout.function((unsigned long)ct);
|
||||
}
|
||||
EXPORT_SYMBOL_GPL(nf_ct_kill);
|
||||
|
||||
#if defined(CONFIG_NF_CT_NETLINK) || defined(CONFIG_NF_CT_NETLINK_MODULE)
|
||||
|
||||
#include <linux/netfilter/nfnetlink.h>
|
||||
|
|
|
@ -812,9 +812,8 @@ ctnetlink_del_conntrack(struct sock *ctnl, struct sk_buff *skb,
|
|||
return -ENOENT;
|
||||
}
|
||||
}
|
||||
if (del_timer(&ct->timeout))
|
||||
ct->timeout.function((unsigned long)ct);
|
||||
|
||||
nf_ct_kill(ct);
|
||||
nf_ct_put(ct);
|
||||
|
||||
return 0;
|
||||
|
|
|
@ -475,8 +475,7 @@ static int dccp_packet(struct nf_conn *ct, const struct sk_buff *skb,
|
|||
if (type == DCCP_PKT_RESET &&
|
||||
!test_bit(IPS_SEEN_REPLY_BIT, &ct->status)) {
|
||||
/* Tear down connection immediately if only reply is a RESET */
|
||||
if (del_timer(&ct->timeout))
|
||||
ct->timeout.function((unsigned long)ct);
|
||||
nf_ct_kill(ct);
|
||||
return NF_ACCEPT;
|
||||
}
|
||||
|
||||
|
|
|
@ -843,8 +843,7 @@ static int tcp_packet(struct nf_conn *ct,
|
|||
/* Attempt to reopen a closed/aborted connection.
|
||||
* Delete this connection and look up again. */
|
||||
write_unlock_bh(&tcp_lock);
|
||||
if (del_timer(&ct->timeout))
|
||||
ct->timeout.function((unsigned long)ct);
|
||||
nf_ct_kill(ct);
|
||||
return -NF_REPEAT;
|
||||
}
|
||||
/* Fall through */
|
||||
|
@ -877,8 +876,7 @@ static int tcp_packet(struct nf_conn *ct,
|
|||
if (LOG_INVALID(IPPROTO_TCP))
|
||||
nf_log_packet(pf, 0, skb, NULL, NULL, NULL,
|
||||
"nf_ct_tcp: killing out of sync session ");
|
||||
if (del_timer(&ct->timeout))
|
||||
ct->timeout.function((unsigned long)ct);
|
||||
nf_ct_kill(ct);
|
||||
return -NF_DROP;
|
||||
}
|
||||
ct->proto.tcp.last_index = index;
|
||||
|
@ -961,8 +959,7 @@ static int tcp_packet(struct nf_conn *ct,
|
|||
problem case, so we can delete the conntrack
|
||||
immediately. --RR */
|
||||
if (th->rst) {
|
||||
if (del_timer(&ct->timeout))
|
||||
ct->timeout.function((unsigned long)ct);
|
||||
nf_ct_kill(ct);
|
||||
return NF_ACCEPT;
|
||||
}
|
||||
} else if (!test_bit(IPS_ASSURED_BIT, &ct->status)
|
||||
|
|
Loading…
Reference in New Issue