OSDN Git Service

net: dsa: hellcreek: Use is_zero_ether_addr() instead of memcmp()
authorZou Wei <zou_wei@huawei.com>
Mon, 7 Jun 2021 02:57:09 +0000 (10:57 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 7 Jun 2021 20:16:36 +0000 (13:16 -0700)
Using is_zero_ether_addr() instead of directly use
memcmp() to determine if the ethernet address is all
zeros.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Zou Wei <zou_wei@huawei.com>
Reviewed-by: Kurt Kanzenbach <kurt@linutronix.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/hirschmann/hellcreek.c

index 4d78219..9fdcc4b 100644 (file)
@@ -927,7 +927,6 @@ static int hellcreek_fdb_dump(struct dsa_switch *ds, int port,
 
        /* Read table */
        for (i = 0; i < hellcreek->fdb_entries; ++i) {
-               unsigned char null_addr[ETH_ALEN] = { 0 };
                struct hellcreek_fdb_entry entry = { 0 };
 
                /* Read entry */
@@ -937,7 +936,7 @@ static int hellcreek_fdb_dump(struct dsa_switch *ds, int port,
                hellcreek_write(hellcreek, 0x00, HR_FDBRDH);
 
                /* Check valid */
-               if (!memcmp(entry.mac, null_addr, ETH_ALEN))
+               if (is_zero_ether_addr(entry.mac))
                        continue;
 
                /* Check port mask */