[ovs-dev] [PATCH ovn 1/7] controller: Flush the netlink notifier message when we disable it.

[Ales Musil via dev]

The netlink notifier could retain messages even after being disabled
which could cause problems when it would be re-enabled. Make sure
we flush them.

Fixes: 5ec13ae6eff6 ("controller: Consolidate the netlink notifiers.")
Signed-off-by: Ales Musil <amu...@redhat.com>
---
 controller/ovn-netlink-notifier.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/controller/ovn-netlink-notifier.c 
b/controller/ovn-netlink-notifier.c
index 7d44f8765..ff0d01ca7 100644
--- a/controller/ovn-netlink-notifier.c
+++ b/controller/ovn-netlink-notifier.c
@@ -217,6 +217,8 @@ ovn_netlink_deregister_notifier(enum 
ovn_netlink_notifier_type type)
         nln_destroy(nln_handle);
         nln_handle = NULL;
     }
+
+    ovn_netlink_notifier_flush(type);
 }
 
 void
@@ -287,7 +289,6 @@ void
 ovn_netlink_notifiers_destroy(void)
 {
     for (size_t i = 0; i < OVN_NL_NOTIFIER_MAX; i++) {
-        ovn_netlink_notifier_flush(i);
         ovn_netlink_deregister_notifier(i);
         vector_destroy(&notifiers[i].msgs);
     }
-- 
2.53.0

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev