radiusd.c (Is it a bug??)

[Nicolas Baradakis]

Leo Lai wrote:

> when reloading config file, there's a section of code that is supposed 
> to wait for all threads to stop, however the condition is like this:
> if (!spawn_flag) for(;;) {
>   // wait for thread here...
> }
> when I check where spawn_flag is set, I found that it's TRUE when 
> multithread is supported, and FALSE when there is no thread.  In this 
> case, shouldn't here be "if (spawn_flag)" instead??

Yes, it has already been reported under bug #306 of the bugzilla.
http://bugs.freeradius.org/show_bug.cgi?id=306

It will be fixed in 1.1.0.

-- 
Nicolas Baradakis