Commit report for v3.0.x branch

Sat May 3 00:00:21 CEST 2014

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

======
Revert "Use realloc to correct the length of char buffers in pairparsevalue"

This reverts commit 4a6511458ec66c4bb9ea14e061f37cc5cc6d2b0e.

Arran Cudbard-Bell at 2014-05-02T20:35:53Z
Files modified:
	* src/lib/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/5f4411bf23f602f873ca8c3bc5ca39266a95f912
====== 
It's ok for length to be <= octets buffer

Arran Cudbard-Bell at 2014-05-02T20:35:10Z
Files modified:
	* src/lib/debug.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/acce89d3c392a9dd7b892bad1458f9841ac76b96
====== 
Exit after asserting in fr_verify_vp

Arran Cudbard-Bell at 2014-05-02T20:34:33Z
Files modified:
	* src/lib/debug.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/ce8995d568996e2858356ac0cfac90735c46c8a4
====== 
If there's no action set we should still need to check if the signal was SIGUSR1

Arran Cudbard-Bell at 2014-05-02T20:31:10Z
Files modified:
	* src/lib/debug.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/611a2a7de78bb2266cac88af8061fda0e7c3b36a
====== 
Remove useless panic action

Alan T. DeKok at 2014-05-02T20:23:27Z
Files modified:
	* src/tests/keywords/radiusd.conf

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/84b6f3de0a6af0d2875ccad8a4230424c952e779
====== 
The attribute for Password-With-Header might be a string

Alan T. DeKok at 2014-05-02T20:21:23Z
Files modified:
	* src/modules/rlm_pap/rlm_pap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/5472f0af95386dc0ac2d783493a3af37150a9092
====== 
It's OK for the buffer to be too big.

Also print out the name of the offending attribute

Alan T. DeKok at 2014-05-02T20:18:38Z
Files modified:
	* src/lib/debug.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/0536e587cfd14239e2ac65f0a5839316cde01c00
====== 
sp

Arran Cudbard-Bell at 2014-05-02T18:42:44Z
Files modified:
	* src/lib/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/3a6ce9f833d58a275aa979ae853f372ac018be5a
====== 
Use realloc to correct the length of char buffers in pairparsevalue

Arran Cudbard-Bell at 2014-05-02T18:41:13Z
Files modified:
	* src/lib/valuepair.c

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

Arran Cudbard-Bell at 2014-05-02T18:41:13Z
Files modified:
	* src/lib/valuepair.c
	* src/main/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/4b0d415902932d4a727c30669ef9d5e13850ae94
====== 
Comments

Arran Cudbard-Bell at 2014-05-02T18:41:13Z
Files modified:
	* src/main/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/dd07cea62de2dc8a50fd0d0587303260037f6dd3
====== 
turn VERIFY_VP into a function, the macro was getting very long

Arran Cudbard-Bell at 2014-05-02T18:41:13Z
Files modified:
	* src/include/libradius.h
	* src/lib/debug.c

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

Alan T. DeKok at 2014-05-02T17:24:24Z
Files modified:
	* doc/ChangeLog

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/48e92699c469f16ba877fcd74267d8d58537174f
====== 
A dup isn't a dup if the old request is done

Alan T. DeKok at 2014-05-02T17:13:01Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/1ec0825e61739fe0459d3e0c44277e894b3598b2
====== 
More tracing on dup / conflicting packet

Alan T. DeKok at 2014-05-02T16:58:53Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/921c4f9d1eb82315e4d8606553cd58ea116e2a3d
====== 
Always print authenticator even if there are no vps

Arran Cudbard-Bell at 2014-05-02T16:27:28Z
Files modified:
	* src/main/radsniff.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/819a4dc2a42795bce1850a9c67be55fbfeaae4ec
====== 
Output contents of authenticator in radsniff

Arran Cudbard-Bell at 2014-05-02T16:22:00Z
Files modified:
	* src/main/radsniff.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/991681a5ebbeccca13215748e0490be586cdbfa3
====== 
Use "ignoring" instead of bureaucratic sounding phrase

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T16:08:01Z
Files modified:
	* src/main/modules.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/273fb32a01bbd4ddef828bdf8165cdc75bab3dfc
====== 
Check length of octets type attributes too

Arran Cudbard-Bell at 2014-05-02T16:03:43Z
Files modified:
	* src/include/libradius.h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6dc33362bfa4c49d0234f53e7381c551d9f0ff67
====== 
Always \0 terminate string type attributes in mschap_add_reply()

Arran Cudbard-Bell at 2014-05-02T15:57:57Z
Files modified:
	* src/modules/rlm_mschap/mschap.h
	* src/modules/rlm_mschap/rlm_mschap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/40f52faf95ccbcb6bdc2ddbaef38b81a9f84b881
====== 
Formatting and cases of vp_strvalue being used for octet type attributes

Arran Cudbard-Bell at 2014-05-02T15:57:57Z
Files modified:
	* src/modules/rlm_eap/types/rlm_eap_mschapv2/rlm_eap_mschapv2.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/06853be89e16915454c1ec3d6fbb5b7227d3442d
====== 
Another regex memory leak...

Arran Cudbard-Bell at 2014-05-02T15:57:57Z
Files modified:
	* src/lib/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/0e66356bd30b6b3b5478ceec227db94b139eb86d
====== 
Don't leak memory when pairmake is called with a regular expression comparison

Arran Cudbard-Bell at 2014-05-02T15:57:57Z
Files modified:
	* src/lib/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/fc8ba9a0a27f5dd5659666b25b99990bd4786147
====== 
Reorder operations in mschap_add_reply to try and unconfuse valgrind

Arran Cudbard-Bell at 2014-05-02T15:57:57Z
Files modified:
	* src/modules/rlm_mschap/rlm_mschap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/358804c509d22659d00423e7a39a75213d28265c
====== 
Valgrind complains about sscanf in rlm_eap_mschapv2.c reading past the end of response->vp_strvalue. Use VERIFY_VP to check that this can't be happening.

Arran Cudbard-Bell at 2014-05-02T15:57:57Z
Files modified:
	* src/modules/rlm_eap/types/rlm_eap_mschapv2/rlm_eap_mschapv2.c

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

Arran Cudbard-Bell at 2014-05-02T15:57:57Z
Files modified:
	* src/modules/rlm_eap/types/rlm_eap_mschapv2/rlm_eap_mschapv2.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2dca8519930737777ab8e0d88e38ee826bc64017
====== 
Add check to VERIFY_VP to check that the string buffer is still \0 terminated, and that the length of the of the buffer matches the length of the VALUE_PAIR.

Arran Cudbard-Bell at 2014-05-02T15:57:57Z
Files modified:
	* src/include/libradius.h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/372899d953708dc2420757220f0f47b38a5278cc
====== 
Port acinclude changes from master

Arran Cudbard-Bell at 2014-05-02T15:57:56Z
Files modified:
	* acinclude.m4

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/48157455ab8466539cf69f15c3cb395db972d193
====== 
Make the server seem less alerted

Arran Cudbard-Bell at 2014-05-02T15:57:56Z
Files modified:
	* src/main/modules.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/406baa76fadf2491ce5e060a77620524776bfef4
====== 
Remove apple SDK hacks. There's now the option to install the headers in their usual location.

Arran Cudbard-Bell at 2014-05-02T15:57:56Z
Files modified:
	* Make.inc.in
	* Makefile
	* configure
	* scripts/libtool.mk

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/99d7d913f8b5108e5cab6954f956313143567b91
====== 
Last two Linux clang --analyzer issues

Arran Cudbard-Bell at 2014-05-02T15:57:56Z
Files modified:
	* src/lib/event.c
	* src/lib/udpfromto.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/e3a105c13707ca8795c0341ab1307971789b2b01
====== 
Switch scan output format to html

Arran Cudbard-Bell at 2014-05-02T15:57:56Z
Files modified:
	* scripts/boiler.mk

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/fb70367ed00add46bf8b91286312a8ef192dfdba
====== 
Still need to NULL terminate env vars array

Arran Cudbard-Bell at 2014-05-02T15:57:56Z
Files modified:
	* src/main/exec.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/00e16c098cd92a11a2da5764a480dce4ea8be84f
====== 
Fix scan complaints in radius_start_program

Arran Cudbard-Bell at 2014-05-02T15:57:56Z
Files modified:
	* src/main/exec.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/e053e33b77a1acce8500a7414731cd21f3ab7c18
====== 
Various scan-build fixes #2

Arran Cudbard-Bell at 2014-05-02T15:57:56Z
Files modified:
	* src/include/radiusd.h
	* src/main/listen.c
	* src/main/mainconfig.c
	* src/main/realms.c
	* src/main/xlat.c
	* src/modules/rlm_opendirectory/rlm_opendirectory.c
	* src/modules/rlm_ruby/rlm_ruby.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/332f6fa8b6f1d5ba929b62f01d19317a7224cfc9
====== 
Remove unused function eapleap_lmpwdhash()

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:56Z
Files modified:
	* src/modules/rlm_eap/types/rlm_eap_leap/eap_leap.h
	* src/modules/rlm_eap/types/rlm_eap_leap/smbdes.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a3b8616f5e0dad80cbfc7675dbad3d76ca33b2f5
====== 
Reduce scope of variables

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:56Z
Files modified:
	* src/main/command.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/9c933b1a4d13c365b7ce3d31102086899d3bec63
====== 
Reduce scope of assignment

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:55Z
Files modified:
	* src/modules/rlm_eap/libeap/fips186prf.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/824dbafbdaea466b7afc57f7614b3135c22d0bd8
====== 
Remove unnecessary "break"

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:55Z
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/62e9e4cf11a7ea4d2a8abb494fd81d4e496d353a
====== 
Fix build error

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:55Z
Files modified:
	* src/main/threads.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/719aed13ddfd6db3acf16d6bb14435a6dad3ecdb
====== 
Various scan-build fixes

Arran Cudbard-Bell at 2014-05-02T15:57:55Z
Files modified:
	* src/include/libradius.h
	* src/include/radiusd.h
	* src/main/threads.c
	* src/modules/rlm_cache/rlm_cache.c
	* src/modules/rlm_eap/types/rlm_eap_peap/peap.c
	* src/modules/rlm_eap/types/rlm_eap_ttls/ttls.c
	* src/modules/rlm_ldap/attrmap.c
	* src/modules/rlm_ldap/rlm_ldap.c
	* src/modules/rlm_linelog/rlm_linelog.c
	* src/modules/rlm_mschap/rlm_mschap.c
	* src/modules/rlm_sqlippool/rlm_sqlippool.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/d073a15037ab345bdc7469e49d2bee04c3869b26
====== 
Fix scan-build detected issue with copying garbage data to ipv4prefix attributes

Arran Cudbard-Bell at 2014-05-02T15:57:55Z
Files modified:
	* src/lib/radius.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/ef566a3081f8dd46b96d4687394df5cd01cbbce4
====== 
Setting RTLD_NOW is pretty useful too

Arran Cudbard-Bell at 2014-05-02T15:57:55Z
Files modified:
	* src/main/modules.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/62f03b6ed7abfe37ce9ac29181b97c9a6fe00482
====== 
Pull additional debugging info from master into v3.0.x

Arran Cudbard-Bell at 2014-05-02T15:57:55Z
Files modified:
	* src/main/modules.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/9885f758b02ef820c195fa306cae6aab88154925
====== 
Always inline doesn't seem to work anywhere

Arran Cudbard-Bell at 2014-05-02T15:57:55Z
Files modified:
	* src/include/log.h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/462f4b7f71df57e7fd49ac748ac7d2048a6dc1f0
====== 
Fix warning

Arran Cudbard-Bell at 2014-05-02T15:57:55Z
Files modified:
	* src/include/log.h
	* src/main/log.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/be905286a07e20291449462bc4e84a8e35978423
====== 
Accept GCC's apology

Arran Cudbard-Bell at 2014-05-02T15:57:55Z
Files modified:
	* src/include/log.h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f4555b772180cf578ec5e4472d44fccb60a698b1
====== 
Move more CC_HINTs to headers

Arran Cudbard-Bell at 2014-05-02T15:57:54Z
Files modified:
	* src/include/radiusd.h
	* src/main/evaluate.c
	* src/modules/rlm_attr_filter/rlm_attr_filter.c
	* src/modules/rlm_eap/eap.c
	* src/modules/rlm_eap/libeap/eap_tls.c
	* src/modules/rlm_eap/libeap/eap_tls.h
	* src/modules/rlm_eap/mem.c
	* src/modules/rlm_eap/rlm_eap.h
	* src/modules/rlm_eap/types/rlm_eap_peap/eap_peap.h
	* src/modules/rlm_eap/types/rlm_eap_peap/peap.c
	* src/modules/rlm_eap/types/rlm_eap_ttls/eap_ttls.h
	* src/modules/rlm_eap/types/rlm_eap_ttls/ttls.c
	* src/modules/rlm_rest/rest.h
	* src/modules/rlm_securid/mem.c
	* src/modules/rlm_securid/rlm_securid.h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/fd9a2fb583f407592760319a2b957d5b987c6486
====== 
Move __attribute__ markup from functions to headers to quiet doxygen

Arran Cudbard-Bell at 2014-05-02T15:57:54Z
Files modified:
	* src/include/radiusd.h
	* src/include/soh.h
	* src/main/log.c
	* src/main/soh.c
	* src/main/valuepair.c
	* src/modules/rlm_rest/rest.c
	* src/modules/rlm_rest/rest.h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6c1f302bcce3f1101d5921579707d7915883cc10
====== 
Remove asserts which were used to mask scan-build errors. Fix RDEBUG macros so they assume request != NULL so scan-build does not use the check as a hint that request can sometimes not be NULL

Arran Cudbard-Bell at 2014-05-02T15:57:54Z
Files modified:
	* src/include/log.h
	* src/main/auth.c
	* src/main/evaluate.c
	* src/main/log.c
	* src/main/modcall.c
	* src/main/soh.c
	* src/main/tls_listen.c
	* src/main/valuepair.c
	* src/main/xlat.c
	* src/modules/rlm_attr_filter/rlm_attr_filter.c
	* src/modules/rlm_cache/rlm_cache.c
	* src/modules/rlm_detail/rlm_detail.c
	* src/modules/rlm_eap/eap.c
	* src/modules/rlm_eap/libeap/eap_tls.c
	* src/modules/rlm_eap/mem.c
	* src/modules/rlm_eap/types/rlm_eap_gtc/rlm_eap_gtc.c
	* src/modules/rlm_eap/types/rlm_eap_leap/rlm_eap_leap.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_tls/rlm_eap_tls.c
	* src/modules/rlm_eap/types/rlm_eap_ttls/ttls.c
	* src/modules/rlm_mschap/rlm_mschap.c
	* src/modules/rlm_pap/rlm_pap.c
	* src/modules/rlm_rest/rest.c
	* src/modules/rlm_securid/mem.c
	* src/modules/rlm_sql/rlm_sql.c
	* src/modules/rlm_sqlcounter/rlm_sqlcounter.c
	* src/modules/rlm_sqlippool/rlm_sqlippool.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/7c1982e9e1cc2b52afbe4b26d0c4d9de63508c0f
====== 
Add CC_HINT(nonnull) to module functions which were missed

Arran Cudbard-Bell at 2014-05-02T15:57:54Z
Files modified:
	* src/modules/rlm_always/rlm_always.c
	* src/modules/rlm_cache/rlm_cache.c
	* src/modules/rlm_counter/rlm_counter.c
	* src/modules/rlm_exec/rlm_exec.c
	* src/modules/rlm_files/rlm_files.c
	* src/modules/rlm_linelog/rlm_linelog.c
	* src/modules/rlm_passwd/rlm_passwd.c
	* src/modules/rlm_realm/rlm_realm.c
	* src/modules/rlm_sometimes/rlm_sometimes.c
	* src/modules/rlm_utf8/rlm_utf8.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/11257b465e0215d40eb40755ec770653111bf69e
====== 
Wrap all access to gdbm with mutex.  Closes CID #1206504

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:54Z
Files modified:
	* src/modules/rlm_counter/rlm_counter.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2b80e68f2147c01e43e4f21a471c053ca04f222a
====== 
Doxygen

Arran Cudbard-Bell at 2014-05-02T15:57:54Z
Files modified:
	* src/modules/rlm_ldap/rlm_ldap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/d4b6b183d24e849c8f38345d30b2763c2a0b4aba
====== 
Add 'accept' policy

Arran Cudbard-Bell at 2014-05-02T15:57:54Z
Files modified:
	* raddb/policy.d/control

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/184bca5d3f322a0fdbb475e0c4b9a13b1a9af312
====== 
Fix overly complicated runtest.sh

Arran Cudbard-Bell at 2014-05-02T15:57:54Z
Files modified:
	* src/tests/runtests.sh

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/603c5314064c0725b8f74b48832d32e59632ea54
====== 
Switch to using new logging macros, and make logging more verbose by default

Arran Cudbard-Bell at 2014-05-02T15:57:54Z
Files modified:
	* src/include/radclient.h
	* src/main/radclient.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/1d4a628c7ef49fe928f9a3a5b1c7c1cdb91d3214
====== 
Make sending/receiving debug messages consistent, and use a slightly compacted form

Arran Cudbard-Bell at 2014-05-02T15:57:53Z
Files modified:
	* src/lib/radius.c
	* src/lib/tcp.c
	* src/modules/proto_dhcp/dhcp.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/29461478ea4e140c2831a4f7f6437c622756e12a
====== 
Simplify rad_mkdir().  Fixes CID #720459

Get rid of stat(), and ensure that we always chmod() the
directory.

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:53Z
Files modified:
	* src/main/util.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/771e30ae5eeb41a7d252fcadab19b5af5d6c8002
====== 
If there's no Acct-Terminate-Cause, it's NAS-Reboot

For the on/off query

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:53Z
Files modified:
	* raddb/mods-config/sql/main/mssql/queries.conf
	* raddb/mods-config/sql/main/mysql/queries.conf
	* raddb/mods-config/sql/main/oracle/queries.conf
	* raddb/mods-config/sql/main/postgresql/queries.conf

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/0b737f3f9b4fc435b1e7327a2070f722e9bb1c1c
====== 
Add CC_HINT(nonnull) to state machine definitions

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:53Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6af3f9ab2e1308296fa444225ff8e71e7e350284
====== 
Add CC_HINT(nonnull) to module methods

perl -p -i -e 's/static rlm_rcode_t mod_/static rlm_rcode_t CC_HINT(nonnull) mod_/' $(find src/modules/ -name "*.[ch]" -print)

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:53Z
Files modified:
	* src/modules/rlm_always/rlm_always.c
	* src/modules/rlm_attr_filter/rlm_attr_filter.c
	* src/modules/rlm_chap/rlm_chap.c
	* src/modules/rlm_counter/rlm_counter.c
	* src/modules/rlm_cram/rlm_cram.c
	* src/modules/rlm_detail/rlm_detail.c
	* src/modules/rlm_digest/rlm_digest.c
	* src/modules/rlm_dynamic_clients/rlm_dynamic_clients.c
	* src/modules/rlm_eap/rlm_eap.c
	* src/modules/rlm_example/rlm_example.c
	* src/modules/rlm_exec/rlm_exec.c
	* src/modules/rlm_expiration/rlm_expiration.c
	* src/modules/rlm_files/rlm_files.c
	* src/modules/rlm_ippool/rlm_ippool.c
	* src/modules/rlm_krb5/rlm_krb5.c
	* src/modules/rlm_ldap/rlm_ldap.c
	* src/modules/rlm_logintime/rlm_logintime.c
	* src/modules/rlm_mschap/rlm_mschap.c
	* src/modules/rlm_opendirectory/rlm_opendirectory.c
	* src/modules/rlm_otp/rlm_otp.c
	* src/modules/rlm_pam/rlm_pam.c
	* src/modules/rlm_pap/rlm_pap.c
	* src/modules/rlm_perl/rlm_perl.c
	* src/modules/rlm_preprocess/rlm_preprocess.c
	* src/modules/rlm_python/rlm_python.c
	* src/modules/rlm_radutmp/rlm_radutmp.c
	* src/modules/rlm_realm/rlm_realm.c
	* src/modules/rlm_rediswho/rlm_rediswho.c
	* src/modules/rlm_replicate/rlm_replicate.c
	* src/modules/rlm_rest/rlm_rest.c
	* src/modules/rlm_ruby/rlm_ruby.c
	* src/modules/rlm_securid/rlm_securid.c
	* src/modules/rlm_smsotp/rlm_smsotp.c
	* src/modules/rlm_soh/rlm_soh.c
	* src/modules/rlm_sometimes/rlm_sometimes.c
	* src/modules/rlm_sql/rlm_sql.c
	* src/modules/rlm_sqlcounter/rlm_sqlcounter.c
	* src/modules/rlm_sqlhpwippool/rlm_sqlhpwippool.c
	* src/modules/rlm_sqlippool/rlm_sqlippool.c
	* src/modules/rlm_unix/rlm_unix.c
	* src/modules/rlm_wimax/rlm_wimax.c
	* src/modules/rlm_yubikey/rlm_yubikey.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/3705ba401bb8188ebc60e2aabc8521a6cf0b35d6
====== 
Hacks for later

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:53Z
Files modified:
	* src/modules/rlm_eap/radeapclient.mk

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c42fdfbb6e3bc26d67c797d0eb859965945dc99b
====== 
Break out radclient structs into their own file

Arran Cudbard-Bell at 2014-05-02T15:57:53Z
Files modified:
	* src/include/radclient.h
	* src/main/radclient.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f8f2c79e9dd250094ba3796dbd84dbbf009b12d5
====== 
Swap out all __attribute__ markers with CC_HINT() this automatically removes them if the CC is not GCClike

Arran Cudbard-Bell at 2014-05-02T15:57:53Z
Files modified:
	* src/include/all.mk
	* src/include/build.h
	* src/include/conffile.h
	* src/include/libradius.h
	* src/include/log.h
	* src/include/missing-h
	* src/include/radiusd.h
	* src/lib/debug.c
	* src/main/command.c
	* src/main/log.c
	* src/modules/rlm_eap/types/rlm_eap_pwd/eap_pwd.h
	* src/modules/rlm_sqlhpwippool/rlm_sqlhpwippool.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/cc255ae3251a97a00bbab35f6506b8c4a22bb726
====== 
Fix error messages complaining about the size of attributes in EAP-SIM Closes #617

Arran Cudbard-Bell at 2014-05-02T15:57:53Z
Files modified:
	* src/modules/rlm_eap/libeap/eap_sim.h
	* src/modules/rlm_eap/libeap/eapcrypto.c
	* src/modules/rlm_eap/types/rlm_eap_sim/rlm_eap_sim.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/4574435116d192a6f9cddd46fd0f6a5109cbee5a
====== 
Output compiler version so we know when Travis upgrade it

Arran Cudbard-Bell at 2014-05-02T15:57:52Z
Files modified:
	* .travis.yml

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/bdf909417d9f46a7c62c2a688620521c0fa832a0
====== 
Out isn't a char array

Arran Cudbard-Bell at 2014-05-02T15:57:52Z
Files modified:
	* src/main/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/1f0989e787bf770c2dee50094583ab0c6e1030ea
====== 
Just use the correct format specifier instead of doing a cast

Arran Cudbard-Bell at 2014-05-02T15:57:52Z
Files modified:
	* src/modules/rlm_eap/types/rlm_eap_mschapv2/rlm_eap_mschapv2.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f7ee085f00a033a452186448b73d9e02a25abd15
====== 
clang scan is dumb...

Arran Cudbard-Bell at 2014-05-02T15:57:52Z
Files modified:
	* src/modules/rlm_sql/rlm_sql.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/3048ee9c9d8a5541dcf8e65f211d023f13df1b44
====== 
Pointers should be initialised to NULL not '\0'

Arran Cudbard-Bell at 2014-05-02T15:57:52Z
Files modified:
	* src/lib/misc.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c1602dfe7b48e6d0536902b491e4843ae0f3c6d6
====== 
Only write out what file were writing to if we have an output handle (radsniff)

Arran Cudbard-Bell at 2014-05-02T15:57:52Z
Files modified:
	* src/main/radsniff.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/000f1c3bdf21e6f6f18b555c341d848ceff7f3bb
====== 
Another fix for clang-scan

Arran Cudbard-Bell at 2014-05-02T15:57:52Z
Files modified:
	* src/modules/rlm_sql/rlm_sql.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/332b306dad96448a697cfb20a1f230205895674d
====== 
Link to -lssl only if we're using ssl

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:52Z
Files modified:
	* src/modules/rlm_eap/radeapclient.mk

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/b457515d3020eed1bb70726e68d15c6034828d2d
====== 
type with %d / size_t

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:52Z
Files modified:
	* src/modules/rlm_eap/types/rlm_eap_mschapv2/rlm_eap_mschapv2.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/4767c195f8d480bebf81b21410fa3ca038d8a538
====== 
Limit size of EAP-MSCHAPv2 data.  Closes CID #1206504

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:52Z
Files modified:
	* src/modules/rlm_eap/types/rlm_eap_mschapv2/rlm_eap_mschapv2.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/36e6cf1cdc0c96395fc1df8c42d500b64dad18c8
====== 
pcap_init may return NULL

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:52Z
Files modified:
	* src/main/radsniff.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/5a1ff0a2dc18ff492dc549d76507cfedfe28ce80
====== 
assertion for clang analyzer

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:51Z
Files modified:
	* src/modules/rlm_eap/types/rlm_eap_peap/peap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/a35e1111ca56d4007323c5a2a5806bce2ed99d50
====== 
Warnings for clang analyzer

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:51Z
Files modified:
	* src/lib/udpfromto.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/66bd66e37e28e7c55db0047e3d3cdde3be143d03
====== 
Fix assertion

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:51Z
Files modified:
	* src/main/modcall.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/445f14045ec2503205c6486435648dc2d71180f9
====== 
Pre-compile Auth-Type and *-Type sections, too

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:51Z
Files modified:
	* src/main/modules.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/9acf14e65edb914bdbb1a97614e8c14e5bf15e89
====== 
Clear libfreeradius buffer before starting the event loop, so we don't get weird messages in debug mode

Arran Cudbard-Bell at 2014-05-02T15:57:51Z
Files modified:
	* src/main/radiusd.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6d49e6c17e841fe767567475d2429879c6725f49
====== 
Simplify cleanup_delay slightly

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:51Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/8f022383750a644ae5a243b3b8c06356233e7447
====== 
Assertions to quiet clang scan

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:51Z
Files modified:
	* src/modules/rlm_pap/rlm_pap.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/79f2df656a9dec4ee70ee6713ddf8b50e37e7d8b
====== 
Remove unused assignment

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:51Z
Files modified:
	* src/modules/rlm_securid/rlm_securid.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/19a6f2557cef84f65ef92202985fbe87c018d9f7
====== 
Don't use variables outside of their scope

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:51Z
Files modified:
	* src/modules/rlm_redis/rlm_redis.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/30cfb54b8d0aa5241c863ed8d44367b0e40ec031
====== 
Assertions for clang scan

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:51Z
Files modified:
	* src/main/tls_listen.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/4d6f58062a53c34e8a83dc44e37ab223da87cbd9
====== 
filter may be NULL

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:51Z
Files modified:
	* src/lib/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/9e968aab94b4a52e7306ff7bf380057a477fa0b5
====== 
Remove unnecessary check

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:51Z
Files modified:
	* src/lib/filters.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/4fff9610530bf467b2acef64f68514c038e169a3
====== 
Assign result of talloc_steal

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:50Z
Files modified:
	* src/lib/cbuff.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/39854a5979acaca992bc69a4fa548d6cf97425f3
====== 
Ensure that variable is initialized

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:50Z
Files modified:
	* src/modules/rlm_rest/rest.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/80343e27f6f56cee3a7142b52c0d6020376ba304
====== 
Remove unused assignment

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:50Z
Files modified:
	* src/modules/rlm_realm/rlm_realm.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/9cd46f0aab735e6bdd3dda61ed1155e1342d24c7
====== 
Assertion to quiet clang scan

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:50Z
Files modified:
	* src/main/soh.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/43c90ed6317d252b058e97643617add6bd5e9441
====== 
remove unused assignment

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:50Z
Files modified:
	* src/main/xlat.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2db26b1651ec787ea9902840706fba40138b594c
====== 
Unused return value

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:50Z
Files modified:
	* src/modules/proto_dhcp/dhcp.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/425d57b3ac5cc5a12f3f1e09788ef98d2470a74d
====== 
Use consistent names for vp_aprint, vp_aprint_value and vp_aprint_type

Arran Cudbard-Bell at 2014-05-02T15:57:50Z
Files modified:
	* src/include/libradius.h
	* src/lib/print.c
	* src/lib/valuepair.c
	* src/main/evaluate.c
	* src/main/map.c
	* src/main/xlat.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c7087edb954bb0be94d0a84c6b7562a532725a2e
====== 
Use next_by_da when we have the da

Alan T. DeKok (via Arran Cudbard-Bell)@2014-05-02T15:57:50Z
Files modified:
	* src/main/modcall.c
	* src/main/xlat.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c35bf461ccd3112e38beac332b3ec574516ca65d
====== 
Merge pull request #616 from spaetow/patch-1

Update operator-name

Arran Cudbard-Bell at 2014-05-02T15:57:50Z
Files modified:
	* raddb/policy.d/operator-name

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/84dc7d7d75ab8a85cd26cf44b587659cb928101c
====== 
Warn users when were screwing with the values they set

Arran Cudbard-Bell at 2014-05-02T15:57:50Z
Files modified:
	* src/include/conffile.h
	* src/main/mainconfig.c
	* src/main/realms.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6ab2decd44cfb6db106eb85ce01a6741dc2a6214
====== 
match could be uninitialised

Arran Cudbard-Bell at 2014-05-02T15:57:49Z
Files modified:
	* src/lib/valuepair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c0cce6568d590533b1842151dce4890166ecabbe
====== 
Add access challenge as a response type in radsniff

Arran Cudbard-Bell at 2014-05-02T15:57:49Z
Files modified:
	* src/main/radsniff.c

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