Commit report for master 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:02:16Z
Files modified:
	* src/main/map.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/ab52be31f3a839d1219bee130bdb323c824fb00c
====== 
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:47Z
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/2f42d46091b639b83bfb4119fb267a1e9c1c207b
====== 
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:46Z
Files modified:
	* src/main/log.c

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

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

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

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

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/51e07bd604ea066822eb4a214ce4891cc84ecca7
====== 
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:48: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/3c55558e5f895480583be9b16d2d4417995c00b1
====== 
Distinguish failure cases

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

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

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

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

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

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b87eb542794ea616044938f356fbb00525d9e218
====== 
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:47:56Z
Files modified:
	* src/main/modcall.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/519ac57acc8df327a90f3ea8804d41e1eb7d663d
====== 
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:47:45Z
Files modified:
	* src/main/map.c
	* src/tests/keywords/update-list-error

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

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

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f2e89e2ff965ad889a936370e30cf6663909cf8b
====== 
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:46:45Z
Files modified:
	* src/main/valuepair.c

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

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

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2bdf3ca702f7a5c7230dd45bc9550b5134359fb8
====== 
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:46:31Z
Files modified:
	* src/main/valuepair.c
	* src/tests/keywords/update-array

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

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

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/43fd032ced9d841b7f3186aea6c883a28f84101e
====== 
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:44:59Z
Files modified:
	* src/main/valuepair.c

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

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

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

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

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/5b85cda20598c4dd4695f52af176f4b9b8afb7f3
====== 
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:36:31Z
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/1df9b3bc8b6e883e23bc9e7ec8f381fd4781de9e
====== 
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:35:49Z
Files modified:
	* src/include/libradius.h
	* src/lib/dict.c

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

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

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/4061e1cc2d073537f9ebb4de1f7c6722f65e7c25
====== 
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:35:22Z
Files modified:
	* src/main/map.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/21e6f521cfefa05e2c3c4cb44366f2db16415587
====== 
Merge pull request #588 from qnet-herwin/proxydocs

Small update to docs/concepts/proxy.rst

Arran Cudbard-Bell at 2014-04-14T16:09:28Z
Files modified:
	* doc/concepts/proxy.rst

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

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

Herwin Weststrate at 2014-04-14T16:03:53Z
Files modified:
	* doc/concepts/proxy.rst

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b687c21ef9cd948c1b322bd688497d2d54464d39
====== 
-- 
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