diff options
| author | David S. Miller <davem@davemloft.net> | 2013-10-01 12:39:35 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-10-01 12:39:35 -0400 |
| commit | e024bdc051ab99eafb5dd9bad87e79afc27f8a44 (patch) | |
| tree | 8c219a19c37c49e89b18e085ffb90b70cc63cb2c /include | |
| parent | 1ed98ed55d6bf47d5a21b1e2db35ceb8b9a4c91c (diff) | |
| parent | f4a87e7bd2eaef26a3ca25437ce8b807de2966ad (diff) | |
Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf
Pablo Neira Ayuso says:
====================
The following patchset contains Netfilter/IPVS fixes for your net
tree, they are:
* Fix BUG_ON splat due to malformed TCP packets seen by synproxy, from
Patrick McHardy.
* Fix possible weight overflow in lblc and lblcr schedulers due to
32-bits arithmetics, from Simon Kirby.
* Fix possible memory access race in the lblc and lblcr schedulers,
introduced when it was converted to use RCU, two patches from
Julian Anastasov.
* Fix hard dependency on CPU 0 when reading per-cpu stats in the
rate estimator, from Julian Anastasov.
* Fix race that may lead to object use after release, when invoking
ipvsadm -C && ipvsadm -R, introduced when adding RCU, from Julian
Anastasov.
====================
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include')
| -rw-r--r-- | include/net/ip_vs.h | 9 | ||||
| -rw-r--r-- | include/net/netfilter/nf_conntrack_synproxy.h | 2 |
2 files changed, 4 insertions, 7 deletions
diff --git a/include/net/ip_vs.h b/include/net/ip_vs.h index f0d70f066f3d..9c4d37ec45a1 100644 --- a/include/net/ip_vs.h +++ b/include/net/ip_vs.h | |||
| @@ -723,8 +723,6 @@ struct ip_vs_dest_dst { | |||
| 723 | struct rcu_head rcu_head; | 723 | struct rcu_head rcu_head; |
| 724 | }; | 724 | }; |
| 725 | 725 | ||
| 726 | /* In grace period after removing */ | ||
| 727 | #define IP_VS_DEST_STATE_REMOVING 0x01 | ||
| 728 | /* | 726 | /* |
| 729 | * The real server destination forwarding entry | 727 | * The real server destination forwarding entry |
| 730 | * with ip address, port number, and so on. | 728 | * with ip address, port number, and so on. |
| @@ -742,7 +740,7 @@ struct ip_vs_dest { | |||
| 742 | 740 | ||
| 743 | atomic_t refcnt; /* reference counter */ | 741 | atomic_t refcnt; /* reference counter */ |
| 744 | struct ip_vs_stats stats; /* statistics */ | 742 | struct ip_vs_stats stats; /* statistics */ |
| 745 | unsigned long state; /* state flags */ | 743 | unsigned long idle_start; /* start time, jiffies */ |
| 746 | 744 | ||
| 747 | /* connection counters and thresholds */ | 745 | /* connection counters and thresholds */ |
| 748 | atomic_t activeconns; /* active connections */ | 746 | atomic_t activeconns; /* active connections */ |
| @@ -756,14 +754,13 @@ struct ip_vs_dest { | |||
| 756 | struct ip_vs_dest_dst __rcu *dest_dst; /* cached dst info */ | 754 | struct ip_vs_dest_dst __rcu *dest_dst; /* cached dst info */ |
| 757 | 755 | ||
| 758 | /* for virtual service */ | 756 | /* for virtual service */ |
| 759 | struct ip_vs_service *svc; /* service it belongs to */ | 757 | struct ip_vs_service __rcu *svc; /* service it belongs to */ |
| 760 | __u16 protocol; /* which protocol (TCP/UDP) */ | 758 | __u16 protocol; /* which protocol (TCP/UDP) */ |
| 761 | __be16 vport; /* virtual port number */ | 759 | __be16 vport; /* virtual port number */ |
| 762 | union nf_inet_addr vaddr; /* virtual IP address */ | 760 | union nf_inet_addr vaddr; /* virtual IP address */ |
| 763 | __u32 vfwmark; /* firewall mark of service */ | 761 | __u32 vfwmark; /* firewall mark of service */ |
| 764 | 762 | ||
| 765 | struct list_head t_list; /* in dest_trash */ | 763 | struct list_head t_list; /* in dest_trash */ |
| 766 | struct rcu_head rcu_head; | ||
| 767 | unsigned int in_rs_table:1; /* we are in rs_table */ | 764 | unsigned int in_rs_table:1; /* we are in rs_table */ |
| 768 | }; | 765 | }; |
| 769 | 766 | ||
| @@ -1649,7 +1646,7 @@ static inline void ip_vs_conn_drop_conntrack(struct ip_vs_conn *cp) | |||
| 1649 | /* CONFIG_IP_VS_NFCT */ | 1646 | /* CONFIG_IP_VS_NFCT */ |
| 1650 | #endif | 1647 | #endif |
| 1651 | 1648 | ||
| 1652 | static inline unsigned int | 1649 | static inline int |
| 1653 | ip_vs_dest_conn_overhead(struct ip_vs_dest *dest) | 1650 | ip_vs_dest_conn_overhead(struct ip_vs_dest *dest) |
| 1654 | { | 1651 | { |
| 1655 | /* | 1652 | /* |
diff --git a/include/net/netfilter/nf_conntrack_synproxy.h b/include/net/netfilter/nf_conntrack_synproxy.h index 806f54a290d6..f572f313d6f1 100644 --- a/include/net/netfilter/nf_conntrack_synproxy.h +++ b/include/net/netfilter/nf_conntrack_synproxy.h | |||
| @@ -56,7 +56,7 @@ struct synproxy_options { | |||
| 56 | 56 | ||
| 57 | struct tcphdr; | 57 | struct tcphdr; |
| 58 | struct xt_synproxy_info; | 58 | struct xt_synproxy_info; |
| 59 | extern void synproxy_parse_options(const struct sk_buff *skb, unsigned int doff, | 59 | extern bool synproxy_parse_options(const struct sk_buff *skb, unsigned int doff, |
| 60 | const struct tcphdr *th, | 60 | const struct tcphdr *th, |
| 61 | struct synproxy_options *opts); | 61 | struct synproxy_options *opts); |
| 62 | extern unsigned int synproxy_options_size(const struct synproxy_options *opts); | 62 | extern unsigned int synproxy_options_size(const struct synproxy_options *opts); |
