[PATCH] net: dsa: return success if there was nothing to do

From: trix
Date: Sat Mar 05 2022 - 12:15:03 EST


From: Tom Rix <trix@xxxxxxxxxx>

Clang static analysis reports this representative issue
dsa.c:486:2: warning: Undefined or garbage value
returned to caller
return err;
^~~~~~~~~~

err is only set in the loop. If the loop is empty,
garbage will be returned. So initialize err to 0
to handle this noop case.

Signed-off-by: Tom Rix <trix@xxxxxxxxxx>
---
net/dsa/dsa.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/dsa/dsa.c b/net/dsa/dsa.c
index 06d5de28a43ea..fe971a2c15cd1 100644
--- a/net/dsa/dsa.c
+++ b/net/dsa/dsa.c
@@ -471,7 +471,7 @@ int dsa_port_walk_fdbs(struct dsa_switch *ds, int port, dsa_fdb_walk_cb_t cb)
{
struct dsa_port *dp = dsa_to_port(ds, port);
struct dsa_mac_addr *a;
- int err;
+ int err = 0;

mutex_lock(&dp->addr_lists_lock);

@@ -491,7 +491,7 @@ int dsa_port_walk_mdbs(struct dsa_switch *ds, int port, dsa_fdb_walk_cb_t cb)
{
struct dsa_port *dp = dsa_to_port(ds, port);
struct dsa_mac_addr *a;
- int err;
+ int err = 0;

mutex_lock(&dp->addr_lists_lock);

--
2.26.3