netfilter: bridge: don't use nf_bridge_info data to store mac header
br_netfilter maintains an extra state, nf_bridge_info, which is attached to skb via skb->nf_bridge pointer. Amongst other things we use skb->nf_bridge->data to store the original mac header for every processed skb. This is required for ip refragmentation when using conntrack on top of bridge, because ip_fragment doesn't copy it from original skb. However there is no need anymore to do this unconditionally. Move this to the one place where its needed -- when br_netfilter calls ip_fragment(). Also switch to percpu storage for this so we can handle fragmenting without accessing nf_bridge meta data. Only user left is neigh resolution when DNAT is detected, to hold the original source mac address (neigh resolution builds new mac header using bridge mac), so rename ->data and reduce its size to whats needed. Signed-off-by: Florian Westphal <fw@strlen.de> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
This commit is contained in:

committed by
Pablo Neira Ayuso

parent
d64d80a2cd
commit
e70deecbf8
@@ -169,7 +169,7 @@ struct nf_bridge_info {
|
||||
unsigned int mask;
|
||||
struct net_device *physindev;
|
||||
struct net_device *physoutdev;
|
||||
unsigned long data[32 / sizeof(unsigned long)];
|
||||
char neigh_header[8];
|
||||
};
|
||||
#endif
|
||||
|
||||
|
Reference in New Issue
Block a user