Re: [PATCH v5] vfs, security: Fix automount superblock LSM init problem, preventing NFS sb sharing

From: Jeff Layton
Date: Tue Sep 06 2022 - 12:31:48 EST


On Tue, 2022-09-06 at 17:01 +0100, David Howells wrote:
> Jeff Layton <jlayton@xxxxxxxxxx> wrote:
>
> > If this or the other allocations below fail, do you need to free the
> > prior ones here? Or do they automagically get cleaned up somehow?
>
> Once the fs_context is allocated, it will always get cleaned up with
> put_fs_context(), which will dispose of the partially constructed
> smack_mnt_opts struct.
>


Ok! In that case, you can add:

Reviewed-by: Jeff Layton <jlayton@xxxxxxxxxx>