[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