The `op` here is a new port structure that is not, yet, attached
anywhere, and the attempt to detach it will result in undefined
behavior.

Signed-off-by: Ihar Hrachyshka <ihrac...@redhat.com>
---
 northd/northd.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/northd/northd.c b/northd/northd.c
index ace663d54..3d2715911 100644
--- a/northd/northd.c
+++ b/northd/northd.c
@@ -4346,7 +4346,6 @@ ls_port_create(struct ovsdb_idl_txn *ovnsb_txn, struct 
hmap *ls_ports,
         if (op->sb) {
             sbrec_port_binding_delete(op->sb);
         }
-        ovs_list_remove(&op->list);
         ovn_port_destroy(ls_ports, op);
         return NULL;
     }
-- 
2.41.0

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

Reply via email to