[Patch] Pre-Acct-Type dict value registration
Lv Zheng
lv.zheng at soliton.com.cn
Fri Jun 8 17:54:38 CEST 2007
Nicolas Baradakis wrote:
> Lv Zheng wrote:
>
>> I noticed that there might be codes missing in modules.c for Pre-Acct-Type.
>> This will crash the radiusd server if thre is pre-acct sub-list in
>> radiusd.conf.
>
> I don't see why you need such a patch. It's possible to set "Acct-Type"
> during pre-accounting because it runs before accounting. But it's no
> use to set "Pre-Acct-Type" because it is the first section to be
> processed. That's why if you set "Pre-Acct-Type" during pre-accounting,
> it's already too late.
>
> I think there is some confusion because the section "authorize" works
> differently. The server runs the section twice if "Auth-Type" has
> been set.
>
Thank you for pointing out this. The code is useless.
The requirement is coming from our QCs, when they were to create
a Pre-Acct-Type sub-list through some configuration APIs (based on
libradius & libradiusd), the libradiusd crashed. So we need this for the
modified libradiusd (according to your statement, there should be a
better way to avoid this).
Thx & rgds / Lv Zheng
More information about the Freeradius-Devel
mailing list