fdo-mirrors/NetworkManager
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git synced 2026-01-09 06:30:21 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

ovs: rename variable

Browse source 
@interface_is_internal is a bad name. The variable indicates whether
the interface is the local interface.
This commit is contained in:
Beniamino Galvani 2020-11-24 13:56:12 +01:00
parent b0c018830e
commit e9e99b8677
1 changed files with 9 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
src/devices/ovs/nm-ovsdb.c
View file

@ -992,7 +992,7 @@ _add_interface(NMOvsdb * self,
nm_auto_decref_json json_t *interfaces = NULL;
nm_auto_decref_json json_t *new_interfaces = NULL;
gboolean has_interface = FALSE;
gboolean interface_is_internal;
gboolean interface_is_local;
gs_free char * bridge_cloned_mac = NULL;
gs_free char * interface_cloned_mac = NULL;
GError * error = NULL;
@ -1006,10 +1006,10 @@ _add_interface(NMOvsdb * self,
new_ports = json_array();
new_interfaces = json_array();
bridge_name = nm_connection_get_interface_name(bridge);
port_name = nm_connection_get_interface_name(port);
interface_name = nm_connection_get_interface_name(interface);
interface_is_internal = nm_streq0(bridge_name, interface_name);
bridge_name = nm_connection_get_interface_name(bridge);
port_name = nm_connection_get_interface_name(port);
interface_name = nm_connection_get_interface_name(interface);
interface_is_local = nm_streq0(bridge_name, interface_name);
/* Determine cloned MAC addresses */
if (!nm_device_hw_addr_get_cloned(bridge_device,
@ -1018,7 +1018,7 @@ _add_interface(NMOvsdb * self,
&bridge_cloned_mac,
NULL,
&error)) {
_LOGW("Cannot determine cloned mac for OVS %s '%s': %s",
_LOGW("Cannot determine cloned MAC for OVS %s '%s': %s",
"bridge",
bridge_name,
error->message);
@ -1031,14 +1031,14 @@ _add_interface(NMOvsdb * self,
&interface_cloned_mac,
NULL,
&error)) {
_LOGW("Cannot determine cloned mac for OVS %s '%s': %s",
_LOGW("Cannot determine cloned MAC for OVS %s '%s': %s",
"interface",
interface_name,
error->message);
g_clear_error(&error);
}
if (interface_is_internal && !bridge_cloned_mac && interface_cloned_mac) {
if (interface_is_local && !bridge_cloned_mac && interface_cloned_mac) {
_LOGT("'%s' is a local ovs-interface, the MAC will be set on ovs-bridge '%s'",
interface_name,
bridge_name);
@ -1108,7 +1108,7 @@ _add_interface(NMOvsdb * self,
g_return_if_fail(ovs_bridge);
_expect_bridge_ports(params, ovs_bridge->name, ports);
_set_bridge_ports(params, bridge_name, new_ports);
if (bridge_cloned_mac && interface_is_internal)
if (bridge_cloned_mac && interface_is_local)
_set_bridge_mac(params, bridge_name, bridge_cloned_mac);
}