Skip to content

Commit ad248f7

Browse files
Colin Ian Kingdavem330
Colin Ian King
authored andcommitted
net: bridge: Fix missing return assignment from br_vlan_replay_one call
The call to br_vlan_replay_one is returning an error return value but this is not being assigned to err and the following check on err is currently always false because err was initialized to zero. Fix this by assigning err. Addresses-Coverity: ("'Constant' variable guards dead code") Fixes: 22f67cd ("net: bridge: add helper to replay VLANs installed on port") Signed-off-by: Colin Ian King <[email protected]> Reviewed-by: Vladimir Oltean <[email protected]> Acked-by: Nikolay Aleksandrov <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent bb11d9a commit ad248f7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

net/bridge/br_vlan.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1815,7 +1815,7 @@ int br_vlan_replay(struct net_device *br_dev, struct net_device *dev,
18151815
if (!br_vlan_should_use(v))
18161816
continue;
18171817

1818-
br_vlan_replay_one(nb, dev, &vlan, extack);
1818+
err = br_vlan_replay_one(nb, dev, &vlan, extack);
18191819
if (err)
18201820
return err;
18211821
}

0 commit comments

Comments
 (0)