Patch for rlm_attr_filters

Chris Parker cparker at
Tue Nov 1 22:36:07 CET 2005

On Nov 1, 2005, at 12:15 PM, Alan DeKok wrote:

> Chris Parker <cparker at> wrote:
>> I'd make one change, and that's make it more obvious what the config
>> argument
>> is doing:
>>     defaultaction = drop | allow
>   Please see also attr_filter in the CVS head.  As part of my work in
> cleaning up the server internals, I'm removing a function used only by
> attr_filter & policy.  As part of that work, I re-wrote much of
> attr_filter.  It *should* do the same thing, but there's less
> duplicated code.

Yeah, I noted that.  Have looked at it, and it makes sense.  I'll run
it through some tests here, as with all the craziness we do with attrs,
I'm pretty sure any bugs would be poke their heads up pretty quickly.

Chris Parker
Director, Engineering
StarNet A Service of US LEC

(888)212-0099   Fax (847)963-1302
Wholesale Internet Services
VoiceEclipse, The Fresh Alternative

NOTICE: Message is sent IN CONFIDENCE to addressees. It may contain
information that is privileged, proprietary or confidential.

More information about the Freeradius-Devel mailing list