[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] Application using libvirt crashes when having concurrent TLS connections (gnutls problem)



On 10/29/2009 10:39 AM, Thomas Treutner wrote:
On Thursday 29 October 2009 01:30:34 Bryan Kearney wrote:
Ok.. patch is attached. You can see a test build at:

http://bkearney.fedorapeople.org/libvirt-0.3.1-test.jar

Let me know if this works for you. If so, I will push a new build.

Yes, works for me, thank you!

A minor improvement: finalize() should have an @override annotation, just to
be sure it actually overrides a method. In case it doesn't (due to f.e. wrong
return type) a warning (or even error?) is thrown at compile time, avoiding
hard to find bugs.

A more important issue: In org.libvirt.Domain.free(), the VDP is not checked
for being NULL, and return type is void, hardening debugging. I did a copy
and paste from your new Connect.close() methods and s/VCP/VDP/, works fine
for me.

kr,
tom
Thanks.. i will make those changes and push a 0.3.1 release. I will send out email when done.. if you would be willing to vote up the karma that would be great.

-- bk


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]