Nest: Prevent withdraws from propagation back to source protocol

The earlier fix loosen conditions for not running filters on old
route when deciding about route propagation to a protocol to avoid
issues with ghost routes in some race conditions.

Unfortunately, the fix also caused back-propagation of withdraws. For
regular updates, back-propagation is prevented in import_control hooks,
but these are not called on withdraws. For them, import_control hooks
are called on old routes instead, changing (old, NULL) notification
to (NULL, NULL), which is ignored. By not calling export processing
in some cases, the withdraw is not ignored and is back-propagated.

This patch fixes that by contract conditions so the earlier fix is not
applied to back-propagated updates.

diff --git a/nest/rt-table.c b/nest/rt-table.c
index ae4bb74d1b536c20daabaadc2cc047192add65d8..9952d91ad38a01774f70705a2d593df23728ca63 100644 (file)
--- a/nest/rt-table.c
+++ b/nest/rt-table.c
@@ -571,7 +571,8 @@ rt_notify_basic(struct channel *c, net *net, rte *new0, rte *old0, int refeed)
   if (new)
     new = export_filter(c, new, &new_free, 0);
 
-  if (old && !(refeed || (old->lastmod <= c->last_tx_filter_change)))
+  if (old && !(refeed || ((old->lastmod <= c->last_tx_filter_change) &&
+                         (p != old->sender->proto))))
     old = export_filter(c, old, &old_free, 1);
 
   if (!new && !old)