Commit report for v3.0.x branch
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@2014-04-14T22:01:48Z Files modified: * src/main/map.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/be9a55d4acde263e0a06a... ====== 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@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/bee57ca5fca2982d1637d... ====== 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@2014-04-14T21:18:05Z Files modified: * src/main/log.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/cc426a2d4f26edda38e9f... ====== Forbid self to self maps Alan T. DeKok@2014-04-14T21:18:05Z Files modified: * src/main/map.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/9b4b44e8e071c0cbaad25... ====== Remove hex2bin, the xlat code would require significant reworking for this to be safe Arran Cudbard-Bell@2014-04-14T20:14:12Z Files modified: * src/modules/rlm_expr/rlm_expr.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/1c41ed8c29f652adc7634... ====== 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@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/a5343ae6452dc59eda08c... ====== Distinguish failure cases Alan T. DeKok@2014-04-14T19:34:14Z Files modified: * src/tests/keywords/update-exec Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/d763b616d267b85d60d64... ====== Set the stack to zero to simplify gdb output Alan T. DeKok@2014-04-14T19:34:14Z Files modified: * src/main/modcall.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/528d8afb55e9641fe34b3... ====== Verify VPs we're printing Alan T. DeKok@2014-04-14T19:34:14Z Files modified: * src/lib/print.c * src/main/valuepair.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/ef52352ef45bf5de0a4fa... ====== 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@2014-04-14T19:34:14Z Files modified: * src/main/modcall.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/7b7eebb296fd793017320... ====== 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@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/ab81dc3e1503452709ae0... ====== Don't core if vpt->vpd is NULL Alan T. DeKok@2014-04-14T19:34:14Z Files modified: * src/main/map.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/a0ae300fd1fd496622a87... ====== 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@2014-04-14T19:34:14Z Files modified: * src/main/valuepair.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/63ef25be89c85e866409d... ====== re-add xlat_spaces for debugging Alan T. DeKok@2014-04-14T19:34:14Z Files modified: * src/main/xlat.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/b48acea6a57422179576f... ====== 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@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/d42b948502db966d1f986... ====== LF at EOL Alan T. DeKok@2014-04-14T19:34:14Z Files modified: * src/tests/keywords/array Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/187b901bb08413cedf532... ====== Move debug print map functionality to its own function And as a side effect, always properly reparent the VP... Alan T. DeKok@2014-04-14T19:34:14Z Files modified: * src/main/valuepair.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/425bf7ba9e71ed65d0eaf... ====== Use radius_list_ctx() rather than re-implementing it Alan T. DeKok@2014-04-14T19:34:14Z Files modified: * src/main/valuepair.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/467fd04000d989a2700c1... ====== allow radius_vpt_get_vp() to use array references Alan T. DeKok@2014-04-14T19:34:13Z Files modified: * src/main/valuepair.c * src/tests/keywords/array Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/a75c02dc9f5f3d5904658... ====== 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@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/17bdd5bf9528ee99ca724... ====== 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@2014-04-14T19:34:13Z Files modified: * src/include/libradius.h * src/lib/dict.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/0c7488fe157b1e6b497b9... ====== radius_parse_attr() doesn't deal with tags or array references Alan T. DeKok@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/71dd09ca1898ccafbedc2... ====== Make str2tmpl call radius_parse_attr() So we don't have 2 pieces of code doing the same thing. Alan T. DeKok@2014-04-14T19:34:13Z Files modified: * src/main/map.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/f0474647a3616e62a7e42... ====== 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/78a89b4bfe33ab8ef11e8... ====== -- This commit summary was generated @2014-04-15T00:00:26Z by lgfeed version 0.00 (https://github.com/arr2036/lgfeed).
participants (1)
-
announceļ¼ freeradius.org