On Fri, 4 Sep 2015 09:12:09 +0100 (BST) Paul Jakma <[email protected]> wrote:
> I've updated the below head. Commits that were in and not commented > on stripped from the quoted email. > > On Fri, 21 Aug 2015, Paul Jakma wrote: > > > I've gone through some more of the patchwork and bugzilla backlog. > > > > http://git.savannah.gnu.org/cgit/quagga.git/log/?h=volatile/patch-tracking/2/proposed/ff > > > rare bgp crash > > http://patchwork.quagga.net/patch/1278/ > > http://patchwork.quagga.net/patch/1279/ > > http://patchwork.quagga.net/patch/1281/ > > https://lists.quagga.net/pipermail/quagga-dev/2015-June/012875.html > > 1279 seems to be the agreed fix? > > What's the correct commit? > > Be interested to hear from Vivek and Timo on this one. The one I sent in 2015-June is not correct. The null check needs to be done in the end, or announce using e.g. "ip network" do not work. I am still using locally 1281, and fixes the problem for me. IIRC, just checking for Established like in 1279 is not enough, but cannot say for sure as I've forgotten the details (maybe I just never had the time to test it). 1279 does have other fixes too, so if you want to be safe applying both 1279 and 1281 would probably make sense. /Timo _______________________________________________ Quagga-dev mailing list [email protected] https://lists.quagga.net/mailman/listinfo/quagga-dev
