diff options
Diffstat (limited to 'net/core/net_namespace.c')
-rw-r--r-- | net/core/net_namespace.c | 28 |
1 files changed, 12 insertions, 16 deletions
diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c index 5221f975a4cc..70d3450588b2 100644 --- a/net/core/net_namespace.c +++ b/net/core/net_namespace.c | |||
@@ -198,8 +198,10 @@ static int __peernet2id(struct net *net, struct net *peer, bool alloc) | |||
198 | */ | 198 | */ |
199 | int peernet2id(struct net *net, struct net *peer) | 199 | int peernet2id(struct net *net, struct net *peer) |
200 | { | 200 | { |
201 | int id = __peernet2id(net, peer, true); | 201 | bool alloc = atomic_read(&peer->count) == 0 ? false : true; |
202 | int id; | ||
202 | 203 | ||
204 | id = __peernet2id(net, peer, alloc); | ||
203 | return id >= 0 ? id : NETNSA_NSID_NOT_ASSIGNED; | 205 | return id >= 0 ? id : NETNSA_NSID_NOT_ASSIGNED; |
204 | } | 206 | } |
205 | EXPORT_SYMBOL(peernet2id); | 207 | EXPORT_SYMBOL(peernet2id); |
@@ -349,7 +351,7 @@ static LIST_HEAD(cleanup_list); /* Must hold cleanup_list_lock to touch */ | |||
349 | static void cleanup_net(struct work_struct *work) | 351 | static void cleanup_net(struct work_struct *work) |
350 | { | 352 | { |
351 | const struct pernet_operations *ops; | 353 | const struct pernet_operations *ops; |
352 | struct net *net, *tmp, *peer; | 354 | struct net *net, *tmp; |
353 | struct list_head net_kill_list; | 355 | struct list_head net_kill_list; |
354 | LIST_HEAD(net_exit_list); | 356 | LIST_HEAD(net_exit_list); |
355 | 357 | ||
@@ -365,6 +367,14 @@ static void cleanup_net(struct work_struct *work) | |||
365 | list_for_each_entry(net, &net_kill_list, cleanup_list) { | 367 | list_for_each_entry(net, &net_kill_list, cleanup_list) { |
366 | list_del_rcu(&net->list); | 368 | list_del_rcu(&net->list); |
367 | list_add_tail(&net->exit_list, &net_exit_list); | 369 | list_add_tail(&net->exit_list, &net_exit_list); |
370 | for_each_net(tmp) { | ||
371 | int id = __peernet2id(tmp, net, false); | ||
372 | |||
373 | if (id >= 0) | ||
374 | idr_remove(&tmp->netns_ids, id); | ||
375 | } | ||
376 | idr_destroy(&net->netns_ids); | ||
377 | |||
368 | } | 378 | } |
369 | rtnl_unlock(); | 379 | rtnl_unlock(); |
370 | 380 | ||
@@ -390,26 +400,12 @@ static void cleanup_net(struct work_struct *work) | |||
390 | */ | 400 | */ |
391 | rcu_barrier(); | 401 | rcu_barrier(); |
392 | 402 | ||
393 | rtnl_lock(); | ||
394 | /* Finally it is safe to free my network namespace structure */ | 403 | /* Finally it is safe to free my network namespace structure */ |
395 | list_for_each_entry_safe(net, tmp, &net_exit_list, exit_list) { | 404 | list_for_each_entry_safe(net, tmp, &net_exit_list, exit_list) { |
396 | /* Unreference net from all peers (no need to loop over | ||
397 | * net_exit_list because idr_destroy() will be called for each | ||
398 | * element of this list. | ||
399 | */ | ||
400 | for_each_net(peer) { | ||
401 | int id = __peernet2id(peer, net, false); | ||
402 | |||
403 | if (id >= 0) | ||
404 | idr_remove(&peer->netns_ids, id); | ||
405 | } | ||
406 | idr_destroy(&net->netns_ids); | ||
407 | |||
408 | list_del_init(&net->exit_list); | 405 | list_del_init(&net->exit_list); |
409 | put_user_ns(net->user_ns); | 406 | put_user_ns(net->user_ns); |
410 | net_drop_ns(net); | 407 | net_drop_ns(net); |
411 | } | 408 | } |
412 | rtnl_unlock(); | ||
413 | } | 409 | } |
414 | static DECLARE_WORK(net_cleanup_work, cleanup_net); | 410 | static DECLARE_WORK(net_cleanup_work, cleanup_net); |
415 | 411 | ||