Commit report for v3.0.x branch

Fri Oct 24 00:00:02 CEST 2014

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

======
Add --disable-openssl-version-check option

Add "--disable-openssl-version-check" configure option, which removes
checking for vulnerable OpenSSL versions. It is supposed to be used by
downstream packagers and distributions who have other means to ensure
vulnerabilities are fixed, such as versioned package dependencies and
vulnerability handling processes.

This avoids the necessity of editing radiusd.conf on package upgrade to
make sure it keeps working. At the same time, it provides safe default
to those installing FreeRADIUS from source.

Nikolai Kondrashov (via Arran Cudbard-Bell)@2014-10-23T23:54:41Z
Files modified:
	* configure
	* configure.ac
	* raddb/radiusd.conf.in
	* src/include/autoconf.h.in
	* src/include/radiusd.h
	* src/include/tls-h
	* src/main/mainconfig.c
	* src/main/radiusd.c
	* src/main/tls.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6867dd8a9c210f390c7acb3afb6ceb4284cd3226
====== 
Add missing param

Arran Cudbard-Bell at 2014-10-23T23:50:29Z
Files modified:
	* src/modules/rlm_couchbase/mod.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/ea3cbc204ed5e589c1c2336a78ca503e7caf575e
====== 
Minor wording tweaks and doxygen fixes

Arran Cudbard-Bell at 2014-10-23T23:47:34Z
Files modified:
	* src/include/md5.h
	* src/modules/rlm_cache/rlm_cache.c
	* src/modules/rlm_couchbase/couchbase.c
	* src/modules/rlm_couchbase/jsonc_missing.c
	* src/modules/rlm_couchbase/jsonc_missing.h
	* src/modules/rlm_couchbase/mod.c
	* src/modules/rlm_couchbase/rlm_couchbase.c
	* src/modules/rlm_ldap/clients.c
	* src/modules/rlm_mschap/rlm_mschap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/3cbcd460e523dc6c852ff93a798f47eda8b4cfc9
====== 
Rename redback dictionary to ericsson.ab

Arran Cudbard-Bell at 2014-10-23T23:47:21Z
Files modified:
	* share/dictionary
	* share/dictionary.ericsson.ab
	* share/dictionary.redback

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/cf06262b17acbc92d0fa971afbfa04bb31383f78
====== 
More debugging messages

Alan T. DeKok (via Arran Cudbard-Bell)@2014-10-23T23:47:20Z
Files modified:
	* src/lib/radius.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f6b23d9e61d438a788a21632326d922230c30602
====== 
Decode Starent attributes properly

Alan T. DeKok (via Arran Cudbard-Bell)@2014-10-23T23:47:20Z
Files modified:
	* src/lib/radius.c
	* src/tests/unit/all.mk
	* src/tests/unit/vendor.txt

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/bed8c51d5c83a30a4d48219561d35b2fdee90152
====== 
Add .plist for osx

Arran Cudbard-Bell at 2014-10-23T23:47:20Z
Files modified:
	* scripts/osx/README
	* scripts/osx/org.freeradius.radiusd.plist

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/1dd03cff1da5b4930fc373e3f77cc4d456d22280
====== 
TYPO

Arran Cudbard-Bell at 2014-10-23T23:47:20Z
Files modified:
	* src/include/libradius.h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c5eb9118984dc1e5fb28fea9ce28510cbc8df999
====== 
Seems to still work, and shuts doxygen up

Arran Cudbard-Bell at 2014-10-23T23:47:20Z
Files modified:
	* src/modules/rlm_couchbase/mod.c
	* src/modules/rlm_couchbase/rlm_couchbase.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f0a68651fdeddd4fa58ba245bfdbbbe3857f1a06
====== 
More minor doxygen fixes

Arran Cudbard-Bell at 2014-10-23T23:47:20Z
Files modified:
	* src/include/build.h
	* src/include/connection.h
	* src/include/libradius.h
	* src/include/log.h
	* src/include/map.h
	* src/include/md5.h
	* src/include/modules.h
	* src/include/radclient.h
	* src/include/radiusd.h
	* src/include/soh.h
	* src/include/tcp.h
	* src/include/threads.h
	* src/include/tls-h
	* src/include/tmpl.h
	* src/include/token.h
	* src/lib/dict.c
	* src/lib/radius.c
	* src/main/modcall.c
	* src/main/process.c
	* src/main/soh.c
	* src/main/tmpl.c
	* src/main/valuepair.c
	* src/main/xlat.c
	* src/modules/proto_vmps/vqp.h
	* src/modules/rlm_couchbase/couchbase.c
	* src/modules/rlm_couchbase/couchbase.h
	* src/modules/rlm_couchbase/jsonc_missing.c
	* src/modules/rlm_couchbase/jsonc_missing.h
	* src/modules/rlm_couchbase/mod.c
	* src/modules/rlm_couchbase/mod.h
	* src/modules/rlm_couchbase/rlm_couchbase.c
	* src/modules/rlm_rest/rest.c
	* src/modules/rlm_rest/rest.h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c8a83bf5791286c9f5c4ede2bf1f78c8f167b42d
====== 
Doxygen fixups

Arran Cudbard-Bell at 2014-10-23T23:47:20Z
Files modified:
	* src/lib/radius.c
	* src/main/soh.c
	* src/modules/rlm_couchbase/couchbase.c
	* src/modules/rlm_couchbase/couchbase.h
	* src/modules/rlm_couchbase/mod.c
	* src/modules/rlm_couchbase/mod.h
	* src/modules/rlm_couchbase/rlm_couchbase.c
	* src/modules/rlm_ldap/clients.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/694075fec297e8240b2f3a7241cbfaa53789e212
====== 
correct example view and change strn* functions to use the strl* counterparts

Aaron Hurt (via Arran Cudbard-Bell)@2014-10-23T23:47:20Z
Files modified:
	* src/modules/rlm_couchbase/README.md
	* src/modules/rlm_couchbase/mod.c
	* src/modules/rlm_couchbase/rlm_couchbase.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/76f14f6734305112eda3d13fd558fc7858f91204
====== 
Implement requested dynamic client capabilities.

* Implement a request to support loading clients from Couchbase doucments.
* Handles nested sections using new rlm_ldap code as an example.
* Complete Doxygen comments for all module structures and functions.

Aaron Hurt (via Arran Cudbard-Bell)@2014-10-23T23:47:20Z
Files modified:
	* raddb/mods-available/couchbase
	* src/modules/rlm_couchbase/README.md
	* src/modules/rlm_couchbase/couchbase.c
	* src/modules/rlm_couchbase/couchbase.h
	* src/modules/rlm_couchbase/jsonc_missing.c
	* src/modules/rlm_couchbase/jsonc_missing.h
	* src/modules/rlm_couchbase/mod.c
	* src/modules/rlm_couchbase/mod.h
	* src/modules/rlm_couchbase/rlm_couchbase.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/9d17a3ee36956888a992e035f9b24fc3a5474ccc
====== 
The delimiter can be \\, too

Alan T. DeKok at 2014-10-21T21:45:30Z
Files modified:
	* src/modules/rlm_realm/rlm_realm.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/93a8cc1e639fe505765388c3cc7e18f84e74f937
====== 
Fix typo @rm --> rm

Alan T. DeKok at 2014-10-21T21:43:10Z
Files modified:
	* Makefile

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/5b1b8fd21fcf1cf96ef4ac91dd91557351f39824
====== 
Document new "correct_escapes" config item

And switch examples to using it.

Alan T. DeKok at 2014-10-21T21:08:35Z
Files modified:
	* raddb/policy.d/canonicalization
	* raddb/policy.d/filter
	* raddb/proxy.conf
	* raddb/radiusd.conf.in

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2d03fd1af814b7a6b443a407a6dad242241cc898
====== 
Allow section name2 to be parsed with correct escapes

Mainly for regexes in realms.  But also for "switch"
and "case" statements

Alan T. DeKok at 2014-10-21T21:08:35Z
Files modified:
	* src/main/conffile.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c31cb62abd6b9e30ab4ff0836751757ba3e6a48b
====== 
Use new escapes, and update tests

Alan T. DeKok at 2014-10-21T21:08:35Z
Files modified:
	* src/tests/keywords/escape-sequences
	* src/tests/keywords/radiusd.conf

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f89474bd50bac8829ab6ddb56660cfc544609387
====== 
Hacks to allow "correct_escapes"

We have to set it BEFORE we parse the rest of the config files,
which means we can't do it in a CONF_PARSERS.

So... it's a hack.  But an efficient one.

Alan T. DeKok at 2014-10-21T21:08:35Z
Files modified:
	* src/main/conffile.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2d4173480658ba154e87a510af6b21fa4a0ccbc5
====== 
Use new escape when compiling "update" sections

Alan T. DeKok at 2014-10-21T21:08:35Z
Files modified:
	* src/main/modcall.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/110a63a9d4848dd2f108530086a7580534bc4986
====== 
Use new escape variable in conditional parser

Alan T. DeKok at 2014-10-21T21:00:53Z
Files modified:
	* src/main/parser.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/7aa606fc99ec2f910bebc0fd85a50d79d2193bff
====== 
Allow for new escaping rules via a global variable

Alan T. DeKok at 2014-10-21T21:00:53Z
Files modified:
	* src/include/conffile.h
	* src/main/conffile.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/dda508aeb76dc86978568f1373d8bde0b5bc84c4
====== 
If we're not escaping, pass \ straight thru

Alan T. DeKok at 2014-10-21T21:00:53Z
Files modified:
	* src/lib/token.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/0efc9aaa33f2ac8ad4dd4c51d65e4da3db6277c1
====== 
Remove extra debugging statement

Alan T. DeKok at 2014-10-21T21:00:53Z
Files modified:
	* src/main/xlat.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2ffd305d1b5c362ce517d8520168e6bb5ed55dc0
====== 
More eval debug messages

Alan T. DeKok at 2014-10-21T21:00:53Z
Files modified:
	* src/main/evaluate.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f0a3660bbc6b721e721579f3796b531b1e29cdc7
====== 
Stupid doxygen

Arran Cudbard-Bell at 2014-10-21T00:06:48Z
Files modified:
	* src/modules/rlm_ldap/clients.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/7a080453dea41ed7db592428bed40d64accb7fed
====== 
Doxygen

Arran Cudbard-Bell at 2014-10-21T00:02:11Z
Files modified:
	* src/modules/rlm_ldap/clients.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/be18242e33d94d26743cb4d38d93b89f40b30bf3
====== 
Fixup LDAP clients so nested sections are processed too

Arran Cudbard-Bell at 2014-10-20T23:57:23Z
Files modified:
	* src/include/conffile.h
	* src/main/conffile.c
	* src/modules/rlm_ldap/clients.c
	* src/modules/rlm_ldap/ldap.h
	* src/modules/rlm_ldap/rlm_ldap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/4920b6e4e881c6e197721bd3569878bb37458669
====== 
Doxygen

Arran Cudbard-Bell at 2014-10-20T01:05:13Z
Files modified:
	* src/main/modcall.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c638c612837effdd266ca3dd76479b057614875d
====== 
Modify map_afrom_cs to take a validation callback to verify and fixup maps

This makes the code much easier to follow, removes a bunch of duplicated validation code,
and avoids duplicating map_afrom_cs for any modules that have different validation rules.

Arran Cudbard-Bell at 2014-10-20T00:56:16Z
Files modified:
	* src/include/map.h
	* src/include/modcall.h
	* src/main/map.c
	* src/main/modcall.c
	* src/modules/rlm_cache/rlm_cache.c
	* src/modules/rlm_ldap/attrmap.c
	* src/modules/rlm_ldap/ldap.h
	* src/modules/rlm_ldap/rlm_ldap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/40ceeb457b0c4e871be187163e7604c863ffe7f3
====== 
Convert some log messages in map_afrom_cp to use the cp not the ci

Arran Cudbard-Bell at 2014-10-20T00:56:16Z
Files modified:
	* src/main/map.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/1b33dfbfe46fcc04f68c87e75a8bdec2eef50f6b
====== 
Force dst_ipaddr to be initialized.  Closes CID #1248167

Alan T. DeKok at 2014-10-19T16:05:22Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a8f6a08122ff5ab04e41cd757e63a3e7291e7ee5
====== 
Initialize 63 entry, too.  Closes CID #1248166

Alan T. DeKok at 2014-10-19T16:03:27Z
Files modified:
	* src/modules/rlm_expr/rlm_expr.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/d032e35194431ef849222d480cb8c621b7f81b2a
====== 
rm \n

Arran Cudbard-Bell at 2014-10-19T03:32:43Z
Files modified:
	* src/include/all.mk

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/d870e602e8939c0521b87eef11bd34ba644ef8dd
====== 
Perform substitutions for #undef autoconf macros in missing.h

Arran Cudbard-Bell at 2014-10-19T03:31:36Z
Files modified:
	* src/include/all.mk

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/4bf86839832d19090d0fe360ef3fb089294f5d13
====== 
fix missing.h substitutions to end matches at end of line (so HTON_LLL doesn't get substituted with the value for HTON_LL)

Arran Cudbard-Bell at 2014-10-19T00:42:18Z
Files modified:
	* src/include/all.mk
	* src/include/missing-h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/49203f6f317c7f9a388da89f17fafaddf44bc1f9
====== 
HAVE_HTONLL not HAVE_HTON_LL (Fix build for Yosemite)

Arran Cudbard-Bell at 2014-10-19T00:04:55Z
Files modified:
	* src/include/missing-h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2ed476cc92b0c336b84b8e38818e8227ee9dff26
====== 
Need to merge suboptions list, not insert the head of it, closes #812

Arran Cudbard-Bell at 2014-10-18T19:42:26Z
Files modified:
	* src/modules/proto_dhcp/dhcp.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/77911d553b46895e233398b5f3b517411a3795c8
====== 
Fixed check in pairvalidate

The only commit in this series that actually changes some behaviour. The test was supposed to check if one of the values (match and check) was false, while the other was true. Because the while loop already validated that at least one of them is true, we can get the desired behaviour by just checking if one of them is false. The old behaviour only tested to see if match was false.

This code is not tested in the unit tests. It would be nice to have it in there, but I've got no idea how to write that.

This fixes the following message of cppcheck:

[src/lib/valuepair.c:547]: (style) Same expression on both sides of '||'.

Herwin Weststrate (via Arran Cudbard-Bell)@2014-10-18T19:35:19Z
Files modified:
	* src/lib/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/cd5cd7d2ed94702c850803a66c0bdbc2492468db
====== 
Use *= 2 instead of += $self for doubling

The generated code is slightly more efficient too (unless you consider compiler optimizations)

This fixes the following message of cppcheck:

[src/modules/rlm_unbound/rlm_unbound.c:157]: (style) Same expression on both sides of '+='.

Herwin Weststrate (via Arran Cudbard-Bell)@2014-10-18T19:35:14Z
Files modified:
	* src/modules/rlm_unbound/rlm_unbound.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a116a1e19f69798bacde790b6bee324d9eb6998c
====== 
Dereference query pointers before checking to see if they're empty

This effectively did a check to see if the pointer was NULL, exactly what we did in the LHS of the and operator.

This fixes the following messages of cppcheck:

[src/modules/rlm_sql/rlm_sql.c:638]: (warning) Char literal compared with pointer 'inst.config.authorize_group_check_query'. Did you intend to dereference it?
[src/modules/rlm_sql/rlm_sql.c:675]: (warning) Char literal compared with pointer 'inst.config.authorize_group_reply_query'. Did you intend to dereference it?

Herwin Weststrate (via Arran Cudbard-Bell)@2014-10-18T19:34:01Z
Files modified:
	* src/modules/rlm_sql/rlm_sql.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/457ece7690e414d461a8cb57e0254351ef2139db
====== 
Removed assignment of function parameter at the end of function

This fixes the following message of cppcheck:

[src/main/tmpl.c:826]: (warning) Assignment of function parameter has no effect outside the function.

Herwin Weststrate (via Arran Cudbard-Bell)@2014-10-18T19:33:57Z
Files modified:
	* src/main/tmpl.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/999c495f7a8c41b8958475b1d7102a439c7ba2b0
====== 
Removed some unused variables

This fixes the following messages of cppcheck:

[src/modules/rlm_sql/drivers/rlm_sql_freetds/rlm_sql_freetds.c:290]: (style) Variable 'ret' is assigned a value that is never used.
[src/modules/rlm_sql/drivers/rlm_sql_freetds/rlm_sql_freetds.c:492]: (style) Variable 'ret' is assigned a value that is never used.
[src/modules/rlm_sql/drivers/rlm_sql_freetds/rlm_sql_freetds.c:573]: (style) Variable 'ret' is assigned a value that is never used.
[src/modules/rlm_passwd/rlm_passwd.c:284]: (style) Variable 'len' is assigned a value that is never used.

Herwin Weststrate (via Arran Cudbard-Bell)@2014-10-18T19:33:53Z
Files modified:
	* src/modules/rlm_passwd/rlm_passwd.c
	* src/modules/rlm_sql/drivers/rlm_sql_freetds/rlm_sql_freetds.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2baae7115b515d0c650c66c29dfbc2ec52791e88
====== 
Use %u in sprintf when printing unsigned integers

This fixes the following messages of cppcheck:

[src/modules/rlm_eap/libeap/eapcrypto.c:108]: (warning) %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'.
[src/modules/rlm_eap/libeap/eapcrypto.c:119]: (warning) %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'.
[src/modules/rlm_eap/libeap/eapcrypto.c:126]: (warning) %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'.
[src/modules/rlm_eap/libeap/eapcrypto.c:133]: (warning) %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'.
[src/modules/rlm_detail/rlm_detail.c:236]: (warning) %d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'.

Herwin Weststrate (via Arran Cudbard-Bell)@2014-10-18T19:33:49Z
Files modified:
	* src/modules/rlm_detail/rlm_detail.c
	* src/modules/rlm_eap/libeap/eapcrypto.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/1920c3ba79feb19e82f49cc9b6a61ae34f4bd75f
====== 
Decode status nibble

Arran Cudbard-Bell at 2014-10-16T17:56:21Z
Files modified:
	* src/main/version.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/d59cf5ac0d1b864c168534cf1c5e3f11953760f2
====== 
Fix OpenSSL version check issues

Arran Cudbard-Bell at 2014-10-16T17:43:36Z
Files modified:
	* src/include/radiusd.h
	* src/main/version.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6d296f2c4f3c58742543cc0508642c6d06747aea
====== 
Gotta catch em all (SIGILL)

Arran Cudbard-Bell at 2014-10-16T17:43:36Z
Files modified:
	* src/lib/debug.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f47341975ee6fe5b28c56c1134725b1eefbe5f96
====== 
doxygen

Alan T. DeKok (via Arran Cudbard-Bell)@2014-10-16T17:43:36Z
Files modified:
	* src/main/exec.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/15364609b381753fafea51d7cb3d6025eaa1e688
====== 
raddb: Use appropriate module names in traps

Specify appropriate module names for all module traps in trigger.conf,
instead of using "ldap" for all.

Nikolai Kondrashov at 2014-10-16T13:09:00Z
Files modified:
	* raddb/trigger.conf

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/ca841a1bf45b68b035be68c2b066603d68990ecd
====== 
raddb: Remove extra apostrophe from trigger.conf

Remove a spurious apostrophe from trigger.conf's trigger.modules.args.

This fixes module triggers, otherwise producing this error:

    rad_expand_xlat: Invalid string passed as argument

Nikolai Kondrashov at 2014-10-16T13:08:56Z
Files modified:
	* raddb/trigger.conf

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/df70a0501badce824b4a42e9c89a84796310f445
====== 
exec: Don't assume request presence when logging

Use DEBUG* macros for logging, instead of RDEBUG* macros in
radius_start_program and radius_readfrom_program as these are not
guaranteed to be invoked with a valid request.

For example, not from most of the exec_trigger invocations.

Nikolai Kondrashov at 2014-10-16T13:08:52Z
Files modified:
	* src/include/radiusd.h
	* src/main/exec.c
	* src/modules/rlm_mschap/rlm_mschap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6302dad574edee4e39d3ec19fbc558e41a96a48b
====== 
Assign cp

Alan T. DeKok at 2014-10-14T18:36:37Z
Files modified:
	* src/modules/rlm_ldap/attrmap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/29445c1946be402be46e3161ce914cfda2eeab2a
====== 
Fix error message

Alan T. DeKok at 2014-10-14T17:06:19Z
Files modified:
	* src/modules/rlm_ldap/attrmap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/59eb5fc57e76b1bd36fcfc8ecf7be3a9d139fade
====== 
My attempt to fix ldap:

	reply: += 'ldapAttrName'

It should work, but it's untested

Alan T. DeKok at 2014-10-14T15:57:26Z
Files modified:
	* src/modules/rlm_ldap/attrmap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2d0809fb34792119941de36d83541717d5152df1
====== 
Clearer error messages

Alan T. DeKok at 2014-10-14T15:07:45Z
Files modified:
	* src/main/map.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c56f2fcbd4a7081901ed1dc1ed402fb92e012e0c
====== 
If we're using basedn in a section, be sure it exists

Alan T. DeKok at 2014-10-14T15:07:27Z
Files modified:
	* raddb/mods-available/ldap

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b3ee058293f85c79ae4c4228f6f4f65ba40b3ab7
====== 
Don't duplicate code

Alan T. DeKok at 2014-10-13T21:24:47Z
Files modified:
	* src/main/realms.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/9604e0cf75539bf31c0f2e88bf0dd264fb72b91e
====== 
Clean up sanitize function

Alan T. DeKok at 2014-10-13T21:24:47Z
Files modified:
	* src/main/realms.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/00eb3e86036ee9fb9f59a4d56421fb4243babfc8
====== 
Remove realm_home_server_add()

It's not needed

Alan T. DeKok at 2014-10-13T21:24:47Z
Files modified:
	* src/include/realms.h
	* src/main/realms.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/195289935ed4929817eb68edf2042add48b287b3
====== 
dn needs wider scope

Arran Cudbard-Bell at 2014-10-13T16:38:43Z
Files modified:
	* src/modules/rlm_ldap/clients.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/97a1c07ab11770d5663f0209a9fc05828fc583d7
====== 
Fixup LDAP client config

Arran Cudbard-Bell at 2014-10-13T16:38:39Z
Files modified:
	* raddb/mods-available/ldap

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/28fbb8dbf71f5e9a3fd2f1f8afe53a91c1cf901b
====== 
Doxygen

Arran Cudbard-Bell at 2014-10-13T16:25:20Z
Files modified:
	* src/main/client.c
	* src/modules/rlm_ldap/clients.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6d788e36e17969ba78a499df8c928f01cd9ddaa0
====== 
Allow arbitrary attributes for LDAP clients

Arran Cudbard-Bell at 2014-10-13T16:19:08Z
Files modified:
	* src/modules/rlm_ldap/clients.c
	* src/modules/rlm_ldap/ldap.h
	* src/modules/rlm_ldap/rlm_ldap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a6ce398282b2c32de7898db9e3c6b9fafb4ce9cd
====== 
Formatting

Arran Cudbard-Bell at 2014-10-13T16:19:05Z
Files modified:
	* src/modules/rlm_sql/rlm_sql.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/09395b0a81f36bb1c56f1be163b2247a5751d21b
====== 
Move some functions around in clients.c and expose client_afrom_cs so it can be used to parse dynamically generated client sections

Arran Cudbard-Bell at 2014-10-11T22:39:24Z
Files modified:
	* src/include/radiusd.h
	* src/main/client.c
	* src/main/listen.c
	* src/modules/rlm_ldap/clients.c
	* src/modules/rlm_sql/rlm_sql.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a58c905eb1acd5012495116b346c9f2acac95b23
====== 
Fix typo

Alan T. DeKok at 2014-10-11T15:13:28Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/9c04badb6b8bca03f6416dc1b3f39501532ac215
====== 
Document 'track = yes' in the correct place

Alan T. DeKok at 2014-10-11T15:00:58Z
Files modified:
	* doc/ChangeLog
	* raddb/mods-available/detail
	* raddb/sites-available/buffered-sql
	* raddb/sites-available/copy-acct-to-home-server
	* raddb/sites-available/decoupled-accounting

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

Alan T. DeKok at 2014-10-10T21:51:03Z
Files modified:
	* doc/ChangeLog

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/89219b2d09e938632efe5c172c4fe7a523181178
====== 
Fix compiler warning

Alan T. DeKok at 2014-10-10T21:44:06Z
Files modified:
	* src/main/client.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b1f7cd47d21e3ade5391507252c86098db5ff94e
====== 
Allow CoA packets to be proxied via Packet-Dst-IP-Address

Alan T. DeKok at 2014-10-10T21:38:41Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/8e008c8b425bf295171c7ece726701522c9610c2
====== 
Allow CoA packets to be read from the detail file

Alan T. DeKok at 2014-10-10T21:08:46Z
Files modified:
	* src/main/detail.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/adf44983df6ef4d6c9f0fe3c7b4e2c916dc25359
====== 
Expose rad_coa_recv

Alan T. DeKok at 2014-10-10T21:08:46Z
Files modified:
	* src/include/radiusd.h
	* src/main/listen.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/884e7c3327a2e9c390307b710703aacf4349dbfd
====== 
Set packet->code from Packet-Type in detail file

but still force everything to be accounting packets.

We'll need a separate rad_coa() function to handle reading
coa packets from the detail file reader

Alan T. DeKok at 2014-10-10T21:08:46Z
Files modified:
	* src/main/detail.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/d9ca0a04a590473518c63edff75e84839906bb21
====== 
Add tracking when reading detail files.

So that we process each packet once, and only once

Alan T. DeKok at 2014-10-10T21:08:46Z
Files modified:
	* raddb/mods-available/detail
	* src/include/detail.h
	* src/main/detail.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/60ab352b52abc3de9a8a356511a65f0ae6c22bf4
====== 
Write timestamps even in non-compat mode

Alan T. DeKok at 2014-10-10T21:08:46Z
Files modified:
	* src/modules/rlm_detail/rlm_detail.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/7def6cd9fa58db26ecd90545c0faa88c70f8d65b
====== 
Rename vpt_types to tmpl_types

Arran Cudbard-Bell at 2014-10-10T04:13:07Z
Files modified:
	* src/include/tmpl.h
	* src/main/valuepair.c
	* src/modules/rlm_cache/rlm_cache.c
	* src/modules/rlm_ldap/attrmap.c

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

Arran Cudbard-Bell at 2014-10-08T22:55:25Z
Files modified:
	* src/include/radiusd.h
	* src/main/client.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/affd7f6614d25047d21f6eb0fd791bde336b2df6
====== 
Better error message for invalid LHS

Arran Cudbard-Bell at 2014-10-08T22:55:13Z
Files modified:
	* src/include/radiusd.h
	* src/main/evaluate.c
	* src/main/map.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/cf63972720a2ec8341eb3d20f9e39572e8616784
====== 
Fixes for detail file reader

Alan T. DeKok at 2014-10-08T22:31:00Z
Files modified:
	* src/main/detail.c
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/fd94c1fe6e0383bcde78185d13adf6ab305e8149
====== 
Clean up debug message

Alan T. DeKok at 2014-10-08T21:35:54Z
Files modified:
	* src/main/detail.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/657affb7329776ce879714869847225b408ffe04
====== 
Don't send packets with code 0

Alan T. DeKok at 2014-10-08T21:24:10Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/dfe93b90ec912860227b7c8eec90afb4fe4e2ea3
====== 
Fix assertion to only check for TCP sockets

Alan T. DeKok at 2014-10-08T21:00:12Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/211ac18fe937ed0f879075467334db98cde8ccca
====== 
Fix TRUSTROUTER -> TRUST_ROUTER

Kevin Wasserman (via Alan T. DeKok)@2014-10-08T20:58:30Z
Files modified:
	* src/modules/rlm_realm/configure
	* src/modules/rlm_realm/configure.ac

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b8a944e7774ee093182c483784b9dbb6e340ff1f
====== 
Update ChangeLog

Arran Cudbard-Bell at 2014-10-07T23:48:42Z
Files modified:
	* doc/ChangeLog

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f31d0371f8fe50fac7db5a77e930297201c06a44
====== 
Correct fix for #808

Arran Cudbard-Bell at 2014-10-07T23:42:32Z
Files modified:
	* src/modules/rlm_rest/rest.c
	* src/modules/rlm_rest/rlm_rest.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/33716e3909e76593ed27eb2f674554722a70e311
====== 
Prevent assert(0) in rlm_rest

If the RADIUS server is compiled without JSON support, it is still possible that the other end of the rlm_rest module returns JSON. This triggered an "assert(0)" and crashed the server. This fixes this by printing an error message to indicate that the reply of the REST server couldn't be parsed, and failing the module gracefully.

Herwin Weststrate (via Alan T. DeKok)@2014-10-07T22:43:18Z
Files modified:
	* src/modules/rlm_rest/rest.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/af3c83baf0d011d63ce663db08eb128a01377bb7
====== 
Even when there's no response, poke the detail file reader

So that it retranmits.

Alan T. DeKok at 2014-10-07T22:27:39Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/ab65c89b3ed2ecfb9c4c9b1e7b462c9b5ff89007
====== 
Handle failures for internally proxied requests.

If there's no reply, deal with that case, rather than
assuming that everything's OK

Alan T. DeKok at 2014-10-07T19:02:21Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/4a9a8e85b256bdfc25ab0c1f0b3f90fa07041188
====== 
Don't check synchronous / nodup for detail sockets

Alan T. DeKok at 2014-10-07T18:01:46Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/54c9a55aac7c9f728e19056fe3f2e96a0786d11e
====== 
Use correct definition

Alan T. DeKok at 2014-10-07T17:53:13Z
Files modified:
	* src/main/parser.c

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