12 Aug
2019
12 Aug
'19
9:41 a.m.
On Aug 12, 2019, at 9:38 AM, Chaigneau, Nicolas via Freeradius-Devel <freeradius-devel@lists.freeradius.org> wrote:
Cf. file cf_parse.c (at line 286 of current HEAD):
case FR_TYPE_UINT64: *(uint64_t *)out = strtoull(cp->value, NULL, 10);
Function strtoull is called with "base" = 10, which means we cannot have an input value "0x..." (it is converted to 0).
This is not consistent with other types such as uint32, where this is allowed ("base" = 0). I think this might not be the intended behaviour. Could you please check ?
I think it's fine to allow "0x...". I'll push a fix. Alan DeKok.