Messages in this thread | | | Date | Mon, 22 Apr 2024 18:25:26 +0800 | Subject | Re: [PATCH -next v3 1/2] erofs: get rid of erofs_fs_context | From | Jingbo Xu <> |
| |
On 4/19/24 8:36 PM, Baokun Li wrote:
> @@ -761,12 +747,15 @@ static void erofs_free_dev_context(struct erofs_dev_context *devs) > > static void erofs_fc_free(struct fs_context *fc) > { > - struct erofs_fs_context *ctx = fc->fs_private; > + struct erofs_sb_info *sbi = fc->s_fs_info; > + > + if (!sbi) > + return;
This is the only difference comparing to the original code literally. Is there any chance that fc->s_fs_info can be NULL when erofs_fc_free() is called?
Otherwise looks good to me.
-- Thanks, Jingbo
| |