Release of 3.0.2 is imminent

Alan DeKok aland at deployingradius.com
Tue Feb 18 18:14:34 CET 2014


Matthew Newton wrote:
> Basic tests seem OK - builds as a debian package on wheezy/amd64,
> and works. Haven't exercised much else though. There's a warning
> about potentially unused variable compare in
> src/lib/valuepair.c:2651, but it can't happen looking at the code.
> Maybe needs a compare=compare to silence it?

  Weird.  Arran saw that, too.  Different compilers produce different
warning messages.  And from what I can tell, that warning message isn't
true.  Coverity doesn't complain, and it does more in-depth checks than gcc.

> Seems reasonable - agree that User-Password should continue to
> show. Need enough to show that people have the wrong shared secret
> (corrupt user-password) or the user-password is wrong, which I
> think this will do. I don't think that it's a good thing to start
> mangling or blanking out attributes in debug output.

  Exactly.

>>   If anyone objects, please let me know.
> 
> Looks OK as is now. Just need to stop people posting -Xx by
> default, which a reasonable number still seem to do :(

  Yes.  It's hard to get people to follow the documentation.

  Alan DeKok.



More information about the Freeradius-Devel mailing list