Commit report for master branch
New activity for FreeRADIUS (the high performance and highly configurable RADIUS server) ====== Made packet data len "size_t". Only one place needs it to be negative: the read from the socket. For the rest of the code, it is ALWAYS positive. As part of this, changed a few other variables, too. And uses "%zu" for printing "size_t" variables. C99 is everywhere. If you want to build the server on a non-C99 compiler, go away. Alan T. DeKok@2012-10-20T09:40:01Z Files modified: * src/include/libradius.h * src/lib/radius.c * src/lib/tcp.c * src/lib/vqp.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/41ae75bf88f17769e6da8... ====== Fix compiler warnings Alan T. DeKok@2012-10-20T09:26:22Z Files modified: * src/lib/radius.c Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/941631859bad00ebfe8d6... ====== Merge branch 'master' of github.com:FreeRADIUS/freeradius-server Alan T. DeKok@2012-10-20T09:05:41Z Files modified: * src/include/Makefile * src/include/all.mk * src/include/base64.h * src/include/libradius.h * src/include/radiusd.h * src/lib/Makefile * src/lib/all.mk * src/lib/base64.c * src/lib/dhcp.c * src/lib/radius.c * src/main/command.c * src/main/xlat.c * src/modules/rlm_expr/rlm_expr.c * src/modules/rlm_ldap/rlm_ldap.c * src/modules/rlm_python/prepaid.py * src/modules/rlm_python/radiusd.py * src/modules/rlm_python/rlm_python.c * src/modules/rlm_sql/rlm_sql.c * src/modules/rlm_cache/rlm_cache.c * src/modules/rlm_sql/drivers/rlm_sql_mysql/all.mk.in Commit diff: https://github.com/FreeRADIUS/freeradius-server/commit/995cfe29007eb98d52dcc... ====== -- This commit summary was generated @2012-10-20T18:00:01Z by lgfeed version 0.00 (https://github.com/arr2036/lgfeed).
Brian Candler wrote:
On Sat, Oct 20, 2012 at 06:00:01PM +0200, The git bot wrote:
Made packet data len "size_t".
Only one place needs it to be negative: the read from the socket. For the rest of the code, it is ALWAYS positive.
ssize_t ?
I changed from ssize_t to size_t. All of the code needs a size_t, except for one line. Therefore, it's better to have it as "size_t" everywhere. That one line contains a simple work-around. The rest of the code can use size_t, and comparisons with other variables also use size_t. If it was using ssize_t, there would have to be conversions all over the place. Alan DeKok.
participants (3)
-
Alan DeKok -
announceļ¼ freeradius.org -
Brian Candler