Patch for updated EAP-TNC support for freeradius from tnc at fhh

Arran Cudbard-Bell a.cudbardb at freeradius.org
Sun Mar 10 20:22:13 CET 2013


On 10 Mar 2013, at 00:13, Avesh Agarwal <avesh.ncsu at gmail.com> wrote:

> 
> 
> On Sat, Mar 9, 2013 at 8:50 AM, Arran Cudbard-Bell <a.cudbardb at freeradius.org> wrote:
> I've refactored the code and shrunk it.
> 
> I really appreciate your help on this. 
> The code you submitted leaked opaque data, which i've fixed.
> 
> I'm concerned about the call to storeUserName, and getConnection, there's no where where the connection is explicitly destroyed, so I don't see how memory passed to those functions or allocated by those functions is ever freed. I guess there must be some magic GC/reference counting i'm missing?
> 
> ------
> 
> Regarding the library.
> 
> I actually packaged it for fedora listed here http://koji.fedoraproject.org/koji/buildinfo?buildID=383324 .
> 
> So I use the that packaged version when building EAP-TNC module for freeradius, and somehow I did not have any of these issues while building the module. I will surely look for and try to eliminate  the errors you noticed.  
>   

Ok i've sent a pull request. It adds the missing dependencies properly, fixes the build system to work on OSX, and fixes the typos.

-Arran


More information about the Freeradius-Devel mailing list