matthieu/frr
1
0
Fork 0
forked from Mirror/frr
Code Pull requests Activity

fixes an assert which doesnt seem to take into account that route_node_get

Browse source 
may return an existing node. (if the code wants a /new/ node why not use
route_node_set? if it doesnt mind - then the assert is wrong).

this bug is in zebra.org CVS. (must be an extremely rare/unlikely bug
though).
This commit is contained in:
paul 2003-02-18 23:25:44 +00:00
parent 4955a0ba43
commit 8c80cb7ee9
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
ospfd/ospf_interface.c
View file

@ -135,7 +135,10 @@ ospf_add_to_if (struct interface *ifp, struct ospf_interface *oi)
p.prefixlen = IPV4_MAX_PREFIXLEN; p.prefixlen = IPV4_MAX_PREFIXLEN;
rn = route_node_get (IF_OIFS (ifp), &p); rn = route_node_get (IF_OIFS (ifp), &p);
assert (! rn->info); /* rn->info should either be NULL or equal to this oi
* as route_node_get may return an existing node
*/
assert (! rn->info || rn->info == oi);
rn->info = oi; rn->info = oi;
} }