Age | Commit message (Collapse) | Author | |
---|---|---|---|
2006-05-02 | The pftable name to ID functions should use the pftable_labels list and not | Claudio Jeker | |
the rt_labels list. Luckily this bug was not visible because of the way the pftable and rtlabel code works. Found by Thomas E. Spanjaard. | |||
2006-04-26 | Remove unused _exists() function. It is not even used in my evil filter tree | Claudio Jeker | |
so I doubt I will need it anytime soon. Additionally I think that the interface is dangerous as it bypasses the ref-counting. | |||
2006-04-26 | Last argument to send_filterset() is a left-over from one of my not so clever | Claudio Jeker | |
ideas that will never be included and always set to 0. Kill it. | |||
2006-04-26 | Remove filterset_names from bgpd.h and replace it with a function because | Claudio Jeker | |
the table was already out of sync now. OK henning@ | |||
2006-04-25 | Mark fatal and fatalx as __dead from deraadt@ | Claudio Jeker | |
2006-04-21 | attr_init() should use attrhashsize and not pathhashsize. Found via a lint | Claudio Jeker | |
cleanup diff from deraadt@. | |||
2006-04-21 | msgbuf_writebound() and msgbuf_bounded() are leftovers from some ancient | Claudio Jeker | |
mrt dump code. These functions are no longer used since we switched to filedescriptor passing. From deraadt@ | |||
2006-04-21 | Remove function prototype that was never used. From deraadt@ | Claudio Jeker | |
2006-04-18 | cannot see the string self in the ANNOUNCE STRING prod any more since | Henning Brauer | |
self is a token now | |||
2006-04-18 | Fix "announce self" that got broken by "nexthop self". Found by Thomas Bader. | Claudio Jeker | |
OK henning@ | |||
2006-04-12 | messages on the routing socket have different headers - rt_msghdr, | Henning Brauer | |
if_msghdr, etc. they only share the first 3 fields, and then differ. this leads to a bug in RTM_IFINFO handling. We did abort on rtm->rtm_errno != 0, but if_msghdr has no errno, so we look at something in the data part instead. Surprising that this didn't bite us before! So we must only do these checks for RTM_ADD/CHANGE/DELETE that actually use rt_msghdr. found whiel checking strange behaviour tony sarendal <dualcyclone@gmail.com> saw, and then totally strange behaviour on my amd64 hackbox. claudio ok | |||
2006-04-12 | It is no longer allowed to change attributes in place. This corrupts the | Claudio Jeker | |
attribute cache. Instead remove attribute and readd it after beeing changed. This should fix the "att_diff: equal attributes encountered" error seen by Tony Sarendal. OK henning@ | |||
2006-04-05 | remote_bgpid is stored in host byte order so a htonl() is needed when putting | Claudio Jeker | |
the value on the wire. This solves the backward originator-ids seend by Tony Sarendal. OK henning@ | |||
2006-04-04 | document set nexthop self, and improve that section whil eon it, claudio ok | Henning Brauer | |
2006-04-04 | add "set nexthop self", force nexthop to be set to own address even with IBGP | Henning Brauer | |
requested & tested Falk Brockerhoff <fb@smartterra.de>, and tony sarendal tested this too. claudio ok | |||
2006-03-23 | Document nexthop qualify via (bgp|default). | Claudio Jeker | |
OK henning, additional input jmc@ | |||
2006-03-23 | Simplify the -s documentation and add /var/run/bgpd.sock to the FILES list. | Claudio Jeker | |
help from jmc@ | |||
2006-03-22 | Change the way bgpd selects nexthops. Up until now every route was considered | Claudio Jeker | |
when calculating the nexthop. Now only non BGP routes and not the default route are used unless forced with the new config options nexthop qualify via bgp nexthop qualify via default This change is required for complex setups e.g. where an additional IGP is running. OK henning@ | |||
2006-03-22 | Add a new SHOW imsg to send optional attributes to bgpctl. This can be used | Claudio Jeker | |
to show communites in bgpctl output. Only send these messages if it they are requested to reduce the overhead for simple listings. Looks good henning@ | |||
2006-03-15 | In nexthop_compare() if the two passed pointers point to the same object | Claudio Jeker | |
the odds are better than good that there is no difference. | |||
2006-03-15 | Sync usage with man page (sort arguments). | Claudio Jeker | |
2006-03-15 | Document new -s path option to select an alternate path for the main control | Claudio Jeker | |
socket. | |||
2006-03-15 | Allow the control socket to be changed on the command line. Useful if you | Claudio Jeker | |
need to run multiple bgpds on a single box to simulate a IX. This helped me massivly debugging error reports. OK henning@ | |||
2006-03-15 | Fix a memory leak when communities are set or deleted. This seems to fix | Claudio Jeker | |
Sylvain Coutant memory issues. A lot of patience and testing by Sylvain. | |||
2006-03-13 | Fix for PR 5052. Be more careful about the announced networks we accept. | Claudio Jeker | |
Make sure that at least the address family and the prefix length are sane. Reported and fix tested by Pete Vickers. | |||
2006-03-08 | Document "set community delete ..." and sync that section with the one | Claudio Jeker | |
from the PARAMETERS section. ok henning@ | |||
2006-03-07 | Fix a bug reported by Xavier Beaudouin. On config reloads set parameters | Claudio Jeker | |
inside group blocks were reset to default values. The problem was that group ids changed on reload as soon as a new peer was added to one group. Make sure that group ids remain the same over reloads a similar thing is already done for peer ids. ok henning@ | |||
2006-03-07 | "set attribute ..." in neighbor blocks needs better documentation. | Claudio Jeker | |
With input from jmc@ and ok henning@ | |||
2006-03-07 | Use 65535 instead of 0xffff to specify the maximum number that can be used | Claudio Jeker | |
for the local part of communities. | |||
2006-03-04 | Typos grab bag of the month, eyeballed by jmc@ | Miod Vallat | |
2006-02-25 | Document the softreconfig (in|out) switch. With help from jmc@ | Claudio Jeker | |
2006-02-23 | Use the new rtm_fmask feature to reset blackhole and reject routes on | Claudio Jeker | |
RTM_CHANGE. Until now it was not possible to get rid of these flags. Until now a fib decouple, fib couple combo was needed to get rid from blackholed routes. OK henning@ | |||
2006-02-10 | Make it possible to turn suftreconfig in/out on or off. Default is on for | Claudio Jeker | |
both directions. Manpage update follows. OK henning@ | |||
2006-02-09 | Implement "set community delete 65001:*" and friends. This will remove | Claudio Jeker | |
communities from the path attributes. Useful to make sure that the ones you set later are set by a (evil) peer. OK henning@ | |||
2006-02-09 | When comparing community type, cast to u_int16_t. The same is done for the | Claudio Jeker | |
AS. OK henning@ | |||
2006-02-09 | attr_free() should not modify others_len as it does not resize the others | Claudio Jeker | |
array. It just clears on entry by setting it to NULL and moving that NULL to the end of the array. With this it will be possible to remove attributes without readding them right afterwards. Because of this attr_diff() needs to be more careful because of passed NULL pointers. OK henning@ | |||
2006-02-08 | For now dump only stuff from the local-RIB into mrt table dumps. | Claudio Jeker | |
Issue found by Andre Oppermann. | |||
2006-02-08 | fix logic both in the decision when to re-init the capabilities negotiation | Henning Brauer | |
structures | |||
2006-02-05 | tweaks; ok claudio | Jason McIntyre | |
2006-02-04 | Document the new special community part "neighbor-as". OK henning@ | Claudio Jeker | |
2006-02-03 | If the aspath gets copied because it is modified use the copy for filter | Claudio Jeker | |
matching. With this it is possible to make later filters depend on previous filter changes -- e.g. via community attributes. | |||
2006-02-03 | Doh! Fix stupid copy paste error. Setting the community type to the AS instead | Claudio Jeker | |
of the more obvious type produces really funny errors and gives you some happy hours of debugging. | |||
2006-02-02 | Implement new special community "neighbor-as". neighbor-as is expanded on | Claudio Jeker | |
the fly to the remote AS of the current neighbor. This can be used to simplify rulesets in a dramatic way -- going from a script based nightmare down to a handfull rules. jajajaja henning@ | |||
2006-01-31 | Dynamic routes (PMTU, etc) are not redistributable even for IPv6. | Claudio Jeker | |
Sure henning@ | |||
2006-01-24 | add -r to synopsis, and tweak its description a little; | Jason McIntyre | |
2006-01-24 | introduce "bgpctl show summary terse", shows summary in an easy to parse | Henning Brauer | |
format, intended for monitoring puposes. claudio ok | |||
2006-01-24 | bgpd does not and will never support route flap damping as defined in the | Claudio Jeker | |
mentioned RFC. Even RIPE (RIPE-229) realized that route flap damping should be considered evil. | |||
2006-01-24 | Last bits for softreconfig in support. Now bgpd will automaticaly rei-filter | Claudio Jeker | |
the RIB after a reload so you no longer need to clear sessions because you modified filters. Looks good henning@. | |||
2006-01-24 | Functions in the poll() loop should only be moved around if there are no | Claudio Jeker | |
side-effects. Revert last changes and make bgpctl reload work again. | |||
2006-01-24 | Check if filter changed on a per peer basis. This should speed up the | Claudio Jeker | |
table run done later as many filter evaluations can be skipped. From the softreconfig in tree. Looks good henning@ |