<br><br><div class="gmail_quote">On Sun, Mar 10, 2013 at 3:22 PM, Arran Cudbard-Bell <span dir="ltr"><<a href="mailto:a.cudbardb@freeradius.org" target="_blank">a.cudbardb@freeradius.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
On 10 Mar 2013, at 00:13, Avesh Agarwal <<a href="mailto:avesh.ncsu@gmail.com">avesh.ncsu@gmail.com</a>> wrote:<br>
<br>
><br>
><br>
> On Sat, Mar 9, 2013 at 8:50 AM, Arran Cudbard-Bell <<a href="mailto:a.cudbardb@freeradius.org">a.cudbardb@freeradius.org</a>> wrote:<br>
> I've refactored the code and shrunk it.<br>
><br>
> I really appreciate your help on this.<br>
> The code you submitted leaked opaque data, which i've fixed.<br>
><br>
> 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?<br>

><br>
> ------<br>
><br>
> Regarding the library.<br>
><br>
> I actually packaged it for fedora listed here <a href="http://koji.fedoraproject.org/koji/buildinfo?buildID=383324" target="_blank">http://koji.fedoraproject.org/koji/buildinfo?buildID=383324</a> .<br>
><br>
> 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.<br>

><br>
<br>
</div>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.<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br><br>Appreciated, merged the patch. <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">
-Arran<br>
-<br>
List info/subscribe/unsubscribe? See <a href="http://www.freeradius.org/list/devel.html" target="_blank">http://www.freeradius.org/list/devel.html</a><br>
</div></div></blockquote></div><br>