From 3e86bf5ce25881dcc50149be62ccc778e640833e Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Mon, 27 Jan 2025 05:05:20 +0000 Subject: util/nvmutil: reset errno when writing a MAC if checksum verification passed, then we should reset in case we're operating on a given part and the last one checked was bad. a catch-all reset is already performed in writeGbe, but it's good to do it here too. in practise, if the 2nd part (part 1) is what failed, errno still wouldn't be reset. Signed-off-by: Leah Rowe --- util/nvmutil/nvmutil.c | 1 + 1 file changed, 1 insertion(+) (limited to 'util/nvmutil/nvmutil.c') diff --git a/util/nvmutil/nvmutil.c b/util/nvmutil/nvmutil.c index f1004ae5..b7174133 100644 --- a/util/nvmutil/nvmutil.c +++ b/util/nvmutil/nvmutil.c @@ -251,6 +251,7 @@ cmd_setmac(void) for (int partnum = 0; partnum < 2; partnum++) { if (!goodChecksum(part = partnum)) continue; + errno = 0; for (int w = 0; w < 3; w++) /* write MAC to gbe part */ setWord(w, partnum, mac[w]); -- cgit v1.2.1