[Xorp-cvs] XORP cvs commit: xorp/bgp

Mark Handley M.Handley@cs.ucl.ac.uk
Tue, 28 Jun 2005 12:36:15 +0100


>On Tue, Jun 28, 2005 at 09:30:16AM +0000, Mark Handley wrote:
>> Log message:
>> 	Fix bug in RibOut/PeerHandler code, where we assumed all routes being
>> 	pushed together came from the same origin peer.  All routes being
>> 	added do, but routes being deleted as part of a replace don't need to
>> 	come from the same place.  The upshot was that we'd sometimes delete
>> 	or add to the wrong OriginTable in the RIB, and subsequently not be
>> 	able to remove the route again.
>
>Good catch, this one has been tripping us up for a while! Glad to see it
>bite the dust.

Not my catch - Atanu found the problem.  All I did was fix my broken code.

 - Mark