possible bug on HUP (2.1.6)

Alan DeKok aland at deployingradius.com
Wed Jul 15 10:58:10 CEST 2009


Stefan Winter wrote:
> Hi again,
> 
> ah, the source seems to reveal where the problem is:
> 
>                 if ((in[0] < 32) ||
>                     strchr(allowed_chars, *in) == NULL) {
> 
> Since "in" is correct, it looks like allowed_chars is invalid - maybe
> the HUP makes the server forget about that variable?

  Ah, yes.  The "allowed_chars" array is being marked NULL, because it's
not checking for multiple instances of the module.

  I'll commit a fix for 2.1.7.

  Alan DeKok.



More information about the Freeradius-Devel mailing list