Commit report for v3.0.x branch

The git bot announce at freeradius.org
Tue Apr 15 00:00:26 CEST 2014 

New activity for FreeRADIUS (the high performance and highly configurable RADIUS server)

======
Revert "Forbid self to self maps"

This reverts commit 9b4b44e8e071c0cbaad25de8156c175ee7aa674b.

Apparently it's needed for the cache module.  We'll have a
better fix later

Alan T. DeKok at 2014-04-14T22:01:48Z
Files modified:
	* src/main/map.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/be9a55d4acde263e0a06a5165cf2144bfa791d4d
====== 
Update logfile API.

We need to close the FP in rlm_detail.  Which means that the
locks on the FD will be closed.  And we don't want to close
the main FD.  So we add a new API to unlock the mutex without
closing the FD.  And we always return a DUP'd FD to the caller,
instead of the main one.

Fixes a whack of Coverity complaints

Alan T. DeKok at 2014-04-14T21:18:05Z
Files modified:
	* src/include/log.h
	* src/main/log.c
	* src/modules/rlm_detail/rlm_detail.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/bee57ca5fca2982d1637d50eeb68be10dc05359d
====== 
Minor fixes to make Coverity happy.

The logfile API is a bit weird with respect to mutexes.  It's
intended reply with mutexes held

Alan T. DeKok at 2014-04-14T21:18:05Z
Files modified:
	* src/main/log.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/cc426a2d4f26edda38e9f5f555a52b5f78d71b85
====== 
Forbid self to self maps

Alan T. DeKok at 2014-04-14T21:18:05Z
Files modified:
	* src/main/map.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/9b4b44e8e071c0cbaad25de8156c175ee7aa674b
====== 
Remove hex2bin, the xlat code would require significant reworking for this to be safe

Arran Cudbard-Bell at 2014-04-14T20:14:12Z
Files modified:
	* src/modules/rlm_expr/rlm_expr.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/1c41ed8c29f652adc76342286bb15ca4b9c221bf
====== 
More sanity checks and messages for list assignment.

list := `...` is now a warning.  This is because the assignment
makes it look like it over-writes the list.  It doesn't.  We now
require

list = `...`  which is clearer that the operators used are the
ones which are returned from the external program.

Alan T. DeKok at 2014-04-14T19:34:15Z
Files modified:
	* src/main/map.c
	* src/main/valuepair.c
	* src/tests/keywords/update-exec

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a5343ae6452dc59eda08cd3a9be4d1e9f29f55a5
====== 
Distinguish failure cases

Alan T. DeKok at 2014-04-14T19:34:14Z
Files modified:
	* src/tests/keywords/update-exec

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/d763b616d267b85d60d64e17d9d487567d83c6ce
====== 
Set the stack to zero to simplify gdb output

Alan T. DeKok at 2014-04-14T19:34:14Z
Files modified:
	* src/main/modcall.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/528d8afb55e9641fe34b34ccfd011aca7119acab
====== 
Verify VPs we're printing

Alan T. DeKok at 2014-04-14T19:34:14Z
Files modified:
	* src/lib/print.c
	* src/main/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/ef52352ef45bf5de0a4fa0fdf2b59d281fe34dc2
====== 
Correctly set fields when compiling literal string

when we have an ATTR on the LHS, and a LITERAL on the RHS,
we need to set the RHS to be VPT_TYPE_DATA, not the LHS
and there are other fields we need to set, too

Alan T. DeKok at 2014-04-14T19:34:14Z
Files modified:
	* src/main/modcall.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/7b7eebb296fd7930173203134591edc3b76556df
====== 
Clean up syntax for list to list assignments

list !* ANY	remove all attributes in the list
list := list2	delete list, copy list2, and assign the copy to list
list += list2	copy list2, and append the copy to list

list := `foo`   read attrs from program, and assign to list
list += `foo`	read attrs from program, and append to list

Alan T. DeKok at 2014-04-14T19:34:14Z
Files modified:
	* src/main/map.c
	* src/tests/keywords/update-list-error

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/ab81dc3e1503452709ae0cf629ad6a245d9eae5d
====== 
Don't core if vpt->vpd is NULL

Alan T. DeKok at 2014-04-14T19:34:14Z
Files modified:
	* src/main/map.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a0ae300fd1fd496622a877e1310f37ed4c826d79
====== 
In radius_map2vp(), always expand XLAT structures.

Looking for a % in the map name is pointless.  If the vpt is
marked as xlat, it's xlat.  The parser should take care of
ensuring that strings without % are created as literals

Alan T. DeKok at 2014-04-14T19:34:14Z
Files modified:
	* src/main/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/63ef25be89c85e866409db7c9e3941d32104574e
====== 
re-add xlat_spaces for debugging

Alan T. DeKok at 2014-04-14T19:34:14Z
Files modified:
	* src/main/xlat.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b48acea6a57422179576fc808c97a87d904af3f8
====== 
Make radius_map2vp() call radius_vpt_get_vp() for map->src

So that we can have array references on the RHS of update
statements. And tags on the RHS, too

Alan T. DeKok at 2014-04-14T19:34:14Z
Files modified:
	* src/main/valuepair.c
	* src/tests/keywords/update-array

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/d42b948502db966d1f986957cd03599c3cfc9fbc
====== 
LF at EOL

Alan T. DeKok at 2014-04-14T19:34:14Z
Files modified:
	* src/tests/keywords/array

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/187b901bb08413cedf532ceb92449cc24a3d26d2
====== 
Move debug print map functionality to its own function

And as a side effect, always properly reparent the VP...

Alan T. DeKok at 2014-04-14T19:34:14Z
Files modified:
	* src/main/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/425bf7ba9e71ed65d0eaf97a0a94c11f79c177c1
====== 
Use radius_list_ctx() rather than re-implementing it

Alan T. DeKok at 2014-04-14T19:34:14Z
Files modified:
	* src/main/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/467fd04000d989a2700c150f07424b2eb875f9a2
====== 
allow radius_vpt_get_vp() to use array references

Alan T. DeKok at 2014-04-14T19:34:13Z
Files modified:
	* src/main/valuepair.c
	* src/tests/keywords/array

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a75c02dc9f5f3d5904658dd6262c2287f7885875
====== 
Add tags and array references to value_pair_tmpl_t

radius_parse_attr() parses them
radius_tmpl2str() prints them
condition.txt tests them
radius_vpt_get_vp() uses tag now, but not array ref

No other code has been updated. e.g. evaluate, etc.

Alan T. DeKok at 2014-04-14T19:34:13Z
Files modified:
	* src/include/map.h
	* src/main/map.c
	* src/main/valuepair.c
	* src/tests/unit/condition.txt

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/17bdd5bf9528ee99ca724e4a715cc8a4ca2b96d3
====== 
Add function to look up "name:0" or "name[1]" or "name:0[1]"

Because it's easier and faster to do it here, than to force
all of the callers to manually massage the dictionary names

Alan T. DeKok at 2014-04-14T19:34:13Z
Files modified:
	* src/include/libradius.h
	* src/lib/dict.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/0c7488fe157b1e6b497b94919802ba2a009fc8e7
====== 
radius_parse_attr() doesn't deal with tags or array references

Alan T. DeKok at 2014-04-14T19:34:13Z
Files modified:
	* src/main/map.c
	* src/tests/unit/condition.txt

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/71dd09ca1898ccafbedc2892f49478a257f4facd
====== 
Make str2tmpl call radius_parse_attr()

So we don't have 2 pieces of code doing the same thing.

Alan T. DeKok at 2014-04-14T19:34:13Z
Files modified:
	* src/main/map.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f0474647a3616e62a7e424b670b075a57f59335c
====== 
Small update to docs/concepts/proxy.rst

Some file locations are updated in more recent versions of FreeRADIUS.

Herwin Weststrate (via Arran Cudbard-Bell)@2014-04-14T16:10:42Z
Files modified:
	* doc/concepts/proxy.rst

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/78a89b4bfe33ab8ef11e81c67bf29ec23af46ece
====== 
-- 
This commit summary was generated @2014-04-15T00:00:26Z by lgfeed version 0.00 (https://github.com/arr2036/lgfeed).

More information about the Freeradius-Devel mailing list