diff options
| author |   Colin Ian King <colin.king@canonical.com> | 2017-02-07 10:56:38 +0000 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2017-02-07 14:04:29 -0500 |
| commit | 1f02b5f42f53af516c4f5f747390e66d7a8f0bfe (patch) | |
| tree | a1391b54ede82b4cb1dc8d891b73f46100e41337 /net | |
| parent | sfc: fix an off by one bug (diff) | |
| download | linux-dev-1f02b5f42f53af516c4f5f747390e66d7a8f0bfe.tar.xz linux-dev-1f02b5f42f53af516c4f5f747390e66d7a8f0bfe.zip | |
net: bridge: remove redundant check to see if err is set
The error check on err is redundant as it is being checked
previously each time it has been updated. Remove this redundant
check.
Detected with CoverityScan, CID#140030("Logically dead code")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
| -rw-r--r-- | net/bridge/br_netlink.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/net/bridge/br_netlink.c b/net/bridge/br_netlink.c index 1cbdc5b96aa7..a8f6acd23e30 100644 --- a/net/bridge/br_netlink.c +++ b/net/bridge/br_netlink.c @@ -612,9 +612,6 @@ static int br_afspec(struct net_bridge *br, return err; break; } - - if (err) - return err; } return err; |