Commit report for v3.0.x branch

The git bot announce at freeradius.org
Sun Oct 19 00:00:02 CEST 2014


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

======
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
====== 
-- 
This commit summary was generated @2014-10-19T00:00:02Z by lgfeed version 0.00 (https://github.com/arr2036/lgfeed).


More information about the Freeradius-Devel mailing list