2015-09-25 18:19:44 |
Assaf Muller |
description |
If OVSBridge.get_ports_attributes is executed on an empty bridge, get_port_name_list will return an empty string. In which case, ovsdb.db_list is executed with port_names = '', meaning that it will return all ports, instead of no ports.
The implication is that if, for example, br-ex (An ancillary bridge in the OVS agent) is currently empty, then scan_ancillary_ports will pick up all ports. All ports on the system will be considered ancillary_ports, which is unexpected and can result in ports going DOWN when they shouldn't. |
If OVSBridge.get_ports_attributes is executed on an empty bridge, get_port_name_list will return an empty list. In which case, ovsdb.db_list is executed with port_names = [], meaning that it will return all ports, instead of no ports.
The implication is that if, for example, br-ex (An ancillary bridge in the OVS agent) is currently empty, then scan_ancillary_ports will pick up all ports. All ports on the system will be considered ancillary_ports, which is unexpected and can result in ports going DOWN when they shouldn't. |
|