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 | |
| parent | ee467fbaaf8eed86efc335753bd95971aa444f7a (diff) | |
| download | op-kernel-dev-1f02b5f42f53af516c4f5f747390e66d7a8f0bfe.zip op-kernel-dev-1f02b5f42f53af516c4f5f747390e66d7a8f0bfe.tar.gz | |
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>
| -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 1cbdc5b9..a8f6acd 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; |
