On Thu, 2025-03-06 at 14:50 +0530, Maninder Singh wrote:
> With rpc_status netlink support, unregister of register_filesystem()
> was missed in case of genl_register_family() fails.
>
> Correcting it by making new label.
>
> Fixes: bd9d6a3efa97 ("NFSD: add rpc_status netlink support")
> Signed-off-by: Maninder Singh <maninder1.s@xxxxxxxxxxx>
> ---
> fs/nfsd/nfsctl.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/fs/nfsd/nfsctl.c b/fs/nfsd/nfsctl.c
> index ac265d6fde35..d773481bcf10 100644
> --- a/fs/nfsd/nfsctl.c
> +++ b/fs/nfsd/nfsctl.c
> @@ -2305,7 +2305,7 @@ static int __init init_nfsd(void)
> goto out_free_cld;
> retval = register_filesystem(&nfsd_fs_type);
> if (retval)
> - goto out_free_all;
> + goto out_free_nfsd4;
> retval = genl_register_family(&nfsd_nl_family);
> if (retval)
> goto out_free_all;
> @@ -2313,6 +2313,8 @@ static int __init init_nfsd(void)
>
> return 0;
> out_free_all:
> + unregister_filesystem(&nfsd_fs_type);
> +out_free_nfsd4:
> nfsd4_destroy_laundry_wq();
> out_free_cld:
> unregister_cld_notifier();
Reviewed-by: Jeff Layton <jlayton@xxxxxxxxxx>
