bridge: mdb: reduce the indentation level in br_mdb_fill_info
authorNikolay Aleksandrov <nikolay@cumulusnetworks.com>
Tue, 16 Feb 2016 11:46:53 +0000 (12:46 +0100)
committerDavid S. Miller <davem@davemloft.net>
Fri, 19 Feb 2016 20:27:36 +0000 (15:27 -0500)
Switch the port check and skip if it's null, this allows us to reduce one
indentation level.

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

index ac089286526ef8b94f39b869275b1f5ff07834ad..e66619171386ee1eb012b0afb64237e663177bda 100644 (file)
@@ -88,25 +88,26 @@ static int br_mdb_fill_info(struct sk_buff *skb, struct netlink_callback *cb,
                        for (pp = &mp->ports;
                             (p = rcu_dereference(*pp)) != NULL;
                              pp = &p->next) {
+                               struct br_mdb_entry e;
+
                                port = p->port;
-                               if (port) {
-                                       struct br_mdb_entry e;
-                                       memset(&e, 0, sizeof(e));
-                                       e.ifindex = port->dev->ifindex;
-                                       e.vid = p->addr.vid;
-                                       __mdb_entry_fill_flags(&e, p->flags);
-                                       if (p->addr.proto == htons(ETH_P_IP))
-                                               e.addr.u.ip4 = p->addr.u.ip4;
+                               if (!port)
+                                       continue;
+                               memset(&e, 0, sizeof(e));
+                               e.ifindex = port->dev->ifindex;
+                               e.vid = p->addr.vid;
+                               __mdb_entry_fill_flags(&e, p->flags);
+                               if (p->addr.proto == htons(ETH_P_IP))
+                                       e.addr.u.ip4 = p->addr.u.ip4;
 #if IS_ENABLED(CONFIG_IPV6)
-                                       if (p->addr.proto == htons(ETH_P_IPV6))
-                                               e.addr.u.ip6 = p->addr.u.ip6;
+                               if (p->addr.proto == htons(ETH_P_IPV6))
+                                       e.addr.u.ip6 = p->addr.u.ip6;
 #endif
-                                       e.addr.proto = p->addr.proto;
-                                       if (nla_put(skb, MDBA_MDB_ENTRY_INFO, sizeof(e), &e)) {
-                                               nla_nest_cancel(skb, nest2);
-                                               err = -EMSGSIZE;
-                                               goto out;
-                                       }
+                               e.addr.proto = p->addr.proto;
+                               if (nla_put(skb, MDBA_MDB_ENTRY_INFO, sizeof(e), &e)) {
+                                       nla_nest_cancel(skb, nest2);
+                                       err = -EMSGSIZE;
+                                       goto out;
                                }
                        }
                        nla_nest_end(skb, nest2);