inet: frags: call inet_frags_fini() after unregister_pernet_subsys()
[ Upstream commit ae7352d384a552d8c799c242e74a934809990a71 ] Both IPv6 and 6lowpan are calling inet_frags_fini() too soon. inet_frags_fini() is dismantling a kmem_cache, that might be needed later when unregister_pernet_subsys() eventually has to remove frags queues from hash tables and free them. This fixes potential use-after-free, and is a prereq for the following patch. Fixes: d4ad4d22e7ac ("inet: frags: use kmem_cache for inet_frag_queue") Signed-off-by: Eric Dumazet <edumazet@google.com> Signed-off-by: David S. Miller <davem@davemloft.net> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
8729512985
commit
6c19fd7108
|
@ -634,7 +634,7 @@ err_sysctl:
|
|||
|
||||
void lowpan_net_frag_exit(void)
|
||||
{
|
||||
inet_frags_fini(&lowpan_frags);
|
||||
lowpan_frags_sysctl_unregister();
|
||||
unregister_pernet_subsys(&lowpan_frags_ops);
|
||||
inet_frags_fini(&lowpan_frags);
|
||||
}
|
||||
|
|
|
@ -770,8 +770,8 @@ err_protocol:
|
|||
|
||||
void ipv6_frag_exit(void)
|
||||
{
|
||||
inet_frags_fini(&ip6_frags);
|
||||
ip6_frags_sysctl_unregister();
|
||||
unregister_pernet_subsys(&ip6_frags_ops);
|
||||
inet6_del_protocol(&frag_protocol, IPPROTO_FRAGMENT);
|
||||
inet_frags_fini(&ip6_frags);
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue
Block a user