net: bridge: minor style adjustments in br_handle_frame_finish
authorNikolay Aleksandrov <nikolay@cumulusnetworks.com>
Thu, 14 Jul 2016 03:09:59 +0000 (06:09 +0300)
committerDavid S. Miller <davem@davemloft.net>
Sun, 17 Jul 2016 02:57:37 +0000 (19:57 -0700)
Trivial style changes in br_handle_frame_finish.

Signed-off-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br_input.c

index a7817e6f306ffcc462428037abab9c202979afe9..0b6d3261946841e838ed218fbfe9d950ca0a20f8 100644 (file)
@@ -131,11 +131,11 @@ static void br_do_proxy_arp(struct sk_buff *skb, struct net_bridge *br,
 /* note: already called with rcu_read_lock */
 int br_handle_frame_finish(struct net *net, struct sock *sk, struct sk_buff *skb)
 {
-       const unsigned char *dest = eth_hdr(skb)->h_dest;
        struct net_bridge_port *p = br_port_get_rcu(skb->dev);
-       struct net_bridge *br;
-       struct net_bridge_fdb_entry *dst;
+       const unsigned char *dest = eth_hdr(skb)->h_dest;
+       struct net_bridge_fdb_entry *dst = NULL;
        struct net_bridge_mdb_entry *mdst;
+       struct net_bridge *br;
        struct sk_buff *skb2;
        bool unicast = true;
        u16 vid = 0;
@@ -166,8 +166,6 @@ int br_handle_frame_finish(struct net *net, struct sock *sk, struct sk_buff *skb
        if (br->dev->flags & IFF_PROMISC)
                skb2 = skb;
 
-       dst = NULL;
-
        if (IS_ENABLED(CONFIG_INET) && skb->protocol == htons(ETH_P_ARP))
                br_do_proxy_arp(skb, br, vid, p);
 
@@ -185,13 +183,12 @@ int br_handle_frame_finish(struct net *net, struct sock *sk, struct sk_buff *skb
                        skb = NULL;
                        if (!skb2)
                                goto out;
-               } else
+               } else {
                        skb2 = skb;
-
+               }
                unicast = false;
                br->dev->stats.multicast++;
-       } else if ((dst = __br_fdb_get(br, dest, vid)) &&
-                       dst->is_local) {
+       } else if ((dst = __br_fdb_get(br, dest, vid)) && dst->is_local) {
                skb2 = skb;
                /* Do not forward the packet since it's local. */
                skb = NULL;
@@ -201,8 +198,9 @@ int br_handle_frame_finish(struct net *net, struct sock *sk, struct sk_buff *skb
                if (dst) {
                        dst->used = jiffies;
                        br_forward(dst->dst, skb, skb2);
-               } else
+               } else {
                        br_flood_forward(br, skb, skb2, unicast);
+               }
        }
 
        if (skb2)