[SK_BUFF]: Introduce skb_mac_header()

For the places where we need a pointer to the mac header, it is still legal to
touch skb->mac.raw directly if just adding to, subtracting from or setting it
to another layer header.

This one also converts some more cases to skb_reset_mac_header() that my
regex missed as it had no spaces before nor after '=', ugh.

Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 3 insertions, 2 deletions

diff --git a/net/bridge/br_netfilter.c b/net/bridge/br_netfilter.c
index 5439a3c46c3e..1163c4f69899 100644
--- a/net/bridge/br_netfilter.c
+++ b/net/bridge/br_netfilter.c
@@ -753,7 +753,8 @@ static unsigned int br_nf_post_routing(unsigned int hook, struct sk_buff **pskb,
 #ifdef CONFIG_NETFILTER_DEBUG
         /* Be very paranoid. This probably won't happen anymore, but let's
          * keep the check just to be sure... */
-        if (skb->mac.raw < skb->head || skb->mac.raw + ETH_HLEN > skb->data) {
+        if (skb_mac_header(skb) < skb->head ||
+            skb_mac_header(skb) + ETH_HLEN > skb->data) {
                 printk(KERN_CRIT "br_netfilter: Argh!! br_nf_post_routing: "
                        "bad mac.raw pointer.\n");
                 goto print_error;
@@ -808,7 +809,7 @@ print_error:
                 if (realoutdev)
                         printk("[%s]", realoutdev->name);
         }
-        printk(" head:%p, raw:%p, data:%p\n", skb->head, skb->mac.raw,
+        printk(" head:%p, raw:%p, data:%p\n", skb->head, skb_mac_header(skb),
                skb->data);
         dump_stack();
         return NF_ACCEPT;