diff options
author | Paul Jakma <paul.jakma@sun.com> | 2007-02-26 17:14:48 +0000 |
---|---|---|
committer | Paul Jakma <paul.jakma@sun.com> | 2007-02-26 17:14:48 +0000 |
commit | bd34fb346db5bb1f0bc8eeeef1868e296d889053 (patch) | |
tree | d29904361a06e21d77fc5cbb163c39dd327962fe /ospfd/ChangeLog | |
parent | ff1dd550b01fd6bb4aa137cfee282e6175f89c50 (diff) |
[ospfd] Fix regression in SPF introduced by bug#330 fixes
2007-02-26 Paul Jakma <paul.jakma@sun.com>
* ospf_spf.c: Fix regression introduced with bug #330 fix: The
cost update added to ospf_spf_add_parent only handled PtP
case, differing from same functionality in higher-level
ospf_spf_next. Regression diagnosed by Anders Pedersen,
mailnews+router-quagga-dev@news.cohaesio.com.
(ospf_vertex_new) Initialise vertices to max-cost.
(ospf_spf_init) Root vertex always creates with 0 cost.
(ospf_spf_add_parent) Remove the buggy V->W cost calculating
code, instead take the new distance as a parameter.
(ospf_nexthop_calculation) Take distance as parameter, so it
can be passed down to add_parent.
(ospf_spf_next) Dont initialise candiate vertex distance,
vertex_new does so already. Pass distance down to
nexthop_calculation (see above).
Diffstat (limited to 'ospfd/ChangeLog')
-rw-r--r-- | ospfd/ChangeLog | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/ospfd/ChangeLog b/ospfd/ChangeLog index d026bf88..35ffd69d 100644 --- a/ospfd/ChangeLog +++ b/ospfd/ChangeLog @@ -1,3 +1,20 @@ +2007-02-26 Paul Jakma <paul.jakma@sun.com> + + * ospf_spf.c: Fix regression introduced with bug #330 fix: The + cost update added to ospf_spf_add_parent only handled PtP + case, differing from same functionality in higher-level + ospf_spf_next. Regression diagnosed by Anders Pedersen, + mailnews+router-quagga-dev@news.cohaesio.com. + (ospf_vertex_new) Initialise vertices to max-cost. + (ospf_spf_init) Root vertex always creates with 0 cost. + (ospf_spf_add_parent) Remove the buggy V->W cost calculating + code, instead take the new distance as a parameter. + (ospf_nexthop_calculation) Take distance as parameter, so it + can be passed down to add_parent. + (ospf_spf_next) Dont initialise candiate vertex distance, + vertex_new does so already. Pass distance down to + nexthop_calculation (see above). + 2007-01-24 Paul Jakma <paul.jakma@sun.com> * ospf_spf.c: Bug #330: Nexthop calculation sometimes may fail, |