Commit report for v3.1.x branch

[The git bot]

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

======
No need for if

Confusing because the rest of the frees don't use a condition

Arran Cudbard-Bell at 2015-09-11T19:04:52Z
Files modified:
	* src/main/pair.c

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

Alan T. DeKok at 2015-09-11T18:53:26Z
Files modified:
	* src/lib/pair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/190e5c02b64ad7f6b7846f3fe8ec0678b7301373
====== 
Use fr_pair_list_mcopy... instead of fr_pair_list_move...

Manual merge of 337bbb9435b8

Alan T. DeKok at 2015-09-11T18:25:14Z
Files modified:
	* src/include/libradius.h
	* src/lib/pair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/0a8051e93c9a2d5bb0bbeeab79c21b5ff153a12e
====== 
Revert "Copy VPs instead of talloc_stealing them"

This reverts commit a529c2d9bdef0f635fa10b2ab7e05527f95551b2.

There's a better fix

Alan T. DeKok at 2015-09-11T18:25:14Z
Files modified:
	* src/main/state.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/0efb0c6900f0e7edd63d3beebc6332186c9f17a8
====== 
Don't leak client_fd on error

Arran Cudbard-Bell at 2015-09-11T18:19:13Z
Files modified:
	* src/main/command.c

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

Arran Cudbard-Bell at 2015-09-11T18:19:08Z
Files modified:
	* src/main/command.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/cb702b0dc8658dfe36cac794328ff0d906d73fb1
====== 
Copy VPs instead of talloc_stealing them

Alan T. DeKok at 2015-09-11T17:57:02Z
Files modified:
	* src/main/state.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/1315863ce19749a07725c3417c1a36b43eb27959
====== 
Check if the socket is in use before unlinking it

Alan T. DeKok at 2015-09-11T16:34:27Z
Files modified:
	* src/main/command.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/fd2c7f536363ef97e73cea58ca326c91f0ad10d6
====== 
Add __packed__ to structs which cast over packet buffers

Arran Cudbard-Bell at 2015-09-11T16:25:15Z
Files modified:
	* src/include/net.h

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2908202e9cf3f028beb69c0275eee864b9b0c8aa
====== 
Syntax errors are errors, not assertions

Alan T. DeKok at 2015-09-11T15:20:01Z
Files modified:
	* src/main/modcall.c
	* src/tests/keywords/else-error

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/2ad12b732dee980d740aac70e78cf68eca2ef66b
====== 
Prevent possible memleak in regex

There was a very small chance that preg was allocated but not freed. This is kind of a sequel to PR #1207.

Herwin Weststrate (via Arran Cudbard-Bell)@2015-09-11T12:16:49Z
Files modified:
	* src/main/pair.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/6aa6e228f93e53caf47fa1a47ef879659c5aaee2
====== 
Optionally send rejects without a delay

Currently there is only one global option to set a delay to every Access-Reject packet: reject_delay. There are use cases where you want certain rejects to have no delay, while others should have a delay. An example might be using 802.1X on Cisco LAN Devices: If a client tries MAC authentication an Access-Reject can force it to switch to 802.1X, this is a reject you want to send without any delay. On the other hand, if the client tries 802.1X with a wrong password, you still want the reject to be delayed.

By setting a value to FreeRADIUS-Response-Delay(-USec) in reply, we overwrite the global delay. The maximum supported value is 10, larger values result in a delay of 10 seconds. A value of 0 removes the delay. Not having this attribute in control results in using the global delay. If both FreeRADIUS-Response-Delay and FreeRADIUS-Response-Delay-USec are set, the second one is ignored.

Herwin Weststrate (via Alan T. DeKok)@2015-09-11T01:21:11Z
Files modified:
	* src/main/process.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/bd857346f0c935aa9c6aaccc784ace8c1b216cd8
====== 
Allow response_delay to be microseconds, too

This was already done with commit 1d1c50bb0c6f5f013b9680def4b7184ecb63f64b, but there was a second assertion that assumed the delay was at least 1 second.

Herwin Weststrate (via Alan T. DeKok)@2015-09-11T01:21:09Z
Files modified:
	* src/main/process.c

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