diff options
author | Jakub Kicinski <jakub.kicinski@netronome.com> | 2018-07-25 18:39:27 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-07-26 01:33:39 -0400 |
commit | c259b4fb33ee6e7667bf1d34bf0803b7c5fdbdce (patch) | |
tree | 7ff41667fe1f0d2ab0af572b4e6b28b4000ee644 | |
parent | b87bac1012c483462e7776c7b7320b659dbb3295 (diff) |
netdevsim: don't leak devlink resources
Devlink resources registered with devlink_resource_register() have
to be unregistered.
Fixes: 37923ed6b8ce ("netdevsim: Add simple FIB resource controller via devlink")
Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: Quentin Monnet <quentin.monnet@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/netdevsim/devlink.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/netdevsim/devlink.c b/drivers/net/netdevsim/devlink.c index ba663e5af168..5135fc371f01 100644 --- a/drivers/net/netdevsim/devlink.c +++ b/drivers/net/netdevsim/devlink.c | |||
@@ -207,6 +207,7 @@ void nsim_devlink_teardown(struct netdevsim *ns) | |||
207 | struct net *net = nsim_to_net(ns); | 207 | struct net *net = nsim_to_net(ns); |
208 | bool *reg_devlink = net_generic(net, nsim_devlink_id); | 208 | bool *reg_devlink = net_generic(net, nsim_devlink_id); |
209 | 209 | ||
210 | devlink_resources_unregister(ns->devlink, NULL); | ||
210 | devlink_unregister(ns->devlink); | 211 | devlink_unregister(ns->devlink); |
211 | devlink_free(ns->devlink); | 212 | devlink_free(ns->devlink); |
212 | ns->devlink = NULL; | 213 | ns->devlink = NULL; |