[PATCH] erofs-utils: lib: fix `1UL << vi->u.chunkbits` on 32-bit platforms

[Gao Xiang]

Hi Colin,

On 2025/4/10 00:51, Colin Walters wrote:
> Patch looks sane to me.

I've pushed it out into -dev branch.

> 
> On Wed, Apr 9, 2025, at 2:17 AM, Gao Xiang wrote:
> 
>> I think it should be fixed on the kernel side too, yet I rarely look
>> after 32-bit platforms due to lack of test environments.
> 
> It is relatively easy to run 32 bit containers on a 64bit host, that’s what the Debian CI environment that hit this is doing.

I know it's easy to test for erofs-utils, but it's somewhat complex
(althrough doable, need 32-bit rootfs too) to test 32-bit kernels.

> 
> I think the bigger question here is fuzzing on 32 bit right? That likely would have caught this quickly.
> 
> I don’t know…roughly though it feels to me as long as the Linux kernel supports 32 bit we are going to keep getting pulled to do so too. Especially there’s a long tail of 32 bit ARM out there as I understand it.

I will find more time working on this, but sigh..

Thanks,
Gao Xiang

> 
>>