diff options
author | Chris Caputo <ccaputo@alt.net> | 2009-06-23 06:06:49 +0000 |
---|---|---|
committer | Paul Jakma <paul@quagga.net> | 2009-06-23 14:25:20 +0100 |
commit | 50aef6f3b0e36b56cfa9f3d374be0c1d25c30c45 (patch) | |
tree | 5dce97de6229713c6c209cbb72abc1093fa1be58 /bgpd/bgp_damp.h | |
parent | 6e79f8bba438823d84b7464a9acc1e6fc9126a27 (diff) |
Fix "show ip bgp dampened-paths" garbage output.
* bgpd/bgp_damp.c: Make bgp_damp_reuse_time_vty() accept a buffer and
length, rather than returning a local var buffer whose contents can get
trounced. Remove duplicate BGP_UPTIME_LEN define.
* bgpd/bgp_damp.h: bgp_damp_reuse_time_vty() prototype change.
* bgpd/bgp_route.c: Provide bgp_damp_reuse_time_vty() with a buffer and
length. Remove duplicate BGP_UPTIME_LEN define.
This problem was noticed in 2005...
http://hibernia.jakma.org/~paul/patches/quagga-test.diff
...but the fix didn't make it into the code.
Signed-off-by: Chris Caputo <ccaputo@alt.net>
Diffstat (limited to 'bgpd/bgp_damp.h')
-rw-r--r-- | bgpd/bgp_damp.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/bgpd/bgp_damp.h b/bgpd/bgp_damp.h index e0bef4a8..e1d319b5 100644 --- a/bgpd/bgp_damp.h +++ b/bgpd/bgp_damp.h @@ -141,6 +141,7 @@ extern void bgp_damp_info_clean (void); extern int bgp_damp_decay (time_t, int); extern void bgp_config_write_damp (struct vty *); extern void bgp_damp_info_vty (struct vty *, struct bgp_info *); -extern const char * bgp_damp_reuse_time_vty (struct vty *, struct bgp_info *); +extern const char * bgp_damp_reuse_time_vty (struct vty *, struct bgp_info *, + char *, size_t); #endif /* _QUAGGA_BGP_DAMP_H */ |