diff options
| author | hasso <hasso> | 2004-03-18 19:18:33 +0000 |
|---|---|---|
| committer | hasso <hasso> | 2004-03-18 19:18:33 +0000 |
| commit | 0d85b9958d6db05fb9d81afbee96332f142654bd (patch) | |
| tree | fb0f35cb44254bdfd9b2d907800b1ec513bfc232 /ospfd/ospf_opaque.c | |
| parent | f1d92e17ab5f1475018e353ce72e94ebe7a81a44 (diff) | |
Amir's fix case where opaque capability is switched "ON -> OFF -> ON".
[quagga-dev 843]
Diffstat (limited to 'ospfd/ospf_opaque.c')
| -rw-r--r-- | ospfd/ospf_opaque.c | 27 |
1 files changed, 21 insertions, 6 deletions
diff --git a/ospfd/ospf_opaque.c b/ospfd/ospf_opaque.c index f1fe783c..636839c0 100644 --- a/ospfd/ospf_opaque.c +++ b/ospfd/ospf_opaque.c @@ -1385,10 +1385,15 @@ ospf_opaque_lsa_originate_schedule (struct ospf_interface *oi, int *delay0) { for (node = listhead (oi->opaque_lsa_self); node; nextnode (node)) { + /* + * removed the test for + * (! list_isempty (oipt->id_list)) * Handler is already active. * + * because opaque cababilities ON -> OFF -> ON result in list_isempty (oipt->id_list) + * not being empty. + */ if ((oipt = getdata (node)) == NULL /* Something wrong? */ || oipt->t_opaque_lsa_self != NULL /* Waiting for a thread call. */ - || oipt->status == PROC_SUSPEND /* Cannot originate now. */ - || ! list_isempty (oipt->id_list)) /* Handler is already active. */ + || oipt->status == PROC_SUSPEND) /* Cannot originate now. */ continue; ospf_opaque_lsa_reoriginate_schedule ((void *) oi, @@ -1401,10 +1406,15 @@ ospf_opaque_lsa_originate_schedule (struct ospf_interface *oi, int *delay0) { for (node = listhead (area->opaque_lsa_self); node; nextnode (node)) { + /* + * removed the test for + * (! list_isempty (oipt->id_list)) * Handler is already active. * + * because opaque cababilities ON -> OFF -> ON result in list_isempty (oipt->id_list) + * not being empty. + */ if ((oipt = getdata (node)) == NULL /* Something wrong? */ || oipt->t_opaque_lsa_self != NULL /* Waiting for a thread call. */ - || oipt->status == PROC_SUSPEND /* Cannot originate now. */ - || ! list_isempty (oipt->id_list)) /* Handler is already active. */ + || oipt->status == PROC_SUSPEND) /* Cannot originate now. */ continue; ospf_opaque_lsa_reoriginate_schedule ((void *) area, @@ -1417,10 +1427,15 @@ ospf_opaque_lsa_originate_schedule (struct ospf_interface *oi, int *delay0) { for (node = listhead (top->opaque_lsa_self); node; nextnode (node)) { + /* + * removed the test for + * (! list_isempty (oipt->id_list)) * Handler is already active. * + * because opaque cababilities ON -> OFF -> ON result in list_isempty (oipt->id_list) + * not being empty. + */ if ((oipt = getdata (node)) == NULL /* Something wrong? */ || oipt->t_opaque_lsa_self != NULL /* Waiting for a thread call. */ - || oipt->status == PROC_SUSPEND /* Cannot originate now. */ - || ! list_isempty (oipt->id_list)) /* Handler is already active. */ + || oipt->status == PROC_SUSPEND) /* Cannot originate now. */ continue; ospf_opaque_lsa_reoriginate_schedule ((void *) top, |