We build an array to handle the multipaths, when a 0 is specified it builds an array of size 0 :) Hence the problem. This does need to be fixed.
donald On Tue, Nov 17, 2015 at 9:55 AM, Martin Winter < [email protected]> wrote: > Donald, > > I’m working on the next issue right now and leave you with the crash. > I assumed you might have a faster idea where to find the problem (and fix) > > Currently looking at BGP IPv6 issues. It seems the “--enable-multipath=0” > got broken (again). “--enable-multipath=64” seems to work, but with > a value of 0 (= unlimited), something breaks in getting IPv6 routes > installed. > > - Martin > > On 17 Nov 2015, at 6:51, Donald Sharp wrote: > > > Martin - > > > > Before you narrow down a specific patch that might be causing the > issue... > > > > I've been contemplating working through our entire patch set and exposing > > it through the take-X branches. I can't imagine we haven't already fixed > > the crash you are seeing( and I'm looking for that by hand now ). > > > > In addition I think you've provided enough of a clue for me to narrow > down > > the problem to the actual bit of code and to find our fix. > > > > donald > > > > On Tue, Nov 17, 2015 at 9:43 AM, Martin Winter < > > [email protected]> wrote: > > > >> Han, > >> > >> On 17 Nov 2015, at 4:27, Han Coumans wrote: > >> > >>> Dear Martin, Donald, Quagga devlist, > >>> > >>> > >>> From CI's RIPng RFC test we learn that even without a new patch (from > the > >>> Cumulus Mega patch set 'Take 2') for RIPng it's possible to have 10 new > >>> failures, making a total of 17 (one is an 'unpredict'). > >> > >> Keep in mind to look at the differences. The cumulus mege set is based > on > >> top of the last round (accepted 4) - so the main thing is to compare to > >> this one. > >> > >> And to give kudos to Cumulus: There are actually quite a few past > failures > >> which show up as fixed. > >> > >> For the failed ones, my experience is that a single issue (i.e. in > zebra) > >> might impact many tests. I’m still working through the failures and will > >> post whenever I find a specific one (and can explain the exact issue or > >> commit or how to reproduce). But it will take some time with such a > large > >> set. > >> > >> Regards, > >> > >> - Martin Winter > >> > >> > >>> That will be an interesting fix. > >>> > >>> > >>> Thanks for your CI, Martin. > >>> > >>> > >>> Regards, > >>> Han Coumans > >> >
_______________________________________________ Quagga-dev mailing list [email protected] https://lists.quagga.net/mailman/listinfo/quagga-dev
