diff options
| author |   Ido Schimmel <idosch@mellanox.com> | 2017-07-12 09:12:52 +0200 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2017-07-12 08:15:51 -0700 |
| commit | a4e75b76b2ce47e3d964c87689962abb83cc4958 (patch) | |
| tree | 8300bdfaa9fee4bf84a98be92a8d8f2dd864efd5 /drivers | |
| parent | samples/bpf: fix a build issue (diff) | |
| download | wireguard-linux-a4e75b76b2ce47e3d964c87689962abb83cc4958.tar.xz wireguard-linux-a4e75b76b2ce47e3d964c87689962abb83cc4958.zip | |
mlxsw: spectrum_router: Add missing rollback
With this patch the error path of mlxsw_sp_nexthop_init() is symmetric
with mlxsw_sp_nexthop_fini(). Noticed during code review.
Fixes: a8c970142798 ("mlxsw: spectrum_router: Refactor nexthop init routine")
Signed-off-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c b/drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c index 192cb93e7669..129afc168fd9 100644 --- a/drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c +++ b/drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c @@ -1790,6 +1790,7 @@ static int mlxsw_sp_nexthop_init(struct mlxsw_sp *mlxsw_sp, return 0; err_nexthop_neigh_init: + mlxsw_sp_nexthop_rif_fini(nh); mlxsw_sp_nexthop_remove(mlxsw_sp, nh); return err; } |