diff options
author | Chris Caputo <ccaputo@alt.net> | 2009-06-23 08:06:49 +0200 |
---|---|---|
committer | Paul Jakma <paul@quagga.net> | 2009-06-23 15:25:20 +0200 |
commit | 50aef6f3b0e36b56cfa9f3d374be0c1d25c30c45 (patch) | |
tree | 5dce97de6229713c6c209cbb72abc1093fa1be58 /bgpd/bgp_damp.h | |
parent | Fix vtysh based "write term" output. (diff) | |
download | frr-50aef6f3b0e36b56cfa9f3d374be0c1d25c30c45.tar.xz frr-50aef6f3b0e36b56cfa9f3d374be0c1d25c30c45.zip |
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 e0bef4a8f..e1d319b56 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 */ |