Michael Chernyakhovsky <[EMAIL PROTECTED]> wrote:
>   So, in line 1176 we have "switch( tok )". tok==RAD_FILTER_IP==1
>   But in 1214 tok == 1 == FILTER_GENERIC_TYPE   ;-)

  Ah, that's the problem, thanks.
  
>   after this "if( radFil.u.generic.more )" evaluated whith unknown condition
> - radFil.u.generic.more have something from radFil.u.generic.
>   In some cases "prevRadPair = pair;" executed.
> 
> I think patch must look like follow:
> ===============================================
> --- lib.orig/filters.c       Thu Feb  8 06:04:24 2001
> +++ lib/filters.c     Wed Aug 29 01:55:02 2001
> @@ -269,8 +269,8 @@
>  };
> 
>  typedef enum {
> -    FILTER_IP_TYPE,
>      FILTER_GENERIC_TYPE,
> +    FILTER_IP_TYPE,

  OK, I've applied that patch.

  Hmm.. the business with RAD_FILTER_GENERIC being the same as
FILTER_GENERIC_TYPE, but with a different definition, is probably
wrong.

  i.e. we probably want:

#define RAD_FILTER_GENERIC FILTER_GENERIC_TYPE

  or, just remove the defs for RAD_FILTER_GENERIC

  Alan DeKok.

- 
List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html

Reply via email to