diff options
author | 2021-01-07 16:48:21 +0200 | |
---|---|---|
committer | 2021-01-07 18:47:18 -0800 | |
commit | 07e61a979ca4dddb3661f59328b3cd109f6b0070 (patch) | |
tree | c10b02e6faaae21b4ff301d91dd2a7eb126d2c5f /net | |
parent | octeontx2-af: fix memory leak of lmac and lmac->name (diff) | |
download | linux-07e61a979ca4dddb3661f59328b3cd109f6b0070.tar.gz linux-07e61a979ca4dddb3661f59328b3cd109f6b0070.tar.bz2 linux-07e61a979ca4dddb3661f59328b3cd109f6b0070.zip |
nexthop: Fix off-by-one error in error path
A reference was not taken for the current nexthop entry, so do not try
to put it in the error path.
Fixes: 430a049190de ("nexthop: Add support for nexthop groups")
Signed-off-by: Ido Schimmel <idosch@nvidia.com>
Reviewed-by: Petr Machata <petrm@nvidia.com>
Reviewed-by: David Ahern <dsahern@kernel.org>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'net')
-rw-r--r-- | net/ipv4/nexthop.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/nexthop.c b/net/ipv4/nexthop.c index 5e1b22d4f939..f8035cfa9c20 100644 --- a/net/ipv4/nexthop.c +++ b/net/ipv4/nexthop.c @@ -1459,7 +1459,7 @@ static struct nexthop *nexthop_create_group(struct net *net, return nh; out_no_nh: - for (; i >= 0; --i) + for (i--; i >= 0; --i) nexthop_put(nhg->nh_entries[i].nh); kfree(nhg->spare); |