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