diff options
author | Paul Jakma <paul.jakma@sun.com> | 2007-08-30 13:36:17 +0000 |
---|---|---|
committer | Paul Jakma <paul.jakma@sun.com> | 2007-08-30 13:36:17 +0000 |
commit | 9eda90ce8094683a5315007fbd0f9249a284f36f (patch) | |
tree | 1d861050148c03dc4c37a1266c4567b24e9e6da1 /watchquagga/watchquagga.c | |
parent | 6d134fb4defecb16591adbf4acb020acd165a75a (diff) |
[bgpd] bug #398 Bogus free on out route-map, and assert() with rsclients
2007-08-27 Paul Jakma <paul.jakma@sun.com>
* bgp_route.c: (bgp_announce_check) Fix bug #398, slight
modification of Vladimir Ivanov's suggested fix - to keep
memory alloc conditional.
(bgp_process_announce_selected) Don't take struct attr as
argument, none of the callers need it and it needlessly
distances allocation from use.
Free the extended attr, the attr itself is on the stack.
Fix bad indentation.
* bgp_attr.c: (bgp_packet_attribute) Remove incorrect assert,
and adjust conditional to test attr->extra, diagnosis by
Vladimir Ivanov in bug #398.
2007-08-27 Vladimir Ivanov <wawa@yandex-team.ru>
* bgp_route.c: (bgp_announce_check_rsclient) copy of
ri->attr is no longer deep enough, due to addition of
attr->extra. It should use bgp_attr_dup, as
bgp_announce_check() does.
Diffstat (limited to 'watchquagga/watchquagga.c')
0 files changed, 0 insertions, 0 deletions