[Toybox] [PATCH] devmem: Fix 8 byte wide writes

Rob Landley rob at landley.net
Mon Oct 9 14:49:49 PDT 2023


On 10/9/23 10:26, enh wrote:
>> Let me know how commit 5f153b56214f works? I don't have a test environment
>> for this, but it seems to produce the errors right on 32 and 64 bit at
>> least...
>
> it's a bit early in the morning for me, but doesn't the change from
> `unsigned long long` to `unsigned long` mean that by the time you test
> the result it can't be too large because you've already truncated?

The theory is that by switching strtoull() to strtoul() we get ERANGE when it's
truncated.

Rob


More information about the Toybox mailing list