On Tue, 6 Sep 2016, Paul Jakma wrote:
I have another patch that makes valgrind clean for me on the bncs, but I left the bncs in place while the peer is defined. See patch posted just there.
Ok, my variation on your fix (hope you don't mind me re-doing that - I noted what the leak was from your patch, but started again to try work out the refcounting) + your other leak fix + your AFI_ETHER patch (which I havn't tested) + my OPEN/FSM fixes (CI with previous hit some collision handling problem in bringing up sessions):
- valgrinds clean from start to finish with some simple BGP sessions for me. - passes Martin's torture testing Which is on top of the other stuff from r8 and NHT, etc. regards, -- Paul Jakma | [email protected] | @pjakma | Key ID: 0xD86BF79464A2FF6A Fortune: A bird in the hand is worth what it will bring. _______________________________________________ Quagga-dev mailing list [email protected] https://lists.quagga.net/mailman/listinfo/quagga-dev
