Automatic report from sources (radiusd) between 10.05.2006 - 11.05.2006 GMT

Alan DeKok aland at nitros9.org
Thu May 11 21:14:49 CEST 2006


Frank Cusack <fcusack at fcusack.com> wrote:
> I removed the const because the compiler complains about Sentinel being
> self-referential (.Left is not const but you are assigning it a const).

  Yes.

> How would you write to the sentinel if it's const?  Or do you mean that
> threads would write to it when they shouldn't.

  You don't write to the sentinel.  It's not thread-specific, and
having multiple threads writing to it is bad.

  In anycase, I updated the code so that everywhere it checks if 
"FOO != NIL" before writing to FOO->Color.  So the change you made
shouldn't hurt anything.

  Alan DeKok.




More information about the Freeradius-Devel mailing list