diff options
| author | paul <paul> | 2003-02-18 23:25:44 +0000 |
|---|---|---|
| committer | paul <paul> | 2003-02-18 23:25:44 +0000 |
| commit | 058739a66b601ecc5f834869a64f51e2253a1e27 (patch) | |
| tree | d8a2f673ecb18a853e026a08ff10a95709e029fb /ospfd/ospf_interface.c | |
| parent | 8287f3f5626a975332423c610122bb2b73e1daba (diff) | |
| download | quagga-058739a66b601ecc5f834869a64f51e2253a1e27.tar.bz2 quagga-058739a66b601ecc5f834869a64f51e2253a1e27.tar.xz | |
fixes an assert which doesnt seem to take into account that route_node_get
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).
Diffstat (limited to 'ospfd/ospf_interface.c')
| -rw-r--r-- | ospfd/ospf_interface.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/ospfd/ospf_interface.c b/ospfd/ospf_interface.c index f0300273..d94cdb8b 100644 --- a/ospfd/ospf_interface.c +++ b/ospfd/ospf_interface.c @@ -135,7 +135,10 @@ ospf_add_to_if (struct interface *ifp, struct ospf_interface *oi) p.prefixlen = IPV4_MAX_PREFIXLEN; 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; } |