Commit cb7d224f authored by Scott Mayhew's avatar Scott Mayhew Committed by J. Bruce Fields

lockd: unregister notifier blocks if the service fails to come up completely

If the lockd service fails to start up then we need to be sure that the
notifier blocks are not registered, otherwise a subsequent start of the
service could cause the same notifier to be registered twice, leading to
soft lockups.
Signed-off-by: default avatarScott Mayhew <smayhew@redhat.com>
Cc: stable@vger.kernel.org
Fixes: 0751ddf7 "lockd: Register callbacks on the inetaddr_chain..."
Signed-off-by: default avatarJ. Bruce Fields <bfields@redhat.com>
parent 4c2e07c6
...@@ -335,12 +335,17 @@ static struct notifier_block lockd_inet6addr_notifier = { ...@@ -335,12 +335,17 @@ static struct notifier_block lockd_inet6addr_notifier = {
}; };
#endif #endif
static void lockd_svc_exit_thread(void) static void lockd_unregister_notifiers(void)
{ {
unregister_inetaddr_notifier(&lockd_inetaddr_notifier); unregister_inetaddr_notifier(&lockd_inetaddr_notifier);
#if IS_ENABLED(CONFIG_IPV6) #if IS_ENABLED(CONFIG_IPV6)
unregister_inet6addr_notifier(&lockd_inet6addr_notifier); unregister_inet6addr_notifier(&lockd_inet6addr_notifier);
#endif #endif
}
static void lockd_svc_exit_thread(void)
{
lockd_unregister_notifiers();
svc_exit_thread(nlmsvc_rqst); svc_exit_thread(nlmsvc_rqst);
} }
...@@ -462,7 +467,7 @@ int lockd_up(struct net *net) ...@@ -462,7 +467,7 @@ int lockd_up(struct net *net)
* Note: svc_serv structures have an initial use count of 1, * Note: svc_serv structures have an initial use count of 1,
* so we exit through here on both success and failure. * so we exit through here on both success and failure.
*/ */
err_net: err_put:
svc_destroy(serv); svc_destroy(serv);
err_create: err_create:
mutex_unlock(&nlmsvc_mutex); mutex_unlock(&nlmsvc_mutex);
...@@ -470,7 +475,9 @@ int lockd_up(struct net *net) ...@@ -470,7 +475,9 @@ int lockd_up(struct net *net)
err_start: err_start:
lockd_down_net(serv, net); lockd_down_net(serv, net);
goto err_net; err_net:
lockd_unregister_notifiers();
goto err_put;
} }
EXPORT_SYMBOL_GPL(lockd_up); EXPORT_SYMBOL_GPL(lockd_up);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment