Browse Source

2004-07-27 Paul Jakma <paul@dishone.st>

        * ospf_packet.c: (ospf_ls_upd_send_queue_event) fix thinko from
          last fix for ospfd wedging due to oversize LSAs: dont list loop on
          ospf_ls_upd_queue_send() - guaranteed segfault.
paul 16 years ago
parent
commit
48fe13bfd7
2 changed files with 7 additions and 5 deletions
  1. 6 0
      ospfd/ChangeLog
  2. 1 5
      ospfd/ospf_packet.c

+ 6 - 0
ospfd/ChangeLog

@@ -1,5 +1,11 @@
 2004-07-27 Paul Jakma <paul@dishone.st>
 
+        * ospf_packet.c: (ospf_ls_upd_send_queue_event) fix thinko from
+          last fix for ospfd wedging due to oversize LSAs: dont list loop on
+          ospf_ls_upd_queue_send() - guaranteed segfault.
+          
+2004-07-27 Paul Jakma <paul@dishone.st>
+
 	* ospf_opaque.c: (ospf_opaque_lsa_flush_schedule) do not NULL out
 	  the LSA as then free_opaque_info_per_id() can never unlock (and
 	  free) the LSA. Reported by Gunnar Stigen.

+ 1 - 5
ospfd/ospf_packet.c

@@ -3123,11 +3123,7 @@ ospf_ls_upd_send_queue_event (struct thread *thread)
       if (rn->info == NULL)
         continue;
 
-      for (tn = update->head; tn; tn = nn)
-        {
-          nn = tn->next;
-          ospf_ls_upd_queue_send (oi, update, rn->p.u.prefix4);
-        }
+      ospf_ls_upd_queue_send (oi, update, rn->p.u.prefix4);
       
       /* list might not be empty.
        * TODO: work out what to do about oversized LSAs.