Commit report for v3.0.x branch

The git bot announce at freeradius.org
Sat Jul 5 00:00:02 CEST 2014


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

======
Merge pull request #725 from nchaigne/v3.0.x

radeapclient - fix send_packet

Arran Cudbard-Bell at 2014-07-04T17:08:19Z
Files modified:
	* src/modules/rlm_eap/radeapclient.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/f3100c303c7cf6e5f50984f126eb93238ce27197
====== 
radeapclient - fix send_packet

I spotted two errors in radeapclient.c, introduced in the following
previous commits.
With this fix, radeapclient is now useable again.

1)
Commit: c8a062a112f17a5810d311dc0e0acfe963b2d440
(2014/06/13)

- send_packet(rep, &req);
-
- if (!req) return -1;
+ send_packet(req, &rep);
+ if (!rep) {

Arguments got reversed. Hence segmentation fault later when doing:

for (vp = req->vps; vp != NULL; vp = vpnext) {

Bit of caution on the wording used in radeapclient:
"req" is the request coming FROM the server. (because this is not a
"request" in RADIUS sense, but an EAP-Request within the EAP-SIM
transaction.)
"rep" is the response from the client TO the server (EAP-Response).

2)
Commit: bc3676835c3dcc220ab518d4c3c35962bc0f8be2
(2014/05/02)

In "send_packet":

+ if (!req || !rep || !*rep) return -1;

*rep == NULL is the expected behaviour...

Nicolas C at 2014-07-04T17:03:33Z
Files modified:
	* src/modules/rlm_eap/radeapclient.c

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/53c1ce349b07a6e51dcc0cde9041e0a5153bdf11
====== 
In redhat spec file, update dependency on json-c to version 0.10 as 0.11 only exists for fedora and 0.10 builds ok

Philippe Wooding (via Alan T. DeKok)@2014-07-04T16:01:54Z
Files modified:
	* redhat/freeradius.spec

Commit diff:
https://github.com/FreeRADIUS/freeradius-server/commit/c0e3a5330804c0873621ca1789255df53dde8012
====== 
Update redhat spec file to reflect the fact rlm_host is no longer experimental.

Philippe Wooding (via Alan T. DeKok)@2014-07-04T16:01:34Z
Files modified:
	* redhat/freeradius.spec

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


More information about the Freeradius-Devel mailing list