Commit report for v3.0.x branch

The git bot announce at freeradius.org
Fri Oct 10 00:00:03 CEST 2014


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

======
cf_item_add defined too many times...

Arran Cudbard-Bell at 2014-10-09T21:01:11Z
Files modified:
	* src/include/conffile.h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f46e3f1e4b983d905de956144035bbcaced666c2
====== 
Support adding arbitrary attributes for dynamic clients

Arran Cudbard-Bell at 2014-10-09T20:18:29Z
Files modified:
	* src/main/client.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/e1a95205af48242450f4b06645c375dbc1506b7d
====== 
Expose cf_item_add and cf_pair_alloc

Arran Cudbard-Bell at 2014-10-09T16:53:43Z
Files modified:
	* src/include/conffile.h
	* src/main/conffile.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c47872be2ce4d4eab3f724c892a3c17ee3ec9bc8
====== 
Still need the loop

Arran Cudbard-Bell at 2014-10-09T16:51:04Z
Files modified:
	* src/lib/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6ef6fea4d19da9962dd9456b4469bd161d10958a
====== 
Adding the same VP twice is bad mm'k Fixes #809

Arran Cudbard-Bell at 2014-10-09T16:32:01Z
Files modified:
	* src/lib/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/ce82761ec3855fe7499c857e85422e3d5226b805
====== 
Typo

Arran Cudbard-Bell at 2014-10-09T16:27:09Z
Files modified:
	* src/lib/debug.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/78503e4bf60b9eb08a987fdf7f495a7149db5b0d
====== 
Note recent changes

Alan T. DeKok at 2014-10-09T16:16:36Z
Files modified:
	* doc/ChangeLog

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/947394e707eb1a7aeedafdb18dea8a0010575574
====== 
Merge pull request #809 from Feh/pairadd-ignore-dups

pairadd: Don't add a VP if it's already present

Alan DeKok at 2014-10-09T16:16:15Z
Files modified:
	* src/lib/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/21391a48cf57a69f43a64345d4734adb563da9f6
====== 
pairadd: Don't add a VP if it's already present

In the 3.0 code base, pairmake() will already do a pairadd()
automatically. If an old code base (e.g. modules originally written for
FreeRADIUS 2.x) uses pairmake() and then pairadd() explicitly, this will
add the VP *twice*, thus creating a cyclic list structure, e.g.:

    A->next == B;
    B->next == B;
    B->next == B;
    B->next == B;
    ...

This makes any function that walks all value pairs end up eating 100%
CPU.

Since VPs are added at the end of the list, we can simply stop
traversing the list and return silently in case the VP is already
present.

Signed-off-by: Julius Plenz <plenz at cis.fu-berlin.de>

Julius Plenz at 2014-10-09T12:48:37Z
Files modified:
	* src/lib/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b73a4c5c0c4c6c67029255e2e1b488f5f050f8e0
====== 
More verbose error messages about VALUE_PAIRs with bad dictionary entries

Arran Cudbard-Bell at 2014-10-09T13:58:41Z
Files modified:
	* src/lib/debug.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/589ef23ff8f8fdf5e67b7859b92347068c9eece8
====== 
-- 
This commit summary was generated @2014-10-10T00:00:03Z by lgfeed version 0.00 (https://github.com/arr2036/lgfeed).


More information about the Freeradius-Devel mailing list