Commit report for v3.1.x branch

Sat Feb 20 14:18:05 CET 2016

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

======
fix empty condition body (NDEBUG)

Arran Cudbard-Bell at 2015-12-11T23:54:06Z
Files modified:
	* src/modules/rlm_cache/drivers/rlm_cache_rbtree/rlm_cache_rbtree.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b8e824bab25f25bb0661a8dd6e4a6cd95304e46d
====== 
clang scan whackamole

Arran Cudbard-Bell at 2015-12-11T23:51:10Z
Files modified:
	* src/main/evaluate.c
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/1e593620d7b708de235c6f1f62469082c57506ae
====== 
Use instance when available

Arran Cudbard-Bell at 2015-12-11T23:40:17Z
Files modified:
	* src/modules/rlm_eap/eap.c
	* src/modules/rlm_eap/rlm_eap.h
	* src/modules/rlm_eap/types/rlm_eap_mschapv2/rlm_eap_mschapv2.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2a332dcc85cf7928e5097a760c537b074580b4ab
====== 
Assert second

Arran Cudbard-Bell at 2015-12-11T23:40:07Z
Files modified:
	* src/modules/rlm_cache/drivers/rlm_cache_redis/rlm_cache_redis.c
	* src/modules/rlm_redis/rlm_redis.c
	* src/modules/rlm_rediswho/rlm_rediswho.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/83ceaeabcd5669cc54b47d6111b5e0b91094cc25
====== 
Maybe just not returning will satisfy everyone...

Arran Cudbard-Bell at 2015-12-11T23:13:59Z
Files modified:
	* src/main/util.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c3d36db103465a245d6fdcd1574d9c7c5fe44de5
====== 
Exit with a non-zero return code

Arran Cudbard-Bell at 2015-12-11T22:39:19Z
Files modified:
	* src/lib/udpfromto.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b632c7504184d60da50018c064d0af3ba7a7074f
====== 
Use the value returned by fr_assert

Arran Cudbard-Bell at 2015-12-11T22:36:00Z
Files modified:
	* src/lib/dict.c
	* src/lib/event.c
	* src/lib/pair.c
	* src/lib/pcap.c
	* src/lib/socket.c
	* src/lib/value.c
	* src/main/tmpl.c
	* src/main/util.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/d3097ff37e4296914d0eb9b99238be8d07ceee10
====== 
Fix another __clang_analyzer__ issue

Arran Cudbard-Bell at 2015-12-11T22:03:49Z
Files modified:
	* src/modules/proto_dhcp/dhcp.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c459971b8161d0d757567b0f32b2f0dc7cc6147c
====== 
Turn off invalid-noreturn for rad_assert_fail... We know it's invalid

Arran Cudbard-Bell at 2015-12-11T21:53:07Z
Files modified:
	* src/main/util.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f79cff2096ae0b9d28ee7cd81605b71b2715d99d
====== 
Revert "Pedantic compiler"

This reverts commit 9f2540adcd23bd6a53f888940dfa965b90894fb5.

Arran Cudbard-Bell at 2015-12-11T21:52:46Z
Files modified:
	* src/include/rad_assert.h
	* src/main/util.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a07435a4aedc2563614a3e1171ad1e06820045d9
====== 
Pedantic compiler

Arran Cudbard-Bell at 2015-12-11T21:22:19Z
Files modified:
	* src/include/rad_assert.h
	* src/main/util.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/9f2540adcd23bd6a53f888940dfa965b90894fb5
====== 
Better warning

Arran Cudbard-Bell at 2015-12-11T21:20:09Z
Files modified:
	* src/main/tls.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/22cd0eccac2477d42ff80b44c6eaa576e7cd278c
====== 
Fix all clang scan NDEBUG issues

Arran Cudbard-Bell at 2015-12-11T20:51:00Z
Files modified:
	* src/include/libradius.h
	* src/include/rad_assert.h
	* src/lib/cursor.c
	* src/lib/debug.c
	* src/main/connection.c
	* src/main/listen.c
	* src/main/map.c
	* src/main/tls_listen.c
	* src/main/util.c
	* src/modules/proto_dhcp/dhcpd.c
	* src/modules/rlm_cache/drivers/rlm_cache_redis/rlm_cache_redis.c
	* src/modules/rlm_eap/rlm_eap.c
	* src/modules/rlm_eap/types/rlm_eap_mschapv2/rlm_eap_mschapv2.c
	* src/modules/rlm_eap/types/rlm_eap_peap/peap.c
	* src/modules/rlm_eap/types/rlm_eap_ttls/ttls.c
	* src/modules/rlm_redis/cluster.c
	* src/modules/rlm_redis/rlm_redis.c
	* src/modules/rlm_rediswho/rlm_rediswho.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/60e6cfdbfbd77ab76034837d9f9de9e5585afebd
====== 
Revert "If we don't care about the result, ignore it."

This reverts commit 84608495a557a89160599971e3391367ee825f2b.

Arran Cudbard-Bell at 2015-12-11T17:44:30Z
Files modified:
	* src/modules/rlm_eap/types/rlm_eap_mschapv2/rlm_eap_mschapv2.c
	* src/modules/rlm_eap/types/rlm_eap_peap/peap.c
	* src/modules/rlm_eap/types/rlm_eap_ttls/ttls.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/8427ae47806695d1be7adc90b8a84cfb633ef1b8
====== 
Pass correct struct to field counting functions

Arran Cudbard-Bell at 2015-12-11T17:33:46Z
Files modified:
	* src/modules/rlm_sql/drivers/rlm_sql_mysql/rlm_sql_mysql.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/301dc4a9fe16aafd739553773fe46cd09110e36f
====== 
disable tls 1.2 for OpenSSL 1.0.1f and 1.0.1g

Alan T. DeKok at 2015-12-11T15:15:54Z
Files modified:
	* src/main/tls.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/884fbe1e4232fcb4cdafc3ee9ea8b68ef4880cc7
====== 
more tests for empty sections

Alan T. DeKok at 2015-12-11T14:44:40Z
Files modified:
	* src/tests/keywords/else-empty-error
	* src/tests/keywords/if-empty-allow

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b4a23970ad481fd6de81d690f19d63880ec893cc
====== 
rename variable to be clearer on what it is

Alan T. DeKok at 2015-12-11T14:44:40Z
Files modified:
	* src/main/modcall.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/683f02bacb261bc20728a322a4c65a67520b29d1
====== 
pull checks for empty blocks into the compiler

Alan T. DeKok at 2015-12-11T14:44:40Z
Files modified:
	* src/main/modcall.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/52719994d5d1ce94b9c531707b691527cd629035
====== 
return if there are no children

Alan T. DeKok at 2015-12-11T14:44:40Z
Files modified:
	* src/main/interpreter.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a1f2e2d49cc3abc8ee23cb196b0fa99474528155
====== 
Removed option winbind_allow_mschapv2 in rlm_mschap

Herwin Weststrate (via Arran Cudbard-Bell)@2015-12-11T14:32:12Z
Files modified:
	* src/modules/rlm_mschap/auth_wbclient.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/5c4bb16585c7cf988d3d3c6937368f26062c17a1
====== 
move check into assert

Alan T. DeKok at 2015-12-11T13:14:12Z
Files modified:
	* src/main/util.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c6704428478bf6eed1e785ed9bd85867a82baa31
====== 
remove redundant assignment

Alan T. DeKok at 2015-12-11T13:13:06Z
Files modified:
	* src/main/interpreter.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/56974bacc08043e013169fdb7aa27ab6e04c0049
====== 
Quiet gcc

Alan T. DeKok at 2015-12-11T03:09:13Z
Files modified:
	* src/main/xlat.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/8dbb2b3aac3fdc90249a85b79fad6cb51adc7225
====== 
move assert to run-time check.

gcc is too damned picky, and we can't annotate fields to say
"we KNOW this isn't NULL"

Alan T. DeKok at 2015-12-11T03:07:25Z
Files modified:
	* src/main/map.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2f60baf60150d5d2f7edf57c05636531c55a94da
====== 
always initialize "found"

Alan T. DeKok at 2015-12-11T03:06:35Z
Files modified:
	* src/main/connection.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/dfe81fe3774cadd5ad11d3664630b3b3d185b327
====== 
Ensure "found" is always set

Alan T. DeKok at 2015-12-11T03:04:53Z
Files modified:
	* src/main/interpreter.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/bd5d268fa76f605643cbe9e2b267fd1e495ff5a4
====== 
always use this assert

Alan T. DeKok at 2015-12-11T03:03:13Z
Files modified:
	* src/modules/proto_dhcp/dhcpd.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/ff45167fdd9239a168165ebf0a612f1ea5cad9d3
====== 
move assert to run-time check

Alan T. DeKok at 2015-12-11T03:01:27Z
Files modified:
	* src/modules/rlm_redis/cluster.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/887f23f6ce2d66b6363f2760c38106a460878f53
====== 
move assert to run-time check

Alan T. DeKok at 2015-12-11T03:00:28Z
Files modified:
	* src/modules/rlm_redis_ippool/rlm_redis_ippool_tool.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/59bbfcfeb643171ff8125ac8c0d9b2a2c8f7953b
====== 
redisCommand can return NULL

Alan T. DeKok at 2015-12-11T02:59:09Z
Files modified:
	* src/modules/rlm_redis/rlm_redis.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/d0ff9f6fc3a933191fffc68a70968a3968d12529
====== 
If we don't care about the result, ignore it.

Alan T. DeKok at 2015-12-11T02:57:52Z
Files modified:
	* src/modules/rlm_eap/types/rlm_eap_peap/peap.c
	* src/modules/rlm_eap/types/rlm_eap_ttls/ttls.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/eee7f5f693cac0898159189fd9fee1f166427f40
====== 
If we don't care about the result, ignore it.

Alan T. DeKok at 2015-12-11T02:55:49Z
Files modified:
	* src/modules/rlm_eap/types/rlm_eap_mschapv2/rlm_eap_mschapv2.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/84608495a557a89160599971e3391367ee825f2b
====== 
redisCommand can return NULL.

https://github.com/redis/hiredis

The return value of redisCommand holds a reply when the command was successfully executed. When an error occurs, the return value is NULL ...

Alan T. DeKok at 2015-12-11T02:54:28Z
Files modified:
	* src/modules/rlm_cache/drivers/rlm_cache_redis/rlm_cache_redis.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/d6e01f4be7068ccb6a8c1a8a02ba2a01d524930c
====== 
more assertions

Alan T. DeKok at 2015-12-11T01:31:20Z
Files modified:
	* src/main/evaluate.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/09bd958273a131326c3f032ea37954496558c819
====== 
better checks for non null

Alan T. DeKok at 2015-12-11T00:06:53Z
Files modified:
	* src/main/evaluate.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b018feab550f2f5b107a5cf43c1c392d99d8a30f
====== 
-- 
This commit summary was generated @2015-12-12T00:00:02Z by lgfeed version 0.00 (https://github.com/arr2036/lgfeed).