Re: [PATCH] appletalk: Fix use-after-free in atalk_proc_exit

From: David Miller
Date: Wed Feb 27 2019 - 13:15:02 EST


From: Yue Haibing <yuehaibing@xxxxxxxxxx>
Date: Wed, 27 Feb 2019 22:59:00 +0800

> @@ -1925,7 +1925,10 @@ static int __init atalk_init(void)
>
> register_netdevice_notifier(&ddp_notifier);
> aarp_proto_init();
> - atalk_proc_init();
> + rc = atalk_proc_init();
> + if (rc)
> + goto out;
> +
> atalk_register_sysctl();
> out:

This leaves the netdevice notifier registered etc. You need to add a proper
error cleanup code path.