[master] a value of type uint64 in configuration cannot be specified as hex

Alan DeKok aland at deployingradius.com
Mon Aug 12 15:41:20 CEST 2019


On Aug 12, 2019, at 9:38 AM, Chaigneau, Nicolas via Freeradius-Devel <freeradius-devel at 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.




More information about the Freeradius-Devel mailing list