Commit report for v3.0.x branch
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/6867dd8a9c210f390c7ac... ====== Add missing param Arran Cudbard-Bell@2014-10-23T23:50:29Z Files modified: * src/modules/rlm_couchbase/mod.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/ea3cbc204ed5e589c1c23... ====== Minor wording tweaks and doxygen fixes Arran Cudbard-Bell@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/3cbcd460e523dc6c852ff... ====== Rename redback dictionary to ericsson.ab Arran Cudbard-Bell@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/cf06262b17acbc92d0fa9... ====== 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/f6b23d9e61d438a788a21... ====== 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/bed8c51d5c83a30a4d482... ====== Add .plist for osx Arran Cudbard-Bell@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/1dd03cff1da5b4930fc37... ====== TYPO Arran Cudbard-Bell@2014-10-23T23:47:20Z Files modified: * src/include/libradius.h Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/c5eb9118984dc1e5fb28f... ====== Seems to still work, and shuts doxygen up Arran Cudbard-Bell@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/f0a68651fdeddd4fa58ba... ====== More minor doxygen fixes Arran Cudbard-Bell@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/c8a83bf5791286c9f5c4e... ====== Doxygen fixups Arran Cudbard-Bell@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/694075fec297e8240b2f3... ====== 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/76f14f6734305112eda3d... ====== 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/9d17a3ee36956888a992e... ====== The delimiter can be \\, too Alan T. DeKok@2014-10-21T21:45:30Z Files modified: * src/modules/rlm_realm/rlm_realm.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/93a8cc1e639fe50576538... ====== Fix typo @rm --> rm Alan T. DeKok@2014-10-21T21:43:10Z Files modified: * Makefile Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/5b1b8fd21fcf1cf96ef4a... ====== Document new "correct_escapes" config item And switch examples to using it. Alan T. DeKok@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/2d03fd1af814b7a6b443a... ====== Allow section name2 to be parsed with correct escapes Mainly for regexes in realms. But also for "switch" and "case" statements Alan T. DeKok@2014-10-21T21:08:35Z Files modified: * src/main/conffile.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/c31cb62abd6b9e30ab4ff... ====== Use new escapes, and update tests Alan T. DeKok@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/f89474bd50bac8829ab6d... ====== 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@2014-10-21T21:08:35Z Files modified: * src/main/conffile.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/2d4173480658ba154e87a... ====== Use new escape when compiling "update" sections Alan T. DeKok@2014-10-21T21:08:35Z Files modified: * src/main/modcall.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/110a63a9d4848dd2f1085... ====== Use new escape variable in conditional parser Alan T. DeKok@2014-10-21T21:00:53Z Files modified: * src/main/parser.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/7aa606fc99ec2f910bebc... ====== Allow for new escaping rules via a global variable Alan T. DeKok@2014-10-21T21:00:53Z Files modified: * src/include/conffile.h * src/main/conffile.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/dda508aeb76dc86978568... ====== If we're not escaping, pass \ straight thru Alan T. DeKok@2014-10-21T21:00:53Z Files modified: * src/lib/token.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/0efc9aaa33f2ac8ad4dd4... ====== Remove extra debugging statement Alan T. DeKok@2014-10-21T21:00:53Z Files modified: * src/main/xlat.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/2ffd305d1b5c362ce517d... ====== More eval debug messages Alan T. DeKok@2014-10-21T21:00:53Z Files modified: * src/main/evaluate.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/f0a3660bbc6b721e72157... ====== Stupid doxygen Arran Cudbard-Bell@2014-10-21T00:06:48Z Files modified: * src/modules/rlm_ldap/clients.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/7a080453dea41ed7db592... ====== Doxygen Arran Cudbard-Bell@2014-10-21T00:02:11Z Files modified: * src/modules/rlm_ldap/clients.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/be18242e33d94d26743cb... ====== Fixup LDAP clients so nested sections are processed too Arran Cudbard-Bell@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/4920b6e4e881c6e197721... ====== Doxygen Arran Cudbard-Bell@2014-10-20T01:05:13Z Files modified: * src/main/modcall.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/c638c612837effdd266ca... ====== 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@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/40ceeb457b0c4e871be18... ====== Convert some log messages in map_afrom_cp to use the cp not the ci Arran Cudbard-Bell@2014-10-20T00:56:16Z Files modified: * src/main/map.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/1b33dfbfe46fcc04f68c8... ====== Force dst_ipaddr to be initialized. Closes CID #1248167 Alan T. DeKok@2014-10-19T16:05:22Z Files modified: * src/main/process.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/a8f6a08122ff5ab04e41c... ====== Initialize 63 entry, too. Closes CID #1248166 Alan T. DeKok@2014-10-19T16:03:27Z Files modified: * src/modules/rlm_expr/rlm_expr.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/d032e35194431ef849222... ====== rm \n Arran Cudbard-Bell@2014-10-19T03:32:43Z Files modified: * src/include/all.mk Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/d870e602e8939c0521b87... ====== Perform substitutions for #undef autoconf macros in missing.h Arran Cudbard-Bell@2014-10-19T03:31:36Z Files modified: * src/include/all.mk Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/4bf86839832d19090d0fe... ====== 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@2014-10-19T00:42:18Z Files modified: * src/include/all.mk * src/include/missing-h Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/49203f6f317c7f9a388da... ====== HAVE_HTONLL not HAVE_HTON_LL (Fix build for Yosemite) Arran Cudbard-Bell@2014-10-19T00:04:55Z Files modified: * src/include/missing-h Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/2ed476cc92b0c336b84b8... ====== Need to merge suboptions list, not insert the head of it, closes #812 Arran Cudbard-Bell@2014-10-18T19:42:26Z Files modified: * src/modules/proto_dhcp/dhcp.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/77911d553b46895e23339... ====== 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/cd5cd7d2ed94702c85080... ====== 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/a116a1e19f69798bacde7... ====== 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/457ece7690e414d461a8c... ====== 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/999c495f7a8c41b895847... ====== 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/2baae7115b515d0c650c6... ====== 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/1920c3ba79feb19e82f49... ====== Decode status nibble Arran Cudbard-Bell@2014-10-16T17:56:21Z Files modified: * src/main/version.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/d59cf5ac0d1b864c16853... ====== Fix OpenSSL version check issues Arran Cudbard-Bell@2014-10-16T17:43:36Z Files modified: * src/include/radiusd.h * src/main/version.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/6d296f2c4f3c58742543c... ====== Gotta catch em all (SIGILL) Arran Cudbard-Bell@2014-10-16T17:43:36Z Files modified: * src/lib/debug.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/f47341975ee6fe5b28c56... ====== 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/15364609b381753fafea5... ====== 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@2014-10-16T13:09:00Z Files modified: * raddb/trigger.conf Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/ca841a1bf45b68b035be6... ====== 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@2014-10-16T13:08:56Z Files modified: * raddb/trigger.conf Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/df70a0501badce824b4a4... ====== 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@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/6302dad574edee4e39d3e... ====== Assign cp Alan T. DeKok@2014-10-14T18:36:37Z Files modified: * src/modules/rlm_ldap/attrmap.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/29445c1946be402be46e3... ====== Fix error message Alan T. DeKok@2014-10-14T17:06:19Z Files modified: * src/modules/rlm_ldap/attrmap.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/59eb5fc57e76b1bd36fcf... ====== My attempt to fix ldap: reply: += 'ldapAttrName' It should work, but it's untested Alan T. DeKok@2014-10-14T15:57:26Z Files modified: * src/modules/rlm_ldap/attrmap.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/2d0809fb34792119941de... ====== Clearer error messages Alan T. DeKok@2014-10-14T15:07:45Z Files modified: * src/main/map.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/c56f2fcbd4a7081901ed1... ====== If we're using basedn in a section, be sure it exists Alan T. DeKok@2014-10-14T15:07:27Z Files modified: * raddb/mods-available/ldap Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/b3ee058293f85c79ae4c4... ====== Don't duplicate code Alan T. DeKok@2014-10-13T21:24:47Z Files modified: * src/main/realms.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/9604e0cf75539bf31c0f2... ====== Clean up sanitize function Alan T. DeKok@2014-10-13T21:24:47Z Files modified: * src/main/realms.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/00eb3e86036ee9fb9f59a... ====== Remove realm_home_server_add() It's not needed Alan T. DeKok@2014-10-13T21:24:47Z Files modified: * src/include/realms.h * src/main/realms.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/195289935ed4929817eb6... ====== dn needs wider scope Arran Cudbard-Bell@2014-10-13T16:38:43Z Files modified: * src/modules/rlm_ldap/clients.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/97a1c07ab11770d5663f0... ====== Fixup LDAP client config Arran Cudbard-Bell@2014-10-13T16:38:39Z Files modified: * raddb/mods-available/ldap Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/28fbb8dbf71f5e9a3fd2f... ====== Doxygen Arran Cudbard-Bell@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/6d788e36e17969ba78a49... ====== Allow arbitrary attributes for LDAP clients Arran Cudbard-Bell@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/a6ce398282b2c32de7898... ====== Formatting Arran Cudbard-Bell@2014-10-13T16:19:05Z Files modified: * src/modules/rlm_sql/rlm_sql.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/09395b0a81f36bb1c56f1... ====== 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@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/a58c905eb1acd50124951... ====== Fix typo Alan T. DeKok@2014-10-11T15:13:28Z Files modified: * src/main/process.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/9c04badb6b8bca03f6416... ====== Document 'track = yes' in the correct place Alan T. DeKok@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/99ad37540f20a99129621... ====== Note recent changes Alan T. DeKok@2014-10-10T21:51:03Z Files modified: * doc/ChangeLog Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/89219b2d09e938632efe5... ====== Fix compiler warning Alan T. DeKok@2014-10-10T21:44:06Z Files modified: * src/main/client.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/b1f7cd47d21e3ade53915... ====== Allow CoA packets to be proxied via Packet-Dst-IP-Address Alan T. DeKok@2014-10-10T21:38:41Z Files modified: * src/main/process.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/8e008c8b425bf295171c7... ====== Allow CoA packets to be read from the detail file Alan T. DeKok@2014-10-10T21:08:46Z Files modified: * src/main/detail.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/adf44983df6ef4d6c9f0f... ====== Expose rad_coa_recv Alan T. DeKok@2014-10-10T21:08:46Z Files modified: * src/include/radiusd.h * src/main/listen.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/884e7c3327a2e9c390307... ====== 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@2014-10-10T21:08:46Z Files modified: * src/main/detail.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/d9ca0a04a590473518c63... ====== Add tracking when reading detail files. So that we process each packet once, and only once Alan T. DeKok@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/60ab352b52abc3de9a8a3... ====== Write timestamps even in non-compat mode Alan T. DeKok@2014-10-10T21:08:46Z Files modified: * src/modules/rlm_detail/rlm_detail.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/7def6cd9fa58db26ecd90... ====== Rename vpt_types to tmpl_types Arran Cudbard-Bell@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/1cbece65b8064506fe04c... ====== cf_item_add defined too many times... Arran Cudbard-Bell@2014-10-09T21:01:11Z Files modified: * src/include/conffile.h Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/f46e3f1e4b983d905de95... ====== Support adding arbitrary attributes for dynamic clients Arran Cudbard-Bell@2014-10-09T20:18:29Z Files modified: * src/main/client.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/e1a95205af48242450f4b... ====== Expose cf_item_add and cf_pair_alloc Arran Cudbard-Bell@2014-10-09T16:53:43Z Files modified: * src/include/conffile.h * src/main/conffile.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/c47872be2ce4d4eab3f72... ====== Still need the loop Arran Cudbard-Bell@2014-10-09T16:51:04Z Files modified: * src/lib/valuepair.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/6ef6fea4d19da9962dd94... ====== Adding the same VP twice is bad mm'k Fixes #809 Arran Cudbard-Bell@2014-10-09T16:32:01Z Files modified: * src/lib/valuepair.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/ce82761ec3855fe7499c8... ====== Typo Arran Cudbard-Bell@2014-10-09T16:27:09Z Files modified: * src/lib/debug.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/78503e4bf60b9eb08a987... ====== Note recent changes Alan T. DeKok@2014-10-09T16:16:36Z Files modified: * doc/ChangeLog Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/947394e707eb1a7aeedaf... ====== Merge pull request #809 from Feh/pairadd-ignore-dups pairadd: Don't add a VP if it's already present Alan DeKok@2014-10-09T16:16:15Z Files modified: * src/lib/valuepair.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/21391a48cf57a69f43a64... ====== 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@cis.fu-berlin.de> Julius Plenz@2014-10-09T12:48:37Z Files modified: * src/lib/valuepair.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/b73a4c5c0c4c6c6702925... ====== More verbose error messages about VALUE_PAIRs with bad dictionary entries Arran Cudbard-Bell@2014-10-09T13:58:41Z Files modified: * src/lib/debug.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/589ef23ff8f8fdf5e67b7... ====== Formatting Arran Cudbard-Bell@2014-10-08T22:55:25Z Files modified: * src/include/radiusd.h * src/main/client.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/affd7f6614d25047d21f6... ====== Better error message for invalid LHS Arran Cudbard-Bell@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/cf63972720a2ec8341eb3... ====== Fixes for detail file reader Alan T. DeKok@2014-10-08T22:31:00Z Files modified: * src/main/detail.c * src/main/process.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/fd94c1fe6e0383bcde781... ====== Clean up debug message Alan T. DeKok@2014-10-08T21:35:54Z Files modified: * src/main/detail.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/657affb7329776ce87971... ====== Don't send packets with code 0 Alan T. DeKok@2014-10-08T21:24:10Z Files modified: * src/main/process.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/dfe93b90ec912860227b7... ====== Fix assertion to only check for TCP sockets Alan T. DeKok@2014-10-08T21:00:12Z Files modified: * src/main/process.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/211ac18fe937ed0f87907... ====== 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/b8a944e7774ee093182c4... ====== Update ChangeLog Arran Cudbard-Bell@2014-10-07T23:48:42Z Files modified: * doc/ChangeLog Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/f31d0371f8fe50fac7db5... ====== Correct fix for #808 Arran Cudbard-Bell@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/33716e3909e76593ed27e... ====== 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/af3c83baf0d011d63ce66... ====== Even when there's no response, poke the detail file reader So that it retranmits. Alan T. DeKok@2014-10-07T22:27:39Z Files modified: * src/main/process.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/ab65c89b3ed2ecfb9c4c9... ====== 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@2014-10-07T19:02:21Z Files modified: * src/main/process.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/4a9a8e85b256bdfc25ab0... ====== Don't check synchronous / nodup for detail sockets Alan T. DeKok@2014-10-07T18:01:46Z Files modified: * src/main/process.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/54c9a55aac7c9f728e190... ====== Use correct definition Alan T. DeKok@2014-10-07T17:53:13Z Files modified: * src/main/parser.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/326c0601ba7c87998e662... ====== -- This commit summary was generated @2014-10-24T00:00:02Z by lgfeed version 0.00 (https://github.com/arr2036/lgfeed).
participants (1)
-
announceļ¼ freeradius.org