Re: [PATCH] ext4: init error handle resource before init group descriptors

From: Theodore Ts'o
Date: Sun Feb 19 2023 - 00:41:51 EST


On Thu, 19 Jan 2023 09:37:11 +0800, Ye Bin wrote:
> From: Ye Bin <yebin10@xxxxxxxxxx>
>
> Now, 's_err_report' timer is init after ext4_group_desc_init() when fill
> super. Theoretically, ext4_group_desc_init() may access to error handle
> as follows:
> __ext4_fill_super
> ext4_group_desc_init
> ext4_check_descriptors
> ext4_get_group_desc
> ext4_error
> ext4_handle_error
> ext4_commit_super
> ext4_update_super
> if (!es->s_error_count)
> mod_timer(&sbi->s_err_report, jiffies + 24*60*60*HZ);
> --> Accessing Uninitialized Variables
> timer_setup(&sbi->s_err_report, print_daily_error_info, 0);
>
> [...]

Applied, thanks!

[1/1] ext4: init error handle resource before init group descriptors
commit: 2806abcc97829d49172632db81deac9119641b78

Best regards,
--
Theodore Ts'o <tytso@xxxxxxx>