Re: [PATCH] binderfs: rework superblock destruction

From: Al Viro
Date: Wed Aug 17 2022 - 09:59:54 EST


On Wed, Aug 17, 2022 at 03:03:06PM +0200, Christian Brauner wrote:

> +static void binderfs_kill_super(struct super_block *sb)
> +{
> + struct binderfs_info *info = sb->s_fs_info;
> +
> + if (info && info->ipc_ns)
> + put_ipc_ns(info->ipc_ns);
> +
> + kfree(info);
> + kill_litter_super(sb);
> +}

Other way round, please - shut the superblock down, *then*
free the objects it'd been using. IOW,

struct binderfs_info *info = sb->s_fs_info;

kill_litter_super(sb);

if (info && info->ipc_ns)
put_ipc_ns(info->ipc_ns);

kfree(info);