From afb60e7112d44d7b38226554bef366503f59d5c5 Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Tue, 10 Nov 2020 17:10:01 +0100 Subject: [PATCH] mb/siemens/mc_apl1: Simplify is_mac_adr_valid() logic A MAC address that is neither 00:00:00:00:00:00 nor ff:ff:ff:ff:ff:ff is considered valid. Instead of using a temporary buffer and memcmp(), use a single loop that exits as soon as the MAC cannot possibly be invalid. Change-Id: I2b15b510092860fbbefd150c9060da38aeb13311 Signed-off-by: Angel Pons Reviewed-on: https://review.coreboot.org/c/coreboot/+/47405 Tested-by: build bot (Jenkins) Reviewed-by: Werner Zeh Reviewed-by: Mario Scheithauer Reviewed-by: Uwe Poeche --- src/mainboard/siemens/mc_apl1/mainboard.c | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/src/mainboard/siemens/mc_apl1/mainboard.c b/src/mainboard/siemens/mc_apl1/mainboard.c index ff0cc597bc..9540d6dca2 100644 --- a/src/mainboard/siemens/mc_apl1/mainboard.c +++ b/src/mainboard/siemens/mc_apl1/mainboard.c @@ -35,15 +35,13 @@ */ static uint8_t is_mac_adr_valid(uint8_t mac[MAC_ADDR_LEN]) { - uint8_t buf[MAC_ADDR_LEN]; - - memset(buf, 0, sizeof(buf)); - if (!memcmp(buf, mac, sizeof(buf))) - return 0; - memset(buf, 0xff, sizeof(buf)); - if (!memcmp(buf, mac, sizeof(buf))) - return 0; - return 1; + for (size_t i = 0; i < MAC_ADDR_LEN; i++) { + if (mac[i] != 0x00 && mac[i] != 0xff) + return 1; + if (mac[i] != mac[0]) + return 1; + } + return 0; } /** \brief This function will search for a MAC address which can be assigned