Re: [PATCH 1/1] ext4: fix incorrect function call when initializing s_resgid

[Liu Jiazi]

Thanks for the review.

This was actually caught in our product. We use gid as the reserve
size id, and we hit this issue there.

Brs
Jiazi

Ritesh Harjani <ritesh.list@xxxxxxxxx> 于2026年7月28日周二 12:43写道：
>
> Jiazi Liu <jiazi.liu1984@xxxxxxxxx> writes:
>
> > In __ext4_fill_super(), s_resgid is initialized by calling
> > ext4_get_resuid() instead of ext4_get_resgid(), resulting in the
> > reserved GID being set to the same value as the reserved UID rather
> > than the value stored in the superblock.
> >
> > Fixes: 12c84dd4d308 ("ext4: add support for 32-bit default reserved uid and gid values")
> > Cc: stable@xxxxxxxxxxxxxxx
> > Signed-off-by: Jiazi Liu <liujiazi@xxxxxxxxxx>
>
> yup, that clearly looks like a simple miss.
>
> Nice catch! Wondering whether this was caught by code review or did you
> hit an issue due to this?
>
> Looks good to me. Feel free to add:
> Reviewed-by: Ritesh Harjani (IBM) <ritesh.list@xxxxxxxxx>
>