diff options
| author | paul <paul> | 2005-05-18 23:29:57 +0000 |
|---|---|---|
| committer | paul <paul> | 2005-05-18 23:29:57 +0000 |
| commit | a3387a4488c35d322d3c78973c967b2be5967b6e (patch) | |
| tree | 678da23a64c1bb41c2c6c87b362f62f9db22de03 /ospfd/ChangeLog | |
| parent | 024a7f06979b467299a85e915b78af456a89e3a6 (diff) | |
2005-05-19 Paul Jakma <paul.jakma@sun.com>
* ospf_interface.c: (ospf_if_table_lookup) Fix a serious bug
a less serious one.
1: this function is supposed to lookup
entries in the oifs ospf_interface route_table and return either
an existing oi or NULL to indicate not found, its caller depends
on this, yet this function uses route_node_get which /always/
returns a route_node - one is created if none exists. Use
route_node_lookup instead. This should fix root cause of the
reports of the (ospf_add_to_if) assert being hit.
2: oi's are inserted into this table with prefixlength set to
/32 (indeed, it should be a hash table, not a route_table),
however prefixlength to lookup was not changed, if no valid entry
can be inserted other than /32, then nothng but /32 should be
looked up. This possibly only worked by fluke..
Fix confirmed by 2 reporters (one list, one IRC), definitely a
backport candidate once it has been incubated in HEAD for a while.
Thanks to Patrick Friedel and Ivan Warren for testing.
Diffstat (limited to 'ospfd/ChangeLog')
| -rw-r--r-- | ospfd/ChangeLog | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/ospfd/ChangeLog b/ospfd/ChangeLog index 774658ce..ec6608ba 100644 --- a/ospfd/ChangeLog +++ b/ospfd/ChangeLog @@ -1,3 +1,23 @@ +2005-05-19 Paul Jakma <paul.jakma@sun.com> + + * ospf_interface.c: (ospf_if_table_lookup) Fix a serious bug + a less serious one. + 1: this function is supposed to lookup + entries in the oifs ospf_interface route_table and return either + an existing oi or NULL to indicate not found, its caller depends + on this, yet this function uses route_node_get which /always/ + returns a route_node - one is created if none exists. Use + route_node_lookup instead. This should fix root cause of the + reports of the (ospf_add_to_if) assert being hit. + 2: oi's are inserted into this table with prefixlength set to + /32 (indeed, it should be a hash table, not a route_table), + however prefixlength to lookup was not changed, if no valid entry + can be inserted other than /32, then nothng but /32 should be + looked up. This possibly only worked by fluke.. + Fix confirmed by 2 reporters (one list, one IRC), definitely a + backport candidate once it has been incubated in HEAD for a while. + Thanks to Patrick Friedel and Ivan Warren for testing. + 2005-05-11 Paul Jakma <paul.jakma@sun.com> * (general) Fix memory leaks in opaque AS-scope LSAs, reported and |