Age | Commit message (Collapse) | Author | |
---|---|---|---|
2009-03-31 | Fixed memory leaks which would occur if the second of two memory | Tobias Stoeckmann | |
allocations fails. looks right deraadt, krw ok henning | |||
2009-03-26 | argh, do not reuse the global trans_as flag to be applied to the peer | Henning Brauer | |
specific or we had to widen the peer specific flags without need. defien PERRFLAG_TRANS_AS instead and use that | |||
2009-03-22 | make transparent-as yes|no settable peer neighbor with the global setting | Henning Brauer | |
acting as default. per-neighbor requested by arnold nipper @ decix, ok claudio | |||
2009-03-19 | First big chunk of mrt rewrite. Simplifies code hopefully and fixes table | Claudio Jeker | |
dumps that were wrong because of the 4byte AS support. Dumps I took seemed to work so far. "Put it in" henning@ | |||
2009-03-19 | Implement a attr_writebuf() function that works on a struct buf instead of | Claudio Jeker | |
a pre allocated piece of memory. Will be used by newer mrt code. OK henning@ | |||
2009-03-18 | Correctly check the peer group for mrt dumps. Without this all non-grouped | Claudio Jeker | |
peers ended up in most mrt dumps. Also switch the if-statements to a hopefully easier to read version. Another mrt fix from my flight home. OK henning@ | |||
2009-03-18 | Correctly check for MRT_TABLE_DUMP_MP and don't send the config to the SE. | Claudio Jeker | |
Make mrt table-mp dumps actually work. Found on the plane from NRT to ZRH. OK henning@ | |||
2009-03-18 | Introduce local_as in the peer config. This allows per peer local AS and | Claudio Jeker | |
simplifies some code because it is possible to get the local AS from the peer struct. Local AS needs more patching to work correctly though. OK henning@ | |||
2009-03-18 | short_as is always initialised to the 16bit AS that is used. So no need | Claudio Jeker | |
use conf->as as source for > 16bit AS numbers. OK henning. | |||
2009-03-13 | No need for a line break, makes code a bit more compact. | Claudio Jeker | |
2009-03-13 | Next step in mrt cleanup. Switch buf_write to a smarter version of itself. | Claudio Jeker | |
Doing more the stuff msgbuf_write does and switch the only user -- mrt -- over to use this simpler way of writing out stuff. | |||
2009-03-13 | Move mrt code over to use the msgbuf way of queuing buffers instead of rolling | Claudio Jeker | |
an own version. More cleanup to come. | |||
2009-03-13 | Sync buffer API with the one from ospfd. The dynamic buffers will make other | Claudio Jeker | |
code much easier to write. OK henning@ some time ago. | |||
2009-03-13 | More paranoia when parsing empty updates. Make sure they are really empty and | Claudio Jeker | |
don't come with some additional stuff attached to them. OK henning@ | |||
2009-03-13 | Correctly encode MP unreach NLRI so that IPv6 prefixes get removed correctly. | Claudio Jeker | |
One bug was hiding another bug and only foundry based routers where unhappy about these bad updates. Found by Arnoud Vermeer and Elisa Jasinska at ams-ix. OK henning@ | |||
2009-02-25 | add a stupid workaround for a race somewhere in the crypto code in the | Henning Brauer | |
kernel. a one second sleep has made it go away in all my tests. ok theo | |||
2009-02-19 | Use a bob sized header for IPv6 not the default pedro sized one. | Claudio Jeker | |
Seems like nobody ever tried to dump IPv6 tables. Found while reworking this code. | |||
2009-02-17 | Fix aspath_prepend() in the case a AS-PATH has a sequence of 255 elements | Claudio Jeker | |
and we try to prepend. The result was a corrupt AS-PATH and a RDE fatal later on when some other operation was run on that path. Found the hard way by henning@ and sthen@. OK henning@, sthen@ | |||
2009-02-16 | typo in error message; "bigger that" -> "bigger than" | Stuart Henderson | |
2009-02-11 | Because IPv6 fucked up and created a badly sized struct sockaddr_in6 we need | Claudio Jeker | |
to add some pading on 64bit archs. Use a helper struct plus some ROUNDUP() magic to get the size correct. Tested on i386 and amd64 by Graeme Lee "graeme (at) omni (dot) net (dot) au" Some sort of OK by henning@ (he ignores the non-v4 code) | |||
2009-02-09 | Correct function name in warning to avoid confusion. After a discussion on | Claudio Jeker | |
misc@ | |||
2009-01-26 | Getting the higest IP only works if the compare is done in host byte order. | Claudio Jeker | |
Same bug as in ospfd and ospf6d. *gulp* | |||
2009-01-21 | timer_nextduein returned crap. fix it. so obvious... how could that happen. | Henning Brauer | |
found out the hard way by Marc Runkel <MRunkel@untangle.com> | |||
2009-01-13 | Reference RFC 4893 instead of draft-ietf-idr-as4bytes-13. | Stuart Henderson | |
ok claudio | |||
2009-01-13 | Replace NEW_ASPATH/NEW_AGGREGATOR with the naming from RFC4893, | Stuart Henderson | |
AS4_PATH/AS4_AGGREGATOR. No binary change. ok claudio | |||
2009-01-01 | Remove unneeded debug messages that made it into the tree. | Claudio Jeker | |
2008-12-28 | Add a ugly workaround for the problem where an invalid AS4_PATH is passed | Claudio Jeker | |
over mulitple hops and causes bgpd to close the connection. This is what the RFC requires us to do but the result is a DoS against all OpenBGPD routers when somebody injects such a bad optional transitive attribute because the intermediate routers don't give a damn about it. As a result we now ignore such bad prefixes and don't allow them in the decision process. The handling of optional transitive attributes needs to be rethinked because all of them can be abused in such a way. Idea OK by a few + henning@, tested myself against my crappy regress test suite that needs way more work. | |||
2008-12-19 | document the default localpref, from discussion with | Henning Brauer | |
Mick Hohmann <mick at IM dot net> | |||
2008-12-12 | Make bgpd route priority aware and while there sync in the basics to correctly | Claudio Jeker | |
track multipath routes. Code is mostly stolen from similar fixes to ospfd but a bit different as this kroute.c implementation is more complex because of additional task (nexthop verification for example). This change and the similar ospfd change were desperatly needed on larger complex network setups which tended to end up in situation where the kernel had a different oppinion about routing than the daemon. Now the situation should be a lot better. Tested and OK sthen@ gollo@ | |||
2008-12-12 | Correctly set the partial bit on the AS4_PATH and AS4_AGGREGATOR attribute. | Claudio Jeker | |
Per RFC4271 the partial bit must be set if the attribute was not generated on the originating router. This is mostly cosmetics. OK sthen@ | |||
2008-12-12 | Follow RFC 4271 more closely by sending the correct error back for optional | Claudio Jeker | |
attributes and use the correct lenght in some other error cases. OK sthen@ | |||
2008-11-21 | Track nexthops when the underlying route is changing. Until now true nexthops | Claudio Jeker | |
were only resolved when they were added. This calls for troubles if something like ospfd starts to change the underlying routes. Tested by gollo@, OK henning@ | |||
2008-11-06 | Mixed up characters found by Gleydson Soares. | Claudio Jeker | |
2008-11-01 | Use correct function names in fatal() messages. | Claudio Jeker | |
2008-10-17 | bring in the findeol() fix from pfctl. list of affected parsers by sthen | Henning Brauer | |
2008-10-03 | Unify code between the various flavors of imsg buffer.c. | Eric Faurot | |
Use unsigned int for msg_iovlen. ok henning@ claudio@ | |||
2008-10-02 | correct path for isakmpd/local.pub; from Matthew Dempsky | Jason McIntyre | |
2008-09-29 | Correct the logic when matching prefixes. This solves issues where IPv6 | Claudio Jeker | |
prefixes where matched on a simple deny from any prefix 0.0.0.0/0 rule. OK henning@ | |||
2008-09-11 | remove hackery to un-demote after the session has been established for | Henning Brauer | |
a while, we can now just use a timer (since i rewrote the timer code) | |||
2008-09-10 | Properly free allocated memory if carp_demote_get encounters an error. | Tobias Stoeckmann | |
ok henning@ | |||
2008-09-05 | ignore unknown SAFIs in the MP capabilities. used by some to distribute | Henning Brauer | |
some kind of VPN stuff | |||
2008-07-08 | Use correct format string specifier for int argument. Don't try to print it | Claudio Jeker | |
as string. Fixes crash seen by Peter Bristow. "obviously ok" henning@ | |||
2008-06-26 | fatalx instead of fatal because errno is not set of pipe is closed. OK henning | Claudio Jeker | |
2008-06-15 | Allow bgpd to delete more then one community per filter rule. Fixes PR5807 | Claudio Jeker | |
tested by Raphael Ho long time ago. | |||
2008-06-15 | Paranoia check. Make sure that the 2 len bytes are actually available. | Claudio Jeker | |
Another easy M hiding in my bgp source forest. | |||
2008-06-11 | use monotime instead of wallclock. | Henning Brauer | |
makes us independent from system time changes. diff from japan,was hiding... ok claudio some time ago | |||
2008-05-12 | Error out with usage line if additional arguments are given after the | Pierre-Yves Ritschard | |
option parsing. Found out the hard way by jdixon on ifstated. ok sobrado@, jdixon@, millert@ | |||
2008-05-11 | allow IMSG_CONTROL_SHOW_TIMER on restricted sockets | Henning Brauer | |
2008-05-09 | insert/delete routes with priority RTP_BGP | Henning Brauer | |
this is mostly cosmetical for the moment, there are bigger changes required to make full use of the route priorities, but this is the first step to do. discussed with claudio | |||
2008-05-08 | backout shit committed on accident | Henning Brauer | |