On Tue 22-09-20 09:24:56, Eric Biggers wrote: > From: Eric Biggers <ebiggers@xxxxxxxxxx> > > ext4_unregister_sysfs() only deletes the kobject. The reference to it > needs to be put separately, like ext4_put_super() does. > > This addresses the syzbot report > "memory leak in kobject_set_name_vargs (3)" > (https://syzkaller.appspot.com/bug?extid=9f864abad79fae7c17e1). > > Reported-by: syzbot+9f864abad79fae7c17e1@xxxxxxxxxxxxxxxxxxxxxxxxx > Fixes: 72ba74508b28 ("ext4: release sysfs kobject when failing to enable quotas on mount") > Cc: stable@xxxxxxxxxxxxxxx > Signed-off-by: Eric Biggers <ebiggers@xxxxxxxxxx> Looks good. You can add: Reviewed-by: Jan Kara <jack@xxxxxxx> Honza > --- > fs/ext4/super.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/fs/ext4/super.c b/fs/ext4/super.c > index ea425b49b345..41953b86ffe3 100644 > --- a/fs/ext4/super.c > +++ b/fs/ext4/super.c > @@ -4872,6 +4872,7 @@ static int ext4_fill_super(struct super_block *sb, void *data, int silent) > > failed_mount8: > ext4_unregister_sysfs(sb); > + kobject_put(&sbi->s_kobj); > failed_mount7: > ext4_unregister_li_request(sb); > failed_mount6: > > base-commit: ba4f184e126b751d1bffad5897f263108befc780 > -- > 2.28.0 > -- Jan Kara <jack@xxxxxxxx> SUSE Labs, CR
