lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20240102142230.hwmjkmbrhproilkp@quack3> Date: Tue, 2 Jan 2024 15:22:30 +0100 From: Jan Kara <jack@...e.cz> To: Haibo Liu <haiboliu6@...il.com> Cc: djwong@...nel.org, linux-ext4@...r.kernel.org, Ted Tso <tytso@....edu> Subject: Re: [v2] ext4/super.c : Fix a goto label This seems to have fallen through the cracks. CCing ext4 maintainer (which is a good idea with any patch BTW)... On Tue 29-08-23 15:52:22, Haibo Liu wrote: > Thank you for Darrick J. Wong's suggestions :). > I wrote a new patch and renamed these 9 labels. > > Original labels -> New labels: > > out -> out_unregister_ext23_and_dentry_cache > out05 -> out_inodecache > out1 -> out_mballoc > out2 -> out_sysfs > out3 -> out_system_zone > out4 -> out_pageio > out5 -> out_post_read_processing > out6 -> out_pending > out7 -> out_es I agree this is more standard and probably more future proof. > v1->v2: > Followed Darrick J. Wong's suggestions, renamed these 9 goto labels. This versioning belongs below the --- line (probably below diffstat). > Signed-off-by: Haibo Liu <haiboliu6@...il.com> Otherwise looks good so feel free to add: Reviewed-by: Jan Kara <jack@...e.cz> Honza > --- > fs/ext4/super.c | 36 ++++++++++++++++++------------------ > 1 file changed, 18 insertions(+), 18 deletions(-) > > diff --git a/fs/ext4/super.c b/fs/ext4/super.c > index 91f20afa1d71..11cffb5a05a4 100644 > --- a/fs/ext4/super.c > +++ b/fs/ext4/super.c > @@ -7347,61 +7347,61 @@ static int __init ext4_init_fs(void) > > err = ext4_init_pending(); > if (err) > - goto out7; > + goto out_es; > > err = ext4_init_post_read_processing(); > if (err) > - goto out6; > + goto out_pending; > > err = ext4_init_pageio(); > if (err) > - goto out5; > + goto out_post_read_processing; > > err = ext4_init_system_zone(); > if (err) > - goto out4; > + goto out_pageio; > > err = ext4_init_sysfs(); > if (err) > - goto out3; > + goto out_system_zone; > > err = ext4_init_mballoc(); > if (err) > - goto out2; > + goto out_sysfs; > err = init_inodecache(); > if (err) > - goto out1; > + goto out_mballoc; > > err = ext4_fc_init_dentry_cache(); > if (err) > - goto out05; > + goto out_inodecache; > > register_as_ext3(); > register_as_ext2(); > err = register_filesystem(&ext4_fs_type); > if (err) > - goto out; > + goto out_unregister_ext23_and_dentry_cache; > > return 0; > -out: > +out_unregister_ext23_and_dentry_cache: > unregister_as_ext2(); > unregister_as_ext3(); > ext4_fc_destroy_dentry_cache(); > -out05: > +out_inodecache: > destroy_inodecache(); > -out1: > +out_mballoc: > ext4_exit_mballoc(); > -out2: > +out_sysfs: > ext4_exit_sysfs(); > -out3: > +out_system_zone: > ext4_exit_system_zone(); > -out4: > +out_pageio: > ext4_exit_pageio(); > -out5: > +out_post_read_processing: > ext4_exit_post_read_processing(); > -out6: > +out_pending: > ext4_exit_pending(); > -out7: > +out_es: > ext4_exit_es(); > > return err; > -- > 2.34.1 > -- Jan Kara <jack@...e.com> SUSE Labs, CR
Powered by blists - more mailing lists