[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freeradius-devel
Subject:    Re: possible bug on HUP (2.1.6)
From:       Alan DeKok <aland () deployingradius ! com>
Date:       2009-07-15 8:58:10
Message-ID: 4A5D9A22.50203 () deployingradius ! com
[Download RAW message or body]

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.
-
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/devel.html
[prev in list] [next in list] [prev in thread] [next in thread]