[PATCH] afs: Replace simple_strtoul with kstrtoul in afs_parse_address
Su Hui
suhui at nfschina.com
Fri May 30 03:42:40 PDT 2025
On 5/30/25 5:32 PM, David Howells wrote:
> Su Hui <suhui at nfschina.com> wrote:
>
>> kstrtoul() is better because simple_strtoul() ignores overflow which
>> may lead to unexpected results.
> Overflow in what sense? Are we talking about a mathematical overflow or not
> checking the text beyond the end of the number?
IMO, It's meaning that the number represented by the string exceeds the
type range. Like this code:
const char str[] = "0xffffffffffffffff0000000000000001";
unsigned long res;
res = simple_strtoul(str, &p, 0); //overflow happends and res = 0x1
err = kstrtoul(str, 0, &res); // overflow happends and res = 0x1, err =
-ERANGE
Su Hui
More information about the linux-afs
mailing list